Formオブジェクト updateが全てに適用されてしまいます

Question

前提・実現したいこと

投稿したものをupdateしようとすると今までに投稿したものが
全て同じものに更新されてしまってます。
指定した投稿記事のみを更新できるようにしたいです。
よろしくお願いします。

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

エラーメッセージ
NoMethodError in MessagesController#update
undefined method `id' for #<Array:0x00007fb3a1cdff58>

エラーメッセージがでた後Topページなどにいくとデータが全て更新されている
きちんとメッセージが指定できていないのではないかと思うがどのようにすればいいか分かりません

該当のソースコード

messages_controller
  def edit
    @message = Message.find(params[:id])
    tag = @message.tags
    if current_user.id != @message.user.id
      redirect_to root_path
    end
  end

  def update
    @message = Message.find(params[:id])
    @message = MessageTag.new(update_message_params)
    @message = @message.update
  end

  private
  def message_params
    params.require(:message_tag).permit(:title, :message, :whom, :open_plan, :name, :video, images: []).merge(user_id: current_user.id)
  end

  def update_message_params
    params.require(:message).permit(:title, :message, :whom, :open_plan, :name, :video, images: []).merge(user_id: current_user.id)
  end

message.rb
class Message < ApplicationRecord
  belongs_to :user
  has_many_attached :images
  has_one_attached :video
  has_many :message_tag_forms, dependent: :destroy
  has_many :tags, through: :message_tag_forms
end

tag.rb
class Tag < ApplicationRecord
  has_many :messages, through: :message_tag_forms
  has_many :message_tag_forms
  validates :name, uniqueness: true
end

message_tag_form.rb
class MessageTagForm < ApplicationRecord
  belongs_to :message
  belongs_to :tag
end

class MessageTag

  include ActiveModel::Model
  include ActiveModel::Attributes
  include ActiveRecord::AttributeAssignment
  attr_accessor :title, :whom, :open_plan, :message, :images, :video, :name, :user_id

  with_options presence: true, length: { maximum: 50 } do
    validates :title
    validates :whom
  end
  
  with_options presence: true do
    validates :message, length: { maximum: 200 }
    validates :open_plan
    validates :user_id
    validates :name, length: { maximum: 20 }
  end

  def save
    @message = Message.create(title: title, whom: whom, open_plan: open_plan, message: message, images: images, video: video, user_id: user_id)
    tag = Tag.find_or_create_by(name: name)

    @message_tag = MessageTagForm.create(message_id: message.id, tag_id: tag.id)
  end

  def update
    @message = Message.update(title: title, whom: whom, open_plan: open_plan, message: message, images: images, video: video, user_id: user_id)
    tag = Tag.find_or_create_by(name: name)
    binding.pry
    if MessageTagForm.update(message_id: message.id, tag_id: tag.id)
      return redirect_to root_path
    else
      render :edit
    end
  end
end

edit.html.erb
<div class="new-message">
  <h1>残したい想い</h1>
  <%= form_with model: @message, local: true do |f| %>
  <%= render 'shared/error_messages', model: f.object %>
  <div class="posting-form">
    <div class="form">
      タイトル
      <span class="indispensable">必須</span>
    </div>
    <%= f.text_field :title, class:"form-title", id:"form-title", placeholder:"タイトル名(必須 50文字以内)", maxlength:"50" %>
    <div class="form">
      メッセージ
      <span class="indispensable">必須</span>
    </div>
    <%= f.text_area :message, class:"form-message", id:"form-message", placeholder:"メッセージ内容(必須 200文字以内)例)20年後の子供の誕生日に見て欲しい動画です。" ,rows:"5", maxlength:"200" %>
    <div class="form">
      誰に対してか
      <span class="indispensable">必須</span>
    </div>
    <%= f.text_field :whom, class:"form-whom", id:"form-whom", placeholder:"誰に宛てたものか(必須 50文字以内)例)20年後の子どもたちへ", maxlength:"50" %>
    <div class="form">
      開封予定日
      <span class="indispensable">必須</span>
    </div>
    <div class="form-open-plan">
      <%= raw sprintf(
                  f.date_select(
                    :open_plan,
                    class: 'open-plan-object',
                    use_month_numbers: true,
                    prompt:'--',
                    start_year: (Time.now.year),
                    end_year: (Time.now.year + 100),
                    date_separator: '%s'),
                  "<p> 年 </p>", "<p> 月 </p>") + "<p> 日 </p>" %>
    </div>
    <div class="form-caution">
      ※※※画像か動画どちらか1つの投稿にしてください。※※※
    </div>
    <div class="image-form">
      <div class="form">
      届ける想い(画像)
      </div>
      <div class="click-upload">
        <p>クリックしてファイルをアップロード</p>
        <%= f.file_field :images, name: 'message[images][]', id:"form-image" %>
        <div id="image-list"></div>
      </div>
    </div>
    <div class="video-form">
      <div class="form">
      届ける想い(動画)
      </div>
      <div class="click-upload">
        <p>クリックしてファイルをアップロード</p>
        <%= f.file_field :video, id:"form-video" %>
      </div>
    </div>
    <div class="tag-form">
    タグ
    </div>
    <%= f.text_field :name, id:"form-tag", placeholder:"誕生日", maxlength:"20"%>
  </div>
  <div class="message-btn-contents">
    <%= f.submit "想いを更新する" ,class:"message-btn" %>
    <%= link_to 'もどる', root_path, class:"back-btn" %>
  </div>
  <% end %>

</div>

pry(#<MessageTag>)> @message
=> [#<Message:0x00007fb3a210e0e8
  id: 20,
  title: "テスト",
  whom: "新規投稿テスト",
  message: "新規投稿テスト",
  open_plan: Sat, 06 Jun 2026,
  encrypted_password: nil,
  user_id: 1,
  created_at: Sun, 10 Jan 2021 21:49:47 UTC +00:00,
  updated_at: Thu, 04 Feb 2021 06:18:56 UTC +00:00>,
 #<Message:0x00007fb3a210e020
  id: 21,
  title: "テスト",
  whom: "新規投稿テスト",
  message: "新規投稿テスト",
  open_plan: Sat, 06 Jun 2026,
  encrypted_password: nil,
  user_id: 1,
  created_at: Sun, 10 Jan 2021 21:50:59 UTC +00:00,
  updated_at: Thu, 04 Feb 2021 06:18:56 UTC +00:00>,
 #<Message:0x00007fb3a210df58
:

pry(#<MessageTag>)> @message_tag
=> nil

追記

edit.html.erbで@messageを@message_tagにした時のエラー発生部分のファイル

error_message.html.erb
<% if model.errors.any? %>

<div class="error">
  <ul>
    <% model.errors.full_messages.each do |message| %>
    <li class="error-message"><%= message %></li>
    <% end %>
  </ul>
</div>
<% end %>

Answer 1

質問に対する直接的な回答としては、MessageTag#update の一行目で Message.update しているからじゃないかなと思います。Message の ID を使っていないので全件更新みたいになっちゃってます。

すいません、これだいぶ混乱する。。。

• エラーが発生するのはどの行ですか？updateアクションの一行目かな？
• updateアクションの一行目が機能していない（二行目でインスタンス変数が上書きされてる）のでは？
• フォームオブジェクトとは具体的にどのクラスですか？すいません、これがすごい分かりづらいです。

名前からしてフォームオブジェクトっぽいのは MessageTagForm ですが、これは Message と Tag を多対多で結合するために使用されるリレーションテーブルのモデルなので、名称的には MessageTag とかになるのが普通。
MessageTag が内容見るからにフォームオブジェクトっぽいけど、なぜか edit.html.erb のフォームで使われていない。update のフォームで使われているか分からないけど update_message_params 見る限り使われていなさそう。
フォームオブジェクトというのは文字通り、Webページのフォームを表現するために使われるのが一般的で、実際そうした方が使いやすいです。Message と Tag を両方いっぺんに更新したいという目的にフォームオブジェクトを導入するのは良いやり方だと思いますが、チグハグになっちゃっているので整理した方が良い状態です。
このまま強引に進めると、今回にみたいに想定外の動作が起きたときに、さらに何が悪いのか分からなくなってきます。

いっぺんにやりすぎて混乱しちゃってるように見えるので、まずは Message だけを新規追加、更新するところから初めてはどうでしょうか？
その上で、多対多の関係にあるタグ付けの機能を足していきましょう。

Comments

[shunxile]

>>okabowさん
回答ありがとうございます。

エラーが発生するのはどの行ですか？updateアクションの一行目かな？
message_tag.rbのupdateアクションの3行目binding.pryが記述してある次の行ifの部分です。

updateアクションの一行目が機能していない（二行目でインスタンス変数が上書きされてる）のでは？
まず更新したい情報を呼びだして
https://teratail.com/questions/291532
こちらを参考にupdateを行うために
```
@message = MessageTag.new(update_message_params)
@message = @message.update
```
と記述しました。

フォームオブジェクトとは具体的にどのクラスですか？すいません、これがすごい分かりづらいです。
フォームオブジェクトはmessage_tag.rbです

editで使用というのは
<%= form_with model: @message_tag, local: true do |f| %>
という記述になるということでしょうか？
@messageから@message_tagにした時は
<% if model.errors.any? %>の部分で
NoMethodError　undefined method `errors' for nil:NilClassが発生します。

[oakbow]

>message_tag.rbのupdateアクション
アクションというのはコントローラのメソッドのうち実際に画面描画に使用されるものの特別な名称です。
message_tag.rbはコントローラではないので、単にupdateメソッドと呼びます。

[oakbow]

初心者のコードは参考にしては行けません。フォームオブジェクトの使い方が間違っていますし、泥沼に陥りつつあります。
回答に書いた通り、まずはフォームオブジェクトを使用せずにMessage単体の更新処理を実装しましょう。
で、updateアクション一行目のこれ
@message = Message.find(params[:id])
が機能していないと思います。二行目で上書きされています。
対象のメッセージを指定しているつもりなんだと思いますが、二行目で上書きしているので指定できていないです。

[oakbow]

>NoMethodError　undefined method `errors' for nil:NilClassが発生します。
フォームオブジェクトとしてまだ不完全だからですね。
form_withの対象モデルを変更すれば良いというのは合っているのですが、それを変えれば終わるというほど修正箇所が少ないわけではないです。
あまりよくないコードを手本に書いたのかなとちょっと想像していたんですが、そうしてしまうと修正箇所が多くなってしまい、「なぜそう修正する必要があるのか」も理解しづらいんじゃないかと思います。
ただ、真っ当に動くように修正し、今後も機能追加や改修を行うことを考えると、Message単体更新の機能をまずきちんと作り終えてからフォームオブジェクトに進むステップを踏んだ方が、回り道に見えて結局近道だと思いますよ。

[shunxile]

>>okabowさん
そうだったんですね。よく分かりました。
ありがとうございます。
仰られた通り「なぜ」がいまいち理解できないというのは確かに感じてました。
フォームオブジェクト使用前のタグがない状態では確かに更新機能はしっかりと作動していたので
１度フォームオブジェクトを使ってない時の状態に戻したいと思います。
ありがとうございます。