質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
Laravel

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

Q&A

解決済

1回答

3929閲覧

Laravelでチェックボックスにバリデーションがかからない

aki0201

総合スコア5

Laravel

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

0グッド

1クリップ

投稿2021/02/25 08:46

編集2021/02/27 05:55

実現したいこと

新規登録フォームで、日付の登録とチェックボックスを使用して値の登録ができるようにしているのですが、日付のバリデーションがかかっているのにチェックボックスのバリデーションがかからず困っています。
最終的にはチェックボックスにチェックが入っていない時は、登録できないようにしたいです。

該当のソースコード

home.blade.php

<form id="task-form" method="POST" action="{{ route('done_task') }}"> @csrf <div class="tasks-group"> <p>できたことをチェック</p> <input id="brush" type="checkbox" name="title[]" value="ブラシ洗浄"><label for="brush">ブラシ洗浄</label> <input id="puff" type="checkbox" name="title[]" value="パフ洗浄"><label for="puff">パフ洗浄</label> <input id="pack" type="checkbox" name="title[]" value="顔のパック"><label for="pack">顔のパック</label><br> <input id="peeling" type="checkbox" name="title[]" value="ピーリング"><label for="peeling">ピーリング</label> <input id="treatment" type="checkbox" name="title[]" value="トリートメント"><label for="treatment">ヘアトリートメント</label><br> <div class="form-check form-check-inline"> <label for="other"><input type="checkbox"><label for="other">その他</label><input id="other" type="text" name="title[]"> </div> </div> <div class="tasks-group"> <label>登録する日付</label><input id="start" type="date" class="task-form" name="start" value=""> </div> <div class="modal-btn"> <button class="modal__btn modal__btn-primary mr-3" type="submit">登録</button> <a id="expire-btn" class="modal__btn cancel-btn" data-micromodal-close aria-label="Close this dialog window">キャンセル</a> </div> </form>

Request\Done.php

class Done extends FormRequest { /** * Determine if the user is authorized to make this request. * * @return bool */ public function authorize() { return true; } /** * Get the validation rules that apply to the request. * * @return array */ public function rules() { return [ 'title' => 'required|max:100', 'start' => 'required', ]; } }

DoneTaskController.php

public function store(Done $request) { $task = new DoneTask; $task->user_id = Auth::id(); $task->start = $request->input('start'); if(isset($_POST['title'])) { $title = implode(',', $_POST['title']); $task->title = $title; } $task->save(); return redirect('/home'); }

Done.phpでチェックボックスのname属性であるtitlerequireにしているのですが、チェックボックスを入力しなくても登録できるようになっています。
どなたか、ご教授いただけると助かります。

補足情報

-各バージョン
-Laravel 6.x
-PHP 7.4.9

追記

アドバイスをもとに、コントローラーを下記のように変更しました。
でもまだバリデーションはかからないようです。どの点に問題があるのでしょうか。

DoneTaskController.php

public function store(Done $request) { $task = new DoneTask; $task->user_id = Auth::id(); $task->start = $request->input('start'); $title = $request->input('title'); $task->title = implode(',', $title); $task->save(); return redirect('/home'); }

コメントを参考に、Done.phpで下記のようにデバッグをしましました。

public function rules() { dd($tihs->title); return [ 'title' => 'required|max:100', 'start' => 'required', ]; }

何もチェックを入れない場合は、array:1 [▼0 => null]となり、チェックを入れるとarray:2 [▼0 => "顔のパック" 1 => null]となりました。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

m.ts10806

2021/02/25 08:52

Laravelで$_POSTを直接参照はないと思います
kai0310

2021/02/27 10:18 編集

バリデーションしているならissetで確認する必要ないのでは?
aki0201

2021/02/25 23:48

アドバイスを参考にコントローラーを変更してみました。 ご確認いただけるとありがたいです。
Lulucom

2021/02/27 01:05 編集

フォームリクエストのクラスのrulesメソッドのreturnの前辺りで$this->titleの値を表示させてみると、チェックが入っていない時の中身はどうなっていますか? dd($this->title); // デバッグ用(一時的)
aki0201

2021/02/27 04:10

dd($tihs->title);をreturnの前で行ったところ、何も選択しない場合は array:1 [▼ 0 => null ] となり、選択をした場合は array:2 [▼ 0 => "顔のパック" 1 => null ] と出てきました。
Lulucom

2021/02/27 04:16

ありがとうございます、こちらへのご返信ではなく質問へ追記された方が良いと思います。 この問題私も解決方法がわからなくて困っています。回避策があるにはあるのですが・・・
aki0201

2021/02/27 05:55

アドバイスありがとうございます! 質問に追記させていただきました。
guest

回答1

0

ベストアンサー

(正しい対処法かどうかはわからないのですが・・・)

prepareForValidationメソッドを使用して、配列からnullの要素を除去するのはいかがでしょうか。

Requests\Done.php

php

1protected function prepareForValidation() 2{ 3 $this->merge([ 4 'title' => array_filter($this->title), 5 ]); 6}

投稿2021/02/27 12:31

Lulucom

総合スコア1899

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

aki0201

2021/02/28 14:27

試したところバリデーションがかかるようになりました! ありがとうございます!! ちなみに、どこか懸念点があるのでしょうか? (正しいかわからないと書かれてたので…)
Lulucom

2021/02/28 16:12

はい、LaravelドキュメントやWebで検索しても、この問題への対処法をまだ見つけることができていません。探し方が悪いのかもしれませんが・・・
aki0201

2021/03/01 12:44

わざわざ調べていただいてありがとうございます。 バリデーションはかかっているので、こちらを利用させていただければと思います。 いつも本当にありがとうございます!!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問