楽天APIを使用した詳細検索機能で、パラメータが不足した状態で検索実行した場合デフォルト値が入るようにしたい

Question

前提・実現したいこと

※プログラミング初心者です。失礼があったら申し訳ございません

楽天のAPIを使用して、キーワード検索と詳細検索の２つの機能を有したアプリを作成しています。
詳細検索では、

• ジャンルID
• キーワード
• 除外キーワード
• 並び替え
• 送料の有無

を選択して検索できるような機能を目指しています。

以下は、コントローラー側でフォームからパラメーターを受け取った後の処理です。
（キーワード検索の場合）

RakutenWebService::Ichiba::Item.search(keyword: params[:keyword], page: page)

例えば、詳細検索で並び替えを選択して楽天APIを使用する場合は下記になります。

RakutenWebService::Ichiba::Item.search(keyword: params[:keyword],  sort: params[:sort] ,page: page)

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

上記記述の場合、keywordやsort以外にもNGKeywordなど、詳細検索で使用できる全てのパラメータを受け取る必要があります。

ただし、下記のように記述した場合、なにか１つでも値が入ってないとエラーとなってしまいます。

RakutenWebService::Ichiba::Item.search(keyword: params[:keyword],  sort: params[:sort], NGKeyword: params[:NGKeyword], genreId: params[:genreId], postageFlag: params[:postagFlag] ,page: page)

試したこと

下記のように、if params[:sort].present? で１つずつのパラメータはチェックさせることができるのですが、膨大なパターンとなりあまり現実的ではないように思ってます。

def searchRequest(page)
    if params[:sort].present?
      RakutenWebService::Ichiba::Item.search(keyword: params[:keyword], NGKeyword: params[:NGKeyword], sort: params[:sort] ,page: page)
    else
      ngword = ""
      RakutenWebService::Ichiba::Item.search(keyword: params[:keyword], page: page)
    end
  end

なにか良いビジネスロジックはございませんでしょうか？

各パラメーターに何かしらのデフォルト値があって、フォームからnilが返ってきた場合はそのデフォルト値を使う・・・とかがいいのかなと思いましたが記述をどうすればよいのか？そもそもこの考え方でよいのか迷っております。

どうかご教授お願いいたします。

Answer 1

「各パラメーターに何かしらのデフォルト値があって」とありますが、
「どういうデフォルト値を入れたらよいかがわからない」のか
「どのようにデフォルト値を定義するのが良いのかがわからない」のか何方でしょう
前者ですとお手伝いできませんが、後者でしたら
default_params = {keyword: "key", NGKeyword: "ng", sort: "sort",page: "page"}
と定義しておいて
merged_params = default_params.merge(params.permit(keylist))
として　merges_params を使ってください。
なお、permit(keylist)をつけないと 「permitしていないparamsはHashにできない」エラーになります。keylistには必要な全項目の列挙を。

Comments

[komiso]

ご回答頂きありがとうございます！
知識不足のため、噛み砕くのに時間がかかってしまいご返信遅れて申し訳ございません。
以下のように記述しました。

```ruby
def searchRequest(page)
default_params = { NGKeyword: "NGKeyword" }
merged_params = default_params.merge(params.permit(:keyword, :NGKeyword, :genreId, :postageFlag, :sort, :page))

RakutenWebService::Ichiba::Item.search(merged_params)
```

この場合、NGKeywordが未入力の場合は、default_paramsの"NGKeyword"が渡される想定だったのですが、どうやらform_withのtext_fieldから未入力の場合はNGKeyword=>""が返ってきてるみたいで、設定値ではなくこちらを適用してしまい、【NGKeyword is not valid】が出てしまいます。

これは、コントローラー側で条件式で、if params[:NGKeyword].present?のように逃げるしかなさそうでしょうか？

それとも、私のデフォルト値の記述違い、もしくは認識違いがございますでしょうか？

[winterboum]

認識の通りですね。
default設定するkeyはいくつ有りますか？たくさんあると
if params[:NGKeyword].present? だと醜いですね、
params.permit(keylist).select{|k,v| v.present?}
をmergeしましょう

[komiso]

即レスありがとうございます！
selectメソッド勉強させていただきました。

教えていただいた書き方ですと、ハッシュの中身をselectメソッドで評価して、valueが真のものだけを抽出できるのですね。

これをdefault_paramsにmergeすることによって、NGKeywordなど未入力の場合はmergeされずにデフォルト値が有効になる、という解釈をしました。

そして、最終的な書き方は以下のようにすることで想定していた機能を実現することが出来ました。

```ruby
def searchRequest(page)
default_params = { NGKeyword: "NGKeyword", postageFlag: 0, sort: "standard" }
merged_params = default_params.merge(params.permit(:keyword, :NGKeyword, :genreId, :postageFlag, :sort, :page).select{ |k,v| v.present?})

RakutenWebService::Ichiba::Item.search(merged_params)
end
```

おかげさまで助かりました！感謝いたします！