【Ruby on Rails】paramsに初期値をセットしたい

Question

初歩的なことで恐れ入ります。

ビューからパラメータ（日付のFromとTo）を受け取り、両方とも空だった場合は
初期値として現在日をセットして表示したいです。

そこでまずパラメータが空かどうかのチェックを行いました。

下記のようにすると、検索から来た場合は
params[:q]の中に、
日付のFromである[:imp_date_gteq]、
日付のToである[:imp_date_lteq]
がそれぞれ存在しているので初期値をセット出来ますが、

そもそも画面の初期表示時に
まだ
params[:q]がnil のため
エラーになるようです。

	# 一覧
  def index

		if params[:q].present?
			if params[:q][:imp_date_gteq].present? != true && params[:q][:imp_date_lteq].present? != true
				params[:q][:imp_date_gteq] = Date.today
				params[:q][:imp_date_lteq] = Date.today
			end
		else
			#ここでエラー↓
			params[:q][:imp_date_gteq] = Date.today
			params[:q][:imp_date_lteq] = Date.today
		end
end

このような要件の場合、初期値をセットする方法があるでしょうか？

何か方法があれば教えて下さい。
宜しくお願い致します。

Comments

[moke]

エラーが出るのはparams[:q]がnilだからです。私はModel.search(params[:q]||{imp_date_gteq: Date.today,imp_date_lteq: Date.today})とやってしまうんですが、細かい条件となると最適解を知らないですね。

[pecchan]

moke様　有難う御座います。間違い(エラーの原因)を修正しました。

[moke]

見苦しいコードになりそうなので書くのを控えてましたが、よろしければ書きましょうか？

[pecchan]

moke様　有難う御座います。どうしてそれが見苦しいかを考えることで私には勉強になります。良ければお願いします。

Answer 1

dig メソッドを用いると、ハッシュ値の参照が簡単になります

def index
  # 参照するキー値がなかった場合にnil が返る
  date_params = [parms.dig(:q, :imp_date_gteq), params.dig(:q, :imp_date_lteq)]
  # 両方の値がnil か判定した結果を代入
  date_both_nil = date_params.compact.size.zero?

  # 日付データが両方空か、内容に応じて代入値を振り分け
  @date_from, @date_to =
    if date_both_nil
      [Date.today, Date.today]
    else
      date_params
    end
end

params の利用は、ビューから渡される値の参照だけに留め、ビュー側ではコントローラから設定されるインスタンス変数を扱うようにするのがよいのではないでしょうか

何か参考になれば幸いです

Link

• Class: Hash (Ruby 2.3.0)
• instance method Hash#dig (Ruby 2.4.0)

Comments

[pecchan]

遅くなり申し訳ございません。
大変参考になりました。
有難う御座います。

Answer 2

これを機に私もベストプラクティスを考えようと思いましたが
ちょっと忙しいのでやっつけですみません。

            search_params={}
        if params[:q].present?
search_params=params[:q]
            if params[:q][:imp_date_gteq].present? != true && params[:q][:imp_date_lteq].present? != true
                search_params[:imp_date_gteq] = Date.today
                search_params[:imp_date_lteq] = Date.today
            end
        else

            search_params[:imp_date_gteq] = Date.today
            search_params[:imp_date_lteq] = Date.today
        end

Model.search(search_params)
params[:q]=search_params#これで検索条件をformに戻す。

とりあえずRails4で動くコードです。Rails5だと動かないかも…。
絶対いい方法があるはずなんですが…とりあえず解決済みにするのは待っててください

Comments

[pecchan]

moke様

お忙しいところ申し訳ないです・・・。
いったん解決済みを外しました。

ひとまず勉強しときます。

有難う御座います。

Answer 3

def indexに引数でデフォルト値を置いてみてはどうでしょうか

def action(hoge="default_hoge")
  @hoge = hoge
end

こうしたとき
params[:hoge]が存在する場合は
hoge = params[:hoge]

params[:hoge]がnilの場合は
hoge = default_hoge
となります

Comments

[pecchan]

haneru様

有難う御座います。

以下のようにしました。
def index(imp_date_gteq = Date.today)

params[:q][:imp_date_lteq] = imp_date_gteq

結果的には同じでした。
検索で来た場合は、初期値セット出来ましたが
初期表示時は、params[:q]そのものがnilのためエラーになりました。