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

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

ただいまの
回答率

88.77%

Laravel 5.8で配列のリクエストのバリデーションができません

解決済

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 1,570

natsume2233

score 116

前提・実現したいこと

Laravel5.8で勤怠システムを作成しています。
テーブルの形で、出勤時間、退勤時間、合計時間、作業内容、プロジェクトコードに入力欄があります。

formは下記のようになっています。(inputが入力欄の想定)

出勤時間 退勤時間 合計時間 作業内容 プロジェクトコード
input input input input input
input input input input input
input input input input input

行は複数行同時登録可能な仕様です。
それを、登録のためPOSTした際のバリデーションで
1行目は必須、2行目は出勤時間があれば、退勤時間必須...のようにバリデーションをかけたいです。

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

POSTした際に、何も入力していない場合、nullの配列が送られ、思うようにバリデーションがかかりません。

イメージ説明

該当のソースコード

ここでは10個決め打ちしていますが実際は何個になるか不明です。

Blade

<table class="table white-space-nowrap">
    <thead>
        <tr>
            <th scope="col">出勤時間</th>
            <th scope="col">退勤時間</th>
            <th scope="col">合計</th>
            <th scope="col">作業内容</th>
            <th scope="col">pj番号</th>
        </tr>
    </thead>
    <tbody>
        @for ($i = 0; $i < 10; $i++) <tr>
            <td>
                <input type="text" placeholder="00:00" name="start_time[]">
            </td>
            <td>
                <input type="text" placeholder="00:00" name="end_time[]">
            </td>

            <td class="input-size-s">
                <input type="text" name="sum_time[]" disabled="disabled" placeholder="合計">
            </td>
            <td>
                <input type="text" placeholder="作業内容" name="work_content[]">
            </td>
            <td>
                <input type="text" disabled="disabled" name="pj_num[]" placeholder="pj番号">
            </td>
            </tr>
            @endfor
    </tbody>
</table>

FormRequest

 public function rules()
    {
        return [
            'start_time' => 'required|array|min:1|',
            'start_time.*' => 'nullable|date_format:H:i|',
            'end_time' => 'required|array|min:1',
            'end_time.*' => 'required_with:start_time.*|date_format:H:i|after:start_time.*',
            'sum_time' => 'required|array|min:1|',
            'sum_time.*' => 'required_with:start_time.*|date_format:H:i|',
        ];
    }

試したこと

バリデーションがかかる前に、array_filterでnullを削除しようとしましたが、全部からだとエラーが出てうまくできていません。

大変申し訳ありませんが、お力を貸していただけますと助かります。
初めての質問で不備等ありましたら追記しますので、宜しくお願い致します。

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 4

checkベストアンサー

0

start_timeとend_timeをについて、

・1行目が必須
・2行目以降は、start_timeがあれば、end_timeも

なら下記でいけるはず・・。

'start_time.0' => 'required',
'end_time.0'   => 'required',
'start_time.*' => 'nullable|date_format:H:i',
'end_time.*'   => 'required_with:start_time.*|nullable|date_format:H:i|after:start_time.*',

提示のbladeを見る限り、inputのnameが配列形式(***[])なので、1行目の必須は「0」指定で書いてしまいます。

なお、「0」もそれ以外「.*」も共通でチェックしたいもの(時間の形式など)は、
「.*」の方に書いておけば「0」もチェックされるので、「0」はrequiredのみで問題ないとおもいます。(※)

(※)動作確認をlaravel6.0で行ったので、もし挙動が違った場合はすみません^^;


それと、質問の本筋とは関係ありませんが、
nullを許容する可能性のある項目には、全てnullableも並記しておいた方が良いですよ。

質問に提示されたコードより、下記のルールを例にあげますと、

'sum_time.*' => 'required_with:start_time.*|date_format:H:i|',

sun_time[1]を、空のまま送信したとします。

start_time[1]がなければ、required_withのバリデーションチェックは通りますが、nullableが設定されていない為、次のdate_formatが空の送信値nullに対して実行され、バリデーションエラーになってしまいます。

以上、ご参考までに・・。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/11/05 11:51

    できました!有難うございました
    とても勉強になりました。
    補足事項つけて頂き有難うございます、まさにそこにも引っかかっていたようです
    助かりました。有難うございます

    キャンセル

0

自動バリデートではなく、カスタムバリデート組む形になると思います。
カスタムというか、行個々にバリデートするかしないか決める形。
つまり、「その行でひとつでも入力があれば」を条件にしてバリデート通すイメージですね。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

出勤時間があれば、退勤時間必須...

以下のサイトが参考になるかと思います。
※Laravelのバージョンは違いますが、大丈夫かと…

https://www.larajapan.com/2016/10/23/バリデーション-8-配列をバリデーションする/

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/10/28 19:05 編集

    回答ありがとうございます
    こちらを参考に実装したものなのですが、うまくいかず...
    また考えてみます

    回答有難うございました

    キャンセル

0

ある項目に入力あった場合は他の項目も必要
みたいなのがしたいならバリデーションのrequired_withを駆使したらいけそうな気がします

バリデーション 5.8 Laravel

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/10/28 19:00

    回答ありがとうございます
    既に使用していますが、POSTしたデータがnullでも配列として存在しているため機能していません...

    回答有難うございました

    キャンセル

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

  • ただいまの回答率 88.77%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る