updateアクションでもう一度@message = Message.find(params[:id])しないとダメ？？？

Question

極めて単純なメッセージ投稿アプリを作っています。その編集機能を実装中なのですが、すでにeditアクションで引き出したメッセージ情報を同じようにまたupdateアクションで引き出さないとならないですか？

<%= form_with(model: @message, local: true) do |f| %>
  <%= f.text_field :message %>
  <%= f.submit %>
<% end %>

class MessagesController < ApplicationController
  
  省略
  def edit
    @message = Message.find(params[:id])
  end
  def update
    @message = Message.find(params[:id])
    if @message.update(message_params)
      redirect_to root_path
    else
      render :edit
    end
  end

  private
  def message_params
    params.require(:message).permit(:message, images_attributes: [:image]).merge(user_id: current_user.id)
  end
  省略
end

このupdateアクションの@message = Message.find(params[:id])を書かない方法があったと思うのですが、わかる方いらっしゃいますでしょうか？

Answer 1

すでにeditアクションで引き出したメッセージ情報を同じようにまたupdateアクションで引き出さないとならないですか？

はい。各アクションで必要な情報をDBから取得する必要があります。

まず、アクションはそれぞれ独立したリクエストとして処理されます。
editアクションとupdateアクションはまったくの別物です。

別物なのでeditアクションで@message = Message.find(params[:id])したからといって、
updateアクションにはまったく何の関係もないという事です。
だから、@messageが必要ならそれぞれのアクションでfindしなければいけません。
このルールは絶対であり大前提です。

このupdateアクションの@message = Message.find(params[:id])を書かない方法があったと思うのですが

例えば、before_actionを使う事により、処理を共通化する事は可能です。
この事を指していると思います。
このようにすると、表面上はedit, updateにfindを書かずに済みます。
しかし、結局は毎回findしている事に変わりはありません。

class MessagesController < ApplicationController
  before_action :set_message, only: [:edit, :update]

  def edit
  end

  def update
    if @message.update(message_params)
      redirect_to root_path
    else
      render :edit
    end
  end

  def set_message
    @message = Message.find(params[:id])
  end

end

Comments

[annaPanda]

> before_actionを使う事により
そうか、そこで勘違いしていたんですね。丁寧にありがとうございました！