Q&A
解決せずに次へすすめるのは愚の骨頂です。 以前の質問が解決済みであればベストアンサーを選択してください
現在laravelで複数選択可のチェックボックスをmysqlに保存するように実装をしているのですが、保存をしようとすると以下のエラーが出てしまします。おそらく入力した値が配列のままの状態で保存をかけたから以下のエラーがでると考えています。そこでimplodeメソッドを用いて入力した値を[,] カンマでくっつけて保存を試みようとしたのですがうまく適用されません。（エラーは以下と同じ）
下記の記事を参考にして作成しました。おそらくimplodeがうまく適用されてないのかなと思うのですが、原因がわからないので解決策をご教示頂けると幸いでございます。companyというカラムが現在あるのですがそのカラムにチェックボックスで複数選択した値をいれるようにしたいです。
create.blade.php {!! Form::open(['route' => ['kuma.store'], 'method' => 'post']) !!} <input type="hidden" name="confirming" value="{{ old('confirming', 'false') }}"> <div class="form-group required {{ $errors->has('body') ? 'has-error' : '' }}"> <label class="control-label" for="body">コメント</label> @if(old('confirming', 'false') === 'false') <input type="text" class="form-control" name="body" value="{{ old('body') }}"> @else <p class="form-control-static">{{ old('body') }}</p> <input type="hidden" name="body" value="{{ old('body') }}"> @endif @if($errors->has('body')) <p class="help-block">{{ $errors->first('body') }}</p> @endif @if(old('confirming', 'false') === 'false') <?php foreach($menu as $key => $val): ?> <label><?php echo $val ?></label> <input type="checkbox" value="<?php echo $val ?>" name="company[<?php echo $key ?>][]"> <?php endforeach; ?> @else <?php foreach(old("company") as $val): ?> <?php echo $val[0]; ?> <?php endforeach; ?> @endif </div> </div> </div>
kumaController.php public function create() { $menu = (['1' => 'google', '2' => 'facebook', '3' => 'twitter', '4' => 'yahoo', '5' => 'apple', '6' => 'microsoft', ]); $price = ([1=>10000,2=>20000 ,3=>30000 ,4=> 40000, 5=>50000, 6=>60000, 7=>70000, 8=>80000, 9=>90000, 10=>100000]); return view('kuma.create', compact("menu","price")); } public function store(ContactRequest $request) { if ($request->get('action') === 'back') { return redirect() ->route('kuma.index') ->withInput($request->except(['action', 'confirming'])); } // $input = $request->all(); // $this->kuma->fill($input); // ２） // $this->kuma->save(); // ３） $data = $request->all(); if(!empty($request->get('company'))){ $str_commuting_division = implode(",", $data['company']); $data['company'] = $str_commuting_division; }else{ $data['company'] = ''; } $this->kuma->fill($data); $this->kuma->save(); return redirect()->to("kuma"); }
エラー文
Invalid
1``
回答2件
0
自己解決
ありがとうございます。ご教示いただいた回答をもとにうまく実装ができました。
投稿2016/10/19 01:51
総合スコア14
0
エラーメッセージをみると「foreachに無効な引数が渡されている」とありますので、おそらくold("company")が配列を返していないということだと思います。
チェックボックスのLabelにはgoogleなどの企業名を出し、valueに1などを入れる。storeメソッドで[1,2,3]の配列を受け取って "1,2,3" とカンマ区切りの文字列に変換し、DBに保存する。という処理の流れで間違いなければ、下のコードはこうじゃないかなと思います。
php
1<?php foreach($menu as $key => $val): ?> 2 <label><?php echo $val ?></label> 3 <input type="checkbox" value="<?php echo $val ?>" name="company[<?php echo $key ?>][]"> 4 5↓ 6 7<?php foreach($menu as $key => $val): ?> 8 <label><?php echo $val ?></label> 9 <input type="checkbox" value="<?php echo $key ?>" name="company[]">
ついでに…
下のコードはblade記法で書き換えることができます。
php
1@if(old('confirming', 'false') === 'false') 2 <?php foreach($menu as $key => $val): ?> 3 <label><?php echo $val ?></label> 4 <input type="checkbox" value="<?php echo $val ?>" name="company[<?php echo $key ?>][]"> 5 <?php endforeach; ?> 6@else 7 <?php foreach(old("company") as $val): ?> 8 <?php echo $val[0]; ?> 9 <?php endforeach; ?> 10@endif
↓
php
1@if(old('confirming', 'false') === 'false') 2 @foreach ($menu as $key => $val) 3 <label>{{$val}}</label> 4 <input type="checkbox" value="{{$val}}" name="company[{{$key}}][]"> 5 @endforeach 6@else 7 @foreach (old("company") as $val) 8 {{$val[0]}} 9 @endforeach 10@endif
投稿2016/10/17 01:11
編集2016/10/17 01:21
総合スコア606
ご回答ありがとうございます。ご教示頂いたblade記法で再度実行したのですが、上記と同じエラーが出てしまいました。この実装の流れとしてはいわば確認画面付きフォーム画面です。まず入力フォームで企業をチェックボックスで複数選択し確認画面に進みます、その内容で良ければ、ボタンを押して保存する形なのですが、どうしても保存の処理がうまくいかないです。ビューよりもコントロールの記述が原因なんでしょうか？
チェックボックスのLabelにはgoogleなどの企業名を出し、valueに1などを入れる。storeメソッドで[1,2,3]の配列を受け取って "1,2,3" とカンマ区切りの文字列に変換し、DBに保存する。＜ 上記の流れからしては最終的に選んだ企業を一覧画面の表示させたいので、valueには google, yahoo,といったようにカンマ区切りでDBに保存する形にしたいと思っております。教えて頂いた方法でも可能であれば特にこだわりはありません。
> どうしても保存の処理がうまくいかないです。
保存する際にどのようなエラーがでますか？
エラーメッセージ書いてありました。失礼しました。foreachの前にdd()を仕込んで、本当に配列なのか調べてみてください。
ブラウザにはdd(old("company")); google facebook と表示されました。
foreachの前というのは@foreach (old("company") as $val)のことでお間違いないでしょうか
もう一つのforeachにも登録する前に仕込んでみてください。
@if(old('confirming', 'false') === 'false')
dd($menu);
@foreach ($menu as $key => $val)
<label>{{$val}}</label>
<input type="checkbox" value="{{$val}}" name="company[{{$key}}][]">
@endforeach
@else
dd($val);
@foreach (old("company") as $val)
{{$val[0]}}
@endforeach
@endif
こういった形で加えたのですが、うまく反映されてないみたいです。ブラウザにはdd($val); google facebookといった形で表示されます
bladeでphpのコードを実行し出力する際は`{{dd($hoge)}}`のように`{{}}`で囲む必要があります。
ありがとうございます。配列がなっていることは確認できました、にも関わらず、上記のエラーが出てしまうのはどうしてでしょうか。
このエラーはforeachに配列でない変数が渡されたときに起こるエラーですので、配列を渡してこのエラーが起こることはありません。この画面を開く全てのタイミング(初期表示時と登録時など)で確認してみてください。
おっしゃる通り、<input type="checkbox" value="{{$val}}" name="company[{{$key}}][]">のvalueのところがvalue[goolgle] value[yahoo]といったようになっていたためエラーが出ていたと考えられます。おそらくvalue[google,yahoo]といった形であればうまく適用されるのではないかと思うのですが。。
そのため私なりに上記の形に再現できるようにコーディングをしたのですが、
@if(old('confirming', 'false') === 'false')
@foreach ($menu as $key => $val)
<label>{{$val}}</label>
<input type="checkbox" value="{{$val}}" name="company[]">
@endforeach
@else
{{$vals = $_GET['company']}}
@foreach ($vals as $val)
{{$val[0]}}
@endforeach
@endif
しかしUndefined index: companyというエラーが出てしまいました。これは値が入ってないということなんでしょうか。
あなたの回答
tips
プレビュー
