Railsのpermitについて

Railsのpermitメソッドについて、モデルに渡す以外のところで使おうと思ったら、中々うまくいかなので、質問させていただきます。

環境

Ruby 2.6.5
Ruby on Rails 5.2.0
CentOS 7.7(Docker)

やりたいこと

検索フォームから受け取った値をストロングパラメータで検証したい。

検索フォームから受け取ったパラメータをto_hashする必要があり、その際に、[params.permit!]みたいな記述で検証をパスしていたが、
受け取るパラメータのみで絞りたいので、permitを使いたい。

できない理由

children_controller.rb

def search
  Rails.logger.error("parameter ==== #{params.class}")
  params = params.permit(
    :paramA,
    :paramB,
    :paramC,
  )
  params = params.to_hash
end

上記のようにしてパラメータ検証しようとおもったが
undefined method ``permit' for nil:NilClassが出力される。。

children_controller.rb
1Rails.logger.debug(params.class)とすると
2=> ActionController::Parameters

Paramertersオブジェクトに対してpermitするとエラーがでる？
モデル検証以外にpermitを使う際はpermit!で全部許可する指定がベターだと思う？のですが、
それ以外の場合上記のようにpermit(:paramA, :paramb...)としたい場合はどのような書き方をすればよいでしょうか？

maisumakun

2019/11/12 05:10

> 上記のようにしてパラメータ検証しようとおもったがとありますが、メソッド全体とそれをどこに書いたかを記載してください。

yuuki0218

2019/11/12 05:16

ご指摘ありがとうございます。少し追記してみました。

行動規範の内容に同意します

回答2件

ベストアンサー

NillClassのオブジェクトにpermitを使用しようとしているので質問以前の問題です。

投稿2019/11/12 04:57

rhiroe

総合スコア2344

yuuki0218

2019/11/12 05:12

上記で載せているソースは簡易的なものであったため、もう少し詳しく載せます。以下のような記述であれば、paramsが`NilClass`オブジェクトではないと思うのですが、認識間違いありますでしょうか。 ``` def search(params) Rails.logger.error("parameter ==== #{params.class}") # => ActionController::Parameters params = params.permit( :A, :B, :C, :D ).to_hash end ```

rhiroe

2019/11/12 05:19

まず、誤解しているところを正すと、元々のRailsのparamsというのは変数名ではなくメソッド名です。ですので params = とした時点でparams変数がnilで初期化されて用意されるのでparamsメソッドの呼び出しができなくなり、代わりにnilのparams変数が呼ばれるようになります。ですのでその後のparams.permitはnilに対してpermitを実行していることになります。

rhiroe

2019/11/12 05:21

ちなみに代入構文で右辺から評価されないというのは違和感があるかもしれませんが、 > params = とした時点でparams変数がnilで初期化されて用意されるというのはRubyの構文解析の仕様です。

rhiroe

2019/11/12 05:28 編集

代入構文の評価順位ついては a = a をrubyで実行しても undefined local variable or method `a' for ... のエラーが発生しないのでそれで納得していただけるはずです。

yuuki0218

2019/11/12 07:19

丁寧な説明ありがとうございます。完全に誤解していました、paramsという変数にパラメータの値が挿入されていると考えていたのですが、確かにご指摘通り、paramsがメソッド名であれば、params = としたら初期化されてエラーになりますね。。代入の評価順位についても、なるほど勉強になりました。 params = の記述をなくし、params.permit(...).to_hashとしたところ、問題なくHash化することまで確認できました。ほかの方の回答でも同様の指摘をいただきましたが、いち早く回答いただいたのでベストアンサーとさせていただきます。ありがとうございました。

行動規範の内容に同意します