FormRequestを改造して。バリデーション時の仕様を独自にさせたい。

お世話になってます。

Laravelのバリデーションの実装をしており、自作したFormRequest下記のように記述し
バリデーションが発生したらJSONでcodeとmessagesを返すようにしました。

しかし、この書き方ではフォームが増えたらその数だけfailedValidationのif文を
追加しないといけないと思いコンパクト化を試みてます。

フォームが増えてもrulesメソッドの追加だけにしたい為
継承しているFormRequestクラスのfailedValidationメソッドを書き換えるなど
試行錯誤しているのですがうまく実装出来ません。

<?php

namespace App\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;
use Illuminate\Contracts\Validation\Validator;
use Illuminate\Http\Exceptions\HttpResponseException;

class LikeRequest 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 [
            'to_user_id' => 'required'
        ];
    }

　　//デフォルトのものに下記メソッドを追加
    protected function failedValidation(Validator $validator)
    {
            $response['code'] = 'XXX';

        //もしidにエラーがあれば
        if($validator->errors()->has('to_user_id')) {
            $response['message'] = $validator->errors()->first('to_user_id');
            throw new HttpResponseException(response()->json($response, 422));
        }
    }

}

試行錯誤内容としましては

・既存のFormRequestクラスのfailedValidationメソッドを下のように変えても

    protected function failedValidation(Validator $validator)
    {
        throw (new ValidationException($validator))
                    ->errorBag($this->errorBag)
                    ->redirectTo($this->getRedirectUrl());
    }
↓
↓
↓
    protected function failedValidation(Validator $validator)
    {
            $response['code'] = 'XXX';

        if($validator->errors()->has($xxxxxxxxx)) {
            $response['message'] = $validator->errors()->first($xxxxxxxxx);
            throw new HttpResponseException(response()->json($response, 422));
        }
    }
$xxxxxxxxxの箇所を汎用的に使用できる変数にして
この部品だけですべてのすべてのバリデーション処理ができるようにさせたい
と色々試しております。

コーディング簡略化に自信のある人がいましたら
よろしくお願いします。

m.ts10806

2019/09/16 20:39

どのあたりを試行錯誤したか、提示のコードだけではわからないので、もっと具体的に記載していただければと。

shoco

2019/09/17 02:29

返信ありがとうございます追記しましたのでよろしくお願いいたします。

行動規範の内容に同意します

回答1件

ベストアンサー

単純に、

        //もしidにエラーがあれば
        if($validator->errors()->has('to_user_id')) {
            $response['message'] = $validator->errors()->first('to_user_id');
            throw new HttpResponseException(response()->json($response, 422));
        }

こんお処理を繰り返したいだけなら、

<?php

namespace App\Http\Requests;

use Illuminate\Contracts\Validation\Validator;
use Illuminate\Foundation\Http\FormRequest;
use Illuminate\Http\Exceptions\HttpResponseException;

class LikeRequest 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 [
            'to_user_id' => 'required'
        ];
    }

　　//デフォルトのものに下記メソッドを追加
    protected function failedValidation(Validator $validator)
    {
        $keys = [
            'to_user_id',
            'xxxxx',
        ];
        $this->checkAll($validator, $keys);
    }

    private function checkAll(Validator $validator, array $keys = [])
    {
        foreach ($keys as $key) {
            if ($validator->errors()->has($key)) {
                $response['message'] = $validator->errors()->first($key);
                throw new HttpResponseException(response()->json($response, 422));
            }
        }
    }

}

こんな風にすればいいと思うけど、ユーザービリティ的にアウトだと思う。
せっかく、Validator がエラーをまとめて表示する機能がそもそもついているのに、エラーごとに処理を止めてしまうのは本末転倒だし、デグレにしかならない。

投稿2019/09/17 05:23