inputで入力された値を使って複数レコードの更新をしたい

Question

前提・実現したいこと

laravel勉強中の初心者です。よろしくお願いします。

同じname属性のinputタグを使い、メッセージを登録し、更新したい
メッセージの登録（create)はできたのですが更新ができなくて困っています（全てnullになる）

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

エラーメッセージはなく、更新処理をすると全てnullになります

該当のソースコード

＄Messageにはあらかじめ登録されたメッセージが入っています。

//edit画面
<form action="">
   <tr>
      <td><input class="form-control" name="receive_message_1[]" value='{{$Message->receive_message_1}}'></td>
      <td><input class="form-control" name="receive_message_2[]" value='{{$Message->receive_message_2}}'></td>
      <td><input class="form-control" name="receive_message_3[]" value='{{$Message->receive_message_3}}'></td>                       
   </tr>
    <tr>
      <td><input class="form-control" name="receive_message_1[]" value='{{$Message->receive_message_1}}'></td>
      <td><input class="form-control" name="receive_message_2[]" value='{{$Message->receive_message_2}}'></td>
      <td><input class="form-control" name="receive_message_3[]" value='{{$Message->receive_message_3}}'></td>                       
   </tr>
</form>

このような感じで、おなじname属性のinputタグが2つづつ、計６つあります。このような時にはinputを配列にする必要がある、と見ましたので[]がついています。

create画面のHTMLは上記とほぼ同じです（value部分がないだけ）
create部分のコントローラーは以下のような感じです

public function store(Request $request,Message $message){
　　　　　//メッセージ登録する:配列で値を受け取る
        $data = array(
            'receive_message_1' => $request->input('receive_message_1'),
            'receive_message_2' => $request->input('receive_message_2'),
            'receive_message_3' => $request->input('receive_message_3'),
        );
        
        //dataを展開して値を入れた
        $i = 0;
            foreach ($data['receive_message_1'] as $d) {
　　　　　　　　　　$message = new Message;
                $message->receive_message_1 = $request->receive_message_1[$i];
　　　　　　　　　 $message->receive_message_2 = $request->receive_message_2[$i];
                $message->receive_message_3 = $request->receive_message_3[$i];
                $message->save();
                $i++;
         }
        return redirect('/');
    }




public function update(Request $request, Message $message){

  // 配列で受け取る
        $data = array(
            'receive_message_1' => $request->input('receive_message_1'),
            'receive_message_2' => $request->input('receive_message_2'),
            'receive_message_3' => $request->input('receive_message_3'),
            'created_at' => Carbon::now(),
            'updated_at' => Carbon::now(),
        );
//dataを展開して更新してみました、がうまくいかず
        $i = 0;
            foreach ($data['receive_message_1'] as $d) {
                $message->receive_message_1 = $request->receive_message_1[$i];
                $message->receive_message_2 = $request->receive_message_2[$i];
                $message->receive_message_3 = $request->receive_message_3[$i];
                $message->save();
                $i++;
         }
        return redirect（"/"）
}

試したこと

$message->receive_message_1や$request->input('receive_message_3')に値が入っていることは確認できましたが、更新がうまくいきません

補足情報（FW/ツールのバージョンなど）

laravel5.6、リソースコントローラー使用して構築しています

Comments

[sola-msr]

> メッセージの登録（create)はできたのですが

このメソッドも記載したほうがいいかと

[yambejp]

inputにtypeが指定されていませんがテキストボックスで間違いないですか？
またformにダイレクトにtrを書くのはNGです

[kentooooo]

type:textです。
メソッドも記述いたしました。よろしくお願いします

[mikkame]

なにがどううまくできないんでしょうか？
エラーが出るのか、更新されないのか、一部だけ更新されるのか
回答者はそれを脳内で実行してor同じ環境を整備して確認するしかありません。

[kentooooo]

エラーがでなくて、DB が全部null になってしまうんです。

[mikkame]

全部というのは、該当のMessageだけですか？
他のMessageもでしょうか？

[kentooooo]

他のメッセージもすべてです。

[mikkame]

他のメッセージも、ということであれば何かしらおかしいですね。
そもそもですが、
配列を送って、ループして更新している。ということは
正常に動いたとしても最後の1行分しか反映されない、という事にはお気づきでしょうか。

まずはupdateの引数に入っている$messageをddして、更新対象があっているかを確認してみてはどうでしょうか

[kentooooo]

＞そもそもですが、
配列を送って、ループして更新している。ということは
正常に動いたとしても最後の1行分しか反映されない、という事にはお気づきでしょうか。

そうだったんですね。うまく理解ができてないみたいです。
配列のやり方以外にどんなやり方があるんでしょうか？

[mikkame]

引数で渡ってくる$messageは確認しましたか？
これが、更新対象のレコード（行）です。
それを繰り返し（foreach）保存(save)しているので、最終的には最後のデータ（配列ないの最後）しか残らない

という解釈です

[kentooooo]

すっきりと理解できる説明です。
ありがとうございます。
ddした結果がnullでした…。
考えが煮詰まってしまっています。
少し話がそれるのですが、
Createは配列を使ってうまくできてるのですが、このやり方で大丈夫なものでしょうか？

[mikkame]

createは問題ありません
リソースコントローラのupdateは基本的に特定のレコードのみ更新する事を前提としています
（PUT /message/1 で、idが1番のものを更新する、　100番を更新したければ　/message/100）
一括更新を行いたいのであれば
あたらしく　function bulkUpdate(Request $request) を追加し（web.phpも同様に修正する）
そこに一括更新用のコードを書くのがベターかと思われます

また、別の問題で、現在のコードですとそれは一体何を更新しようとしているか？
（更新対象のid）が抜け落ちています。
それを含めて更新する必要があると思います。

私なら
<td><input class="form-control" name="receive_message_1[{{$Message->id}}]" value='{{$Message->receive_message_1}}'></td>

とし
foreach ($request->receive_message_1 as $message_id => $value) {
$message = Message::find($message_id);
$message->receive_message_1 = $request->receive_message_1[$message_id];
$message->receive_message_2 = $request->receive_message_2[$message_id];
$message->receive_message_3 = $request->receive_message_3[$message_id];
$message->save();
}

のような感じで対応します

[kentooooo]

勉強になります。ありがとうございます
現在夜勤中でして、また明日試してみようと思います。
またお返事書かせてやって下さい。

[kentooooo]

夜勤中にこっそりと失礼します。
教えていただいた方法で、無事updateできるようになりました！

リソースコントローラーが便利なので、何も考えずに使っていたのがよくなかったです。
もっと基礎の部分を勉強しないといけないなと思わされた次第です。
本当にありがとうございました！

Answer 1

ほぼコメント欄で解決しましたが一応回答を書いておきます。

updateで複数のレコードを対象に更新しているように見えますが
リソースコントローラでは、特定の（引数に入ってくるモデルのインスタンス）レコードに対しての
操作を想定しています。
一括アップデート用に新しくメソッドとルートを追加するのが良いでしょう

また、現在の内容では、どのレコードを更新するかというのが明確に書かれていないのも
（通常はid等で条件を絞って更新する）
うまくいかない要因かと思われます。

Comments

[kentooooo]

本当にありがとうございました。
これからも勉強を続けていきます！