楽天API フォームで検索させたい＆見つからなかった場合にエラーがでてしまう。

Question

現在、railsにてゴルフスコアアプリを制作しており、楽天APIからゴルフ場情報を取得するところまではできました。ただ現状、下記コードのようにコントローラに検索用のキーワードをコントローラに記述しております。
やりたいこととしては、
１.コントローラに記述している部分をフォームにしたい。
具体的にはtodoufukenという変数にフォームの値を入れたい。
調べてはみたのですが、form_forを使えばよいのかなとは思っているのですが、どのようにコントローラ側に値を渡したら良いかわかりません。
また、このようなケースは、JSをつかうべきでしょうか。

2.該当なしの場合、エラーを出さないように、「該当なし」等の表示にしたい。
該当なしの場合、画像のようなエラーが出てしまいます。
どのように回避したらよいかわからない状態です。

class CoursesController < ApplicationController
  def index
    require 'net/http'
    require 'json'
    require 'uri'
    
    base_url = "https://app.rakuten.co.jp/services/api/Gora/GoraGolfCourseSearch/20170623"
 
    todoufuken = 8
      params = {
        'keyword'       => URI.encode('ワイルドダック'),
        'format'        => 'json',
        'applicationId' => '123456789',
        'areaCode'      => todoufuken,
        'hits'          => 10
      }
      uri = URI(base_url + '?' + params.map{|k,v| "#{k}=#{v}"}.join('&'))

    
    begin
      response = Net::HTTP.start(uri.host, uri.port, use_ssl: uri.scheme == 'https') do |http|
        http.open_timeout = 5
        http.read_timeout = 10
        http.get(uri.request_uri)
      end

      case response
        when Net::HTTPSuccess
          @json = JSON.parse(response.body)
         
      end

    rescue => e
      Rails.logger.error(e.message)
      raise e
    end
  end
  
  def show
    @score = Score.find(params[:id])
  end
 

end

宜しくお願い致します。

以下、追記
view

 <table>
   <tr>
     <th>Name</th>
   </tr>
     
　<% if @json['items'].nil? %>
   <tr>
    <th>該当データなし！</th>
   </tr>           
  <% else %>
    <% @json["Items"].each do|item| %>
    <tr>
     <td>
       <%= item["Item"]["golfCourseName"] %>
     </td>
    </tr>
   <% end %>
 <% end %>
</table>

Answer 1

１.コントローラに記述している部分をフォームにしたい。
具体的にはtodoufukenという変数にフォームの値を入れたい。
調べてはみたのですが、form_forを使えばよいのかなとは思っているのですが、どのようにコントローラ側に値を渡したら良いかわかりません。

一番簡単な実装としてはクエリパラメータ（ URLの?に続くやつです ）に追加する感じかと思います。

• まずは controllerでクエリパラメーターを受け取れるようにする
  • http://localhost:3000/xxx?todoufuken='東京都'
• クエリパラメーターでの検索結果をViewに表示できるようにする
• Viewにクエリパラメーターでの画面にアクセスできるようにリンクを作成する

の手順が良いかと思います。

また、このようなケースは、JSをつかうべきでしょうか。

画面遷移があっても良いのであれば、JSなしでも良いのではないでしょうか。
JSで処理するのって、だいぶハードルが上がると思うので。
もしやろうとするのであれば、まずはRailsだけで完結したものを実装できるようになってから、次のステップでやってみるのが良いかと思います。

2.該当なしの場合、エラーを出さないように、「該当なし」等の表示にしたい。
該当なしの場合、画像のようなエラーが出てしまいます。
どのように回避したらよいかわからない状態です。

@json['items'] が nil な時（データがない）があるということなので、

if @json['items'].nil? と else で条件分岐するのが良いかと思われます

Comments

[hurousyotoku500]

ご回答ありがとうございます。
まず、２のほうでお聞きしたいのですが、.nil?はnilの場合がtrueでよろしいでしょうか。
追記したコードで実行してみましたが、うまく表示されなく、データが存在する場合に該当データなし！と表示されてしまいました。どの辺に問題ありますでしょうか。

[s4na_penguin]

> まず、２のほうでお聞きしたいのですが、.nil?はnilの場合がtrueでよろしいでしょうか。

はい

> 追記したコードで実行してみましたが、うまく表示されなく、データが存在する場合に該当データなし！と表示されてしまいました。どの辺に問題ありますでしょうか。

あっ、すみません、 `Items` の `i` が小文字だったみたいですね
そこを直せば表示できるのではないでしょうか。

[hurousyotoku500]

ありがとうございます。該当するデータが存在する場合は表示できるようになりましたが、該当データない場合がまだ表示されない状態です。

[s4na_penguin]

> 該当データない場合がまだ表示されない状態です。

> <tr>
> <th>該当データなし！</th>
> </tr>

みたいに表示できないってことですよね？

うーん????

何か想定外のデータが入ってしまっているんでしょうか・・・

`= @json['Items']` でViewにデータを表示するか、
`pp @json['Items']` でログにデータを表示するなどして、

データが `nil` かどうか確認したいですね。

[hurousyotoku500]

すみません、データの表示のしかたがよくわからず…
ちなみにですが、エラーがでる、該当データが存在しない場合は下記のようなjsonになります。
{
"error": "not_found",
"error_description": "Data Not Found"
}
https://webservice.rakuten.co.jp/explorer/api/Gora/GoraGolfCourseSearch/

[hurousyotoku500]

下のように書いて、該当ありの場合は表示されましたが、該当なしの場合はNoMethodError in Courses#indexというエラーになってしまう状況です。
<table>
<tr>
<th>Name</th>
</tr>

　<% if @json['error'] == "not_found" %>
<tr>
<th>該当データなし！</th>
</tr>
<% else %>
<% @json["Items"].each do|item| %>
<tr>
<td>
<%= item["Item"]["golfCourseName"] %>
</td>
</tr>
<% end %>
<% end %>
</table>

[s4na_penguin]

`if @json['error'] == "not_found"` より `if @json['error'].present?` の方が良さそうですね

errorの中に入ってくるものはパターンがいろいろあるみたいなので。

> すみません、データの表示のしかたがよくわからず…

`<% = pp @json %>` こういう感じでViewに直接表示する感じですね
CSSの状態によっては見えなかったりするので、そしたらChrome Developer tools でHTMLを見てみると良いと思います〜

[hurousyotoku500]

すみません、該当なしの場合はエラー分が表示されてしまい、確認できないのですが、方法があるのでしょうか？

[s4na_penguin]

なるほど。

`<% = pp @json %>` 一旦、以外消しちゃうのがいいかと思います。
（今のコードはどこか別のところに保存しておいて）

[hurousyotoku500]

ありがとうございます。確認出来ました。
該当ありの場合は、全ての情報が表示され、該当なしの場合は何も表示されませんでした。