rubyでAPIを叩きたい。

Question

前提・実現したいこと

叩いたJSON形式のAPIをrubyで取得したいです。

発生している問題・エラーメッセージ

APIを」叩きurlを所得したのですがその部分でnil class
とエラーが出ており困っております。

エラーメッセージI, [2019-10-03T00:56:21.711041 #4]  INFO -- : [eb3f8fb8-27c3-413a-93cf-c2b167063968] Completed 500 Internal Server Error in 922ms
2019-10-03T00:56:21.711691+00:00 app[web.1]: F, [2019-10-03T00:56:21.711621 #4] FATAL -- : [eb3f8fb8-27c3-413a-93cf-c2b167063968]
2019-10-03T00:56:21.711762+00:00 app[web.1]: F, [2019-10-03T00:56:21.711705 #4] FATAL -- : [eb3f8fb8-27c3-413a-93cf-c2b167063968] NoMethodError (undefined method `sample' for nil:NilClass):
2019-10-03T00:56:21.711817+00:00 app[web.1]: F, [2019-10-03T00:56:21.711765 #4] FATAL -- : [eb3f8fb8-27c3-413a-93cf-c2b167063968]
2019-10-03T00:56:21.711880+00:00 app[web.1]: F, [2019-10-03T00:56:21.711829 #4] FATAL -- : [eb3f8fb8-27c3-413a-93cf-c2b167063968] app/controllers/linebot_controller.rb:48:in `block in callback'
2019-10-03T00:56:21.711881+00:00 app[web.1]: [eb3f8fb8-27c3-413a-93cf-c2b167063968] app/controllers/linebot_controller.rb:28:in `each'
2019-10-03T00:56:21.711883+00:00 app[web.1]: [eb3f8fb8-27c3-413a-93cf-c2b167063968] app/controllers/linebot_controller.rb:28:in `callback'
sasaotookinoAir:ra-mennapp sasaotoyoki$ 

該当のソースコード

events.each { |event|
      if event.message['text'] != nil
        place = event.message['text'] #ここでLINEで送った文章を取得
        result = `curl -X GET https://api.gnavi.co.jp/RestSearchAPI/v3/?keyid=78d2d49f3aa8747a6cc03da01cf41bdd&category_s=RSFST08008&category_s=RSFST08009&#{place}`#ここでぐるなびAPIを叩く
      else
        latitude = event.message['latitude']
        longitude = event.message['longitude']

        result = `curl -X GET https://api.gnavi.co.jp/RestSearchAPI/v3/?keyid=78d2d49f3aa8747a6cc03da01cf41bdd&category_s=RSFST08008category_s=RSFST08009&latitude=#{latitude}longitude=#{longitude}`  #ここでぐるなびAPIを叩く
      end
      
      hash_result = JSON.parse result  #レスポンスが文字列なのでhashにパースする
      shops = hash_result["rest"] #ここでお店情報が入った配列となる
      shop = shops.sample #任意のものを一個選ぶ

全体ソースコード

class LinebotController < ApplicationController
  require 'line/bot' # gem 'line-bot-api'
  require "json"
  

  # callbackアクションのCSRFトークン認証を無効
  protect_from_forgery :except => [:callback]

  def client
    @client ||= Line::Bot::Client.new { |config|
      config.channel_secret = ENV["LINE_CHANNEL_SECRET"]
      config.channel_token = ENV["LINE_CHANNEL_TOKEN"]
    }
  end

  def callback
    body = request.body.read

    signature = request.env['HTTP_X_LINE_SIGNATURE']
    unless client.validate_signature(body, signature)
      error 400 do 'Bad Request' end
    end

    events = client.parse_events_from(body)

    #ここでlineに送られたイベントを検出している
    # messageのtext: に指定すると、返信する文字を決定することができる
    #event.message['text']で送られたメッセージを取得することができる
    events.each { |event|
      if event.message['text'] != nil
        place = event.message['text']  #ここでLINEで送った文章を取得
        result = `curl -X GET https://api.gnavi.co.jp/RestSearchAPI/v3/?keyid=78d2d49f3aa8747a6cc03da01cf41bdd&category_s=RSFST08008&category_s=RSFST08009&#{place}`#ここでぐるなびAPIを叩く#{place}
      else
        latitude = event.message['latitude']
        longitude = event.message['longitude']
        puts event.message['latitude']
        puts event.message['longitude']
        puts event.message['latitude'].class
        result = `curl -X GET https://api.gnavi.co.jp/RestSearchAPI/v3/?keyid=78d2d49f3aa8747a6cc03da01cf41bdd&category_s=RSFST08008&category_s=RSFST08009&latitude=#{latitude}&longitude=#{longitude}`#ここでぐるなびAPIを叩くlatitude=#{latitude}longitude=#{longitude}
      end
      hash_result = JSON.parse(result)#レスポンスが文字列なのでhashにパースする
      shops = hash_result["rest"] #ここでお店情報が入った配列となる
      shop = shops.sample #任意のものを一個選ぶ
      puts shop

      店の情報
      url = shop["url_mobile"] #サイトのURLを送る
      shop_name = shop["name"] #店の名前
      category = shop["category"] #カテゴリー
      open_time = shop["opentime"] #空いている時間
      holiday = shop["holiday"] #定休日

      if open_time.class != String #空いている時間と定休日の二つは空白の時にHashで返ってくるので、文字列に直そうとするとエラーになる。そのため、クラスによる場合分け。
        open_time = ""
     end
     if holiday.class != String
        holiday = ""
      end

      response = "【店名】" + shop_name + "\n" + "【カテゴリー】" + category + "\n" + "【営業時間と定休日】" + open_time + "\n" + holiday + "\n" + url
       case event #case文　caseの値がwhenと一致する時にwhenの中の文章が実行される(switch文みたいなもの)
      when Line::Bot::Event::Message
        case event.type
        when Line::Bot::Event::MessageType::Text,Line::Bot::Event::MessageType::Location
          message = {
            type: 'text',
            text: response
          }
          client.reply_message(event['replyToken'], message)
        end

      end
    } 

    head :ok
  end
end

試したこと

NET::htmlを試したり、このサイトのapi関連の記事を読み自分なりに実行してみましたが解決せず。

補足情報

hash_result = JSON.parse result までは値が入っているのですが、shopsにnilが出てしまいます。
文法的なミスを探したのですが、特に該当しそうな部分がなく質問させていただきました。

大変稚拙な質問で恐縮なのですがお答えいただけると幸いです。

Answer 1

この質問だけ見た場合

keyidがテスト用なので、おそらく使えない
変数placeに入る文字列が不明だが、相当面倒くさい

おそらく、hash_result["error"]["message"]にエラーメッセージが入っているので表示してみてはいかがでしょうか？

Comments

[sekimori0802]

拙い質問にお答えいただきありがとうございます。ぐるなびキーは登録時のものを使用し、エラーメッセージの方確認してみます。

Answer 2

shopsがnilということなので、hash_resultが期待通りの内容でないということです。
よく見ましょう。（としか言いようがない。情報がないので）

#追記
あと、

result = `curl -X GET https://～～～`
```のようなGETメソッドであれば、
```Ruby
require "open-uri"

result = open("https://～～～",&:read)
```と、外部コマンドを呼ばないで書けます。

Comments

[sekimori0802]

わかりづらい質問にご丁寧にお答えいただきありがとうございます。json型で取ってきたapiのコードをハッシュに変換出来ていないみたいなのですが、調べてみます。

[otn]

JSON.parse で、エラーが出ていないので、JSON形式の文字列が得られていることは大丈夫です。
おそらくエラーを通知するデータになっていて、"rest"のキーがないのでしょう。

hash_result = JSON.parse result の前に、
p result
を入れてみるという発想が出てくるようにしましょう。