formから複数のレコードを更新したい

Question

複数の<input>タグからフォームに送信を実行すると、最後に送信したデータの内容が全てのレコードに反映されてしまうという問題で困っています。
原因または解決策をご存知の方はいらっしゃいませんか。

私の行った手順は以下です。
(1)同一テーブルカラムの中にある複数のレコードに更新処理を掛けるFormの記述
(2)Controller側で受け取ったデータを「$a = 〇〇::where(‘id’, ‘=’, $request->id)->get();」という形で受け取り、foreachで展開。
(3)カラムごとに「$a->xx = $request->xx;」という形でRequestを受け取り、最後にsave()メソッドで更新。

すると、全てのレコードが最後に送信したデータの内容になってしまうという状況になりました。

私は【foreachで展開して、それぞれsaveをしているため、where(‘id’, ‘=’, $request->id)で各レコードが区別されて、それぞれレコードが対応したデータに更新する】と思いました。

原因を確かめるため、以下のようなテストを行ってみましたが、
問題の解決には至りませんでした。
(a) 入力を「$a = 〇〇::where(‘id’, ‘=’, $request->id)->first();」にし、foreachでの展開をやめてみた
→上記と同じく全てのレコードが最後に送信したデータの内容になってしまうという状況になった。
なお、私の環境は以下の通りです。
【MacBook Air (13-inch, Early 2015), 8 GB 1600 MHz DDR3, AWS, Laravel5.8.21】

検索エンジンで「laravel レコード　複数挿入 form」などのキーワードを使用して検索しましたが、
formから受け取ったデータを複数挿入する方法などについては見つかりませんでした。

以下に該当コードを抜粋して記載いたしますので、何卒解決策やアドバイスをいただきますよう、よろしくお願いいたします。

●Controller

public function makeInvoice(Request $request, $invoiceId, $clientId)
     {
:
:
        $bills = Bill::where('invoice_id', '=', $request->bill_invoice_id)->get();
        foreach($bills as $bill){
        $bill->billing_item = $request->billing_item;
        $bill->unit = $request->unit;
        $bill->quantity = $request->quantity;
        $bill->bill_unit_price = $request->bill_unit_price;
        $bill->save();
        }
:
:
}

●blade（foreachで該当レコード分だけinputタグが出てくるようにしています。）

{{Form::open(['url' => route('makeInvoice',['clientId'=>$val->client_id, 'invoiceId' => $val->id]),'method'=>'POST', 'target'=>'_blank'])}}
    {{ csrf_field() }}
:
:
            <tr>
                <!--品番・品名-->
                <td>{{Form::text('billing_item', $val->billing_item,['class' => 'validate', 'id' => 'billing_item'])}}</td>
                <!--数量-->
                <td>{{Form::text('quantity', $val->quantity,['class' => 'validate', 'id' => 'quantity'])}}</td>
                <!--単位-->
                <td>{{Form::text('unit', $val->unit,['class' => 'validate', 'id' => 'unit'])}}</td>
                <!--単価-->
                <td>{{Form::text('bill_unit_price', ceil($val->bill_unit_price),['class' => 'validate', 'id' => 'bill_unit_price'])}}</td>
                <!--金額-->
                <td>{{$val->quantity * $val->bill_unit_price}}</td>
                <td><a href="#" data-id="{{$val->id}}" class="waves-effect waves-light btn del">削除</a>
                </td>
                {{Form::hidden('bill_invoice_id', $val->invoice_id)}}
            </tr>
:
:
  {{Form::close()}}

・・・追記
manualには、
name属性（例えばbilling_item）に対して[]を付ければ、
indexが数値として入るとのことでしたので、以下のようにbladeを変更しました。

@foreach($billList as $val)
            <tr>
                <!--品番・品名-->
                <td>{{Form::text('billing_item[]', $val->billing_item,['class' => 'validate', 'id' => 'billing_item'])}}</td>
                <!--数量-->
                <td>{{Form::text('quantity[]', $val->quantity,['class' => 'validate', 'id' => 'quantity'])}}</td>
                <!--単位-->
                <td>{{Form::text('unit[]', $val->unit,['class' => 'validate', 'id' => 'unit'])}}</td>
                <!--単価-->
                <td>{{Form::text('bill_unit_price[]', ceil($val->bill_unit_price),['class' => 'validate', 'id' => 'bill_unit_price'])}}</td>
                <!--金額-->
                <td>{{$val->quantity * $val->bill_unit_price}}</td>
                <td><a href="#" data-id="{{$val->id}}" class="waves-effect waves-light btn del">削除</a>
                </td>
                {{Form::hidden('bill_invoice_id', $val->invoice_id)}}
            </tr>
                <?php $subtotal += $val->bill_unit_price; ?>
                @endforeach

また、Controllerを以下のように変更しました。

        $bills = Bill::where('invoice_id', '=', $request->bill_invoice_id)->get();
        foreach($request->input("bills") as $id => $bill){
        $bill->billing_item = $request->billing_item;
        $bill->unit = $request->unit;
        $bill->quantity = $request->quantity;
        $bill->bill_unit_price = $request->bill_unit_price;
        $bill->save();
        }

しかし、ソースコードを確認するとarray[]の中が空欄となっていました。

https://php.net/manual/ja/faq.html.php#faq.html.arraysについても参照したのですが、問題が分かりませんでした…。再度、助言いただけましたら幸いです。

Answer 1

以下で想定通りの挙動を確認できました。ありがとうございました。

・Controller

   $i = 0;
   foreach($request->bill_id as $id){
       $bill = Bill::find($id);
       $bill->billing_item = $request->billing_item[$i];
       $bill->unit = $request->unit[$i];
       $bill->quantity = $request->quantity[$i];
       $bill->bill_unit_price = $request->bill_unit_price[$i];
       $bill->save();
       $i++;
   }

・Blade

   $i = 0;
   foreach($request->bill_id as $id){
       $bill = Bill::find($id);
       $bill->billing_item = $request->billing_item[$i];
       $bill->unit = $request->unit[$i];
       $bill->quantity = $request->quantity[$i];
       $bill->bill_unit_price = $request->bill_unit_price[$i];
       $bill->save();
       $i++;
   }

Answer 2

formで複数のレコードを扱う場合は、form内のinputのnameを配列にしてあげると可能です。

#bills[id または index][properly]
name="bills[1][billing_item]"
name="bills[1][quantity]"

PHP HTML フォームで配列を使用するにはどうすればよいですか？

その後、リクエストデータをforeachで回すと良いです。

foreach($request->input("bills") as $id=>$bill){
//処理
}

Answer 3

まずはSQLのテーブル構成とサンプルデータをつくり
具体的などういったデータを元にどう更新したいかSQL文を作るところからですね