Laravelでチェックボックスから複数のvalue値を取り出すには

Question

前提・実現したいこと

PHP(Laravel)でレシピ投稿のシステムを作っています。
レシピの投稿画面で、材料をチェックボックスから複数選択して、valueの合計値をcontroller側で計算する機能を実装中に以下のエラーメッセージが発生しました。
※「合計値」というのは、例えば「たまねぎ(value=1)」と「にんじん(value=2)」にチェックをしたら「3」を格納するということです。

Form::checkboxで複数のチェックボックスの値を取り出すには、どこを改善したらよいのでしょうか。
アドバイス何卒よろしくお願い致します。

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

Illegal string offset 'value'

該当のソースコード

                <div class="form-group">
                    {!! Form::label('ingredients[]', '材料') !!}
                    {!! Form::label('ingredients[]', 'たまねぎ') !!}
                    {!! Form::checkbox('ingredients[]', 1,['class' => 'form-control']) !!}
                    {!! Form::label('ingredients[]', 'にんじん') !!}
                    {!! Form::checkbox('ingredients[]', 2,['class' => 'form-control']) !!}
                    {!! Form::label('ingredients[]', 'たまご') !!}
                    {!! Form::checkbox('ingredients[]', 4,['class' => 'form-control']) !!}
                    {!! Form::label('ingredients[]', 'きゃべつ') !!}
                    {!! Form::checkbox('ingredients[]', 8,['class' => 'form-control']) !!}
                </div>

        //チェックした材料の配列を取り出す
        if(isset($_POST['ingredients']) && is_array($_POST['ingredients'])){
            $ingredients = $_POST['ingredients'];
        }
        

        $ingredient_total　= 0;//材料のvalue値の合計値
        //チェックした配列のvalueを取り出し加算
        foreach($ingredients as $ingredient){
            $ingredient_total += $ingredient['value'];
        }

試したこと

foreachの代わりに、
$ingredient_total = array_sum(array_column($ingredients,'value'));
も試してみましたが、同じくvalueが認識されませんでした。

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

PHP 7.2.22
Laravel 5.5

Comments

[退会済みユーザー]

コントローラー側のソースコードは最低限メソッドのブロック単位で提示してください。

[m.ts10806]

Laravelで$_POST使っている時点で「なぜ？」という疑問が出てきます。
何を参考に作っているのでしょうか。
あと、ブラウザに出力されているhtml、本当に問題ないですか？

[kyontama]

m.ts10806さん
返信ありがとうございます。

qiitaの記事を参考にしたのですが、Laravelではそもそも必要ありませんでした。
理解しないまま実装してしまったので、今後は目的とやり方を一致させるように頑張ります。

[m.ts10806]

ドキュメント読んでください。よほどのことがない限りドキュメントで事足ります

Answer 1

<?php
/**
 * RecipeController
 */
namespace App\Http\Controllers;

use Illuminate\Http\Request;

class RecipeController extends Controller
{
    public function create()
    {
        $ingredients = [
            1 => 'たまねぎ',
            2 => 'にんじん',
            4 => 'たまご',
            8 => 'きゃべつ',
        ];
        return view('recipes.create', compact('ingredients'));
    }

    public function store(Request $request)
    {
        $ingredients = $request->get('ingredients');
        dd($ingredients);
    }
}

<?php
/**
 * recipes/create.blade.php
 */
?>
            @foreach($ingredients as $key => $ingredient)
                <div class="checkbox">
                    <label>
                        {!! Form::checkbox('ingredients[]', $key, null, ['class' => 'form-checkbox']) !!}
                        {{ $ingredient }}
                    </label>
                </div>
            @endforeach

Comments

[kyontama]

Kosuke_Shibuyaさん
ありがとうございました。
無事値を取り出すことができました。
また、次回からはメソッド単位でコードを提示するよう気を付けます。

bladeファイル上ではなく、Controller側に配列を持たせるやり方に気づきませんでした。
ありがとうございます。

[退会済みユーザー]

> bladeファイル上ではなく、Controller側に配列を持たせるやり方に気づきませんでした

ビューにロジックを入れてはいけない。これはフレームワークを使う中で、前提となる使い方です。
MVCの概念をしっかり押さえていないと、フレームワークを使いながらスパゲッティコードを量産する恐れがあります。

マニュアルを読まずに書くとこうなるんだろうなあと思います。

Answer 2

コード
 if(isset($_POST['ingredients']) && is_array($_POST['ingredients'])){
    $ingredients = $_POST['ingredients'];
   //1 
 }
 $ingredient_total　= 0;//材料のvalue値の合計値
  //チェックした配列のvalueを取り出し加算
  foreach($ingredients as $ingredient){
    $ingredient_total += $ingredient;
  }

1の段階で変数にvalue値を入れているのでvar_dump($ingredients)をすると,
[0] => 1, [1] => 3のように格納されているため

foreach内で加算する際にvalueの添え字がなくerrorが発生している。
1で値を取得しているのだからforeachの中では$ingredientを加算していけば
うまくいくと思う。

Comments

[kyontama]

shin0301さん
返信ありがとうございます。

foreachを使った値の加算のやり方があいまいでした。
ディベロッパーツールだとvalue="1"というように表示されるので、てっきりforeach内でもvalueの表示が必要だと勘違いしてました。

foreachを使った配列の処理も今後は使いこなせるように頑張ります。
ありがとうございました。