rails：strong parameterについて教えてください

Question

前提・実現したいこと

railsを学習中の初学者です。
railsでの写真の投稿機能を作製中、空の投稿を行った際にエラーが発生してしまいます。

ParameterMissingの示す通り、strongparameterで指定したmicropostが空だと言われていると思うのですが、どのようにコードを記述すれば問題がないのか、解決の方法がわかりません。

micropost_paramsに何も入っていないため@micropost.saveが失敗し、if文のelseの分岐に入らないのはなぜなのだろう？と疑問に思っています。

お力をお貸しいただけると幸いです。

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

Started POST "/microposts" for 103.5.142.123 at 2018-10-21 08:45:03 +0000
Cannot render console from 103.5.142.123! Allowed networks: 127.0.0.1, ::1, 127.0.0.0/127.255.255.255
Processing by MicropostsController#create as HTML
  Parameters: {"utf8"=>"✓", "authenticity_token"=>"PPaLrLIV51b+cnsyD5lpWrO+TbisckvNPcVURZWQiu7g+TUR2/wPjTHEJyM+vk5t975hvrQqj+PElz8PaF+dcg==", "commit"=>"Create Micropost"}
  User Load (0.2ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = ? ORDER BY "users"."id" ASC LIMIT ?  [["id", 3], ["LIMIT", 1]]
Completed 400 Bad Request in 3ms (ActiveRecord: 0.2ms)


  
ActionController::ParameterMissing (param is missing or the value is empty: micropost):
  
app/controllers/microposts_controller.rb:26:in `micropost_params'
app/controllers/microposts_controller.rb:7:in `create'

該当のソースコード

class MicropostsController < ApplicationController
  before_action :authenticate_user!
  before_action :collect_user,only: :destroy
  

  def create
    @micropost = current_user.microposts.build(micropost_params)
    if @micropost.save
      flash[:success] = "Micropost created!"
      redirect_to root_path
    else
      @feed_items = []
      render "static_pages/home"
    end
  end
  
  def destroy
    @micropost.destroy
    flash[:success] = "Micropost deleted"
    redirect_to root_path
  end
  
     private
  
    def micropost_params
      params.require(:micropost).permit(:picture)
    end
    
    def collect_user
      @micropost = current_user.microposts.find_by(id: params[:id])
      redirect_to root_path if @micropost.nil?
    end
    
end

class Micropost < ApplicationRecord
  belongs_to :user
  default_scope -> {order(created_at: :desc)}
  mount_uploader :picture, PictureUploader
  validates :user_id, presence: true
  validates :picture, presence: true
  validate :picture_size
  
  def picture_size
    if picture.size > 5.megabytes
      errors.add(:picture, "should be less than 5MB")
    end
  end
end

<%= form_for(@micropost) do |f|%>
  <%= render 'shared/error_messages', object: f.object %>

<%= f.label :picture%>
<%= f.file_field :picture,accept: "image/png,image/jpeg"%>

<%= f.submit class: "btn btn-primary" %>

<% end %>

Answer 1

Rais学んで半年ぐらいの者です。うまくいかない可能性もありますが、よろしくお願いします。

画像のアップロードをする際に、nilでも許容したい！という質問に受け取れるので、
fetchメソッドを使うといいと思います。

使い方としては

params.fetch([オブジェクト],{}).permit([カラム１],[カラム2],…)

みたいな感じですね！
これで、オブジェクトがnilでも許容されます！

fetchメソッドは、rubyのメソッドです。
ハッシュオブジェクトのメソッドで、対応するキーがないと第２引数に指定したオブジェクトを返すメソッドです。

params: {
  micropost: {
    picture: nil
  }
}

を渡したいと思うので、

params.fetch(:microposts, {}).permit(:picture)

でうまくいくのではないかと思います。

また、DBでNotNull制約やModelでvalidationを組んでいたら@micropost.saveで弾かれる可能性もあると思いますので、気をつけてください！

参考文献

• rubyのfetchメソッドに関して

https://ref.xaio.jp/ruby/classes/hash/fetch

• Railsのストロングパラメータ

https://qiita.com/mochio/items/45b9172a50a6ebb0bee0#%E6%8C%87%E5%AE%9A%E3%81%97%E3%81%9F%E3%82%AD%E3%83%BC%E3%81%8C%E3%81%AA%E3%81%84%E3%81%A8%E3%81%8D%E3%81%AB%E3%82%A8%E3%83%A9%E3%83%BC%E3%82%92%E5%87%BA%E3%81%95%E3%81%AA%E3%81%84%E3%82%88%E3%81%86%E3%81%AB%E3%81%99%E3%82%8B

Answer 2

私も初学者なので、的外れなことを申し上げているかもしれませんが…。

micropost_paramsに何も入っていないため@micropost.saveが失敗し、if文のelseの分岐に入らないのはなぜなのだろう？と疑問に思っています。

　こちらの件に関してお答えしたうえで、どうすれば意図した動作をしてくれるのかに関して記述させていただければと思います。

　今回のエラーに関して結論から申し上げますと、@mivropost.saveに処理が到達する前にエラーが発生しているということだと思われます。そして、エラーメッセージが示している通り、その原因はstrong_parameterにあります。

　まず最初に確認しておくべきなのは、「今回発生したエラーはどんなエラーか」という点ですね。

ActionController::ParameterMissing (param is missing or the value is empty: micropost)

ParameterMissingの示す通り、strongparameterで指定したmicropostが空だと言われていると思うのですが、

　もちろんその認識で問題ないのですが、今回の場合は**「micropost？そんなもんは存在しないよ！」と言われているように思います。param is missing の部分にあたりますね。strongparameterを活用する際、特に今回のようにrequire(:foo).permit(:bar)という書き方をしている場合、:fooに当たる部分が送られてこない場合には、nilではなくてParameterMissingを返します。**これが今回、if文に到達してくれない理由だと考えられます。

さて、では、「どうしたらこのエラーを回避することができるのか」という話題に移ります。
先にサンプルコードを記述させていただきますね。

def create
  if micropost_params.present?
    @micropost = current_user.microposts.build(micropost_params)
    @micropost.save 
    flash[:success] = "Micropost created!"
    redirect_to root_path

  else
    @feed_items = []
    render "static_pages/home"
  end
end

上記のコードでは、if micropost_params.present?を最初に持ってくることによって、micropost_params自体が渡ってきているかどうかをstrongparameterに行く前に確認しています。すると、micropost_paramsが存在しない－つまりはParameterMissingを吐かれるような状況－では、strongparameterが読まれません。

おそらくこれで、「if文が動かない理由」と「どうしたらエラーを回避できるのか」に関してお答えできていると思います。
おかしなことを言っていたら、ご指摘いただけますと幸いです。
よろしくお願いいたします。

Comments

[su--ss]

お二方とも大変丁寧なご回答ありがとうございます。
ご解説頂いたことも含め大変勉強になりました。

結果から申し上げますとGonzaemonさんの方法で動作しました。
しかしながらtake77さんにお教え頂いた方法でも動きそうなのに、なぜ動作しないのだろう？と調べて見た所、present？メソッドに原因があるのかと思いました。

railドキュメントによると、present？は、変数.present?のような形で使用し「nil, "", " "(半角スペースのみ), [](空の配列), {}(空のハッシュ) のときにfalseを返します。」とありました。
http://railsdoc.com/references/present

１、parameterMissingが帰っている現状
２、method.present?

になっているため使えないということなのかな？と思いました。
この解釈も間違っていたらご指摘頂けるとありがたいです。

重ね重ねになりますが回答ありがとうございました。

[take77]

丁寧な返信、ありがとうございます。

１、parameterMissingが帰っている現状
２、method.present?

解釈はそれであっていると思います。

今回の件ですが、これは私の記述がおかしかったです…。
"micropost_params.present?"と記載していたところですが、正しくは"micropost_params[:picture]"でした。
こうしないと、micropost_paramsが値として取ってもらえないですね。
失礼いたしました（また動かなかったらどうしよう、と内心ビビっています）。

また補足になりますが、私自身の慣習としてDBでnull: falseを可能な限り記載しています。
この辺りの記述に関しては人の趣味が現れるみたいですね。
Gonzaemonさんの記述方法に関しては全く考えていなかった記述でしたので、私も勉強になりました。
Gonzaemonさんにもこの場を借りてお礼申し上げます。