Laravelのバリデーション

Question

Laravelでフォームのバリデーションはどのように行うのが適切なのでしょうか？
参考書を見てもいろいろなやり方があって混乱しています。

(1) コントローラのvalidateメソッドを使う。↓

//ルートは Route::post('hello', 'HelloController@post'); とする

//App\Http\Controllers\HelloController.php
class HelloController extends Controller
{
    public function post(Request $request){
        $rule = [
            'name' => 'required',
            'mail' => 'email',
        ];
        $this->validate($request, $rule);
        return view('hello.form');
    }
}

(2) フォームリクエストを作り、POSTリクエストがコントローラに渡される前にバリデーションする↓

// App\Http\Requests\HelloRequest.php
class HelloRequest extends FormRequest
{
    public function rules(){
        return [
            'name' => 'required',
            'mail' => 'email',
        ];
    }
}

// App\Http\Requests\HelloRequest.php
class HelloController extends Controller
{
    public function post(HelloRequest $request){
        return view('hello.form');
    }
}

(3) コントローラ内でValidatorクラスのインスタンスを使う↓

// App\Http\Requests\HelloRequest.php
class HelloController extends Controller
{
    public function post(Request $request){
        $validator = Validator::make($request->all(), [
            'name' => 'required',
            'mail' => 'email',
        ]);
        return view('hello.form');
    }
}

(3)はエラーがあった時、フォームにリダイレクトせずに他の処理を行うことができたりと一番汎用的に使えますか？

Laravelを使っている方、教えていただきたいです。

Comments

[m.ts10806]

公式リファレンスの記事やコードを確認された上での質問と言うことですよね？

[kriht]

はい。リファレンスには「より複雑なバリデーションのシナリオではフォームリクエストを生成したほうが良い」とありました。なので基本はvalidateメソッドを使うか、Validatorインスタンスを使う方法で良いのかと思いました。

[kriht]

コードを動かしましたがバリデーションは同じくできたので、はじめは使いやすい方法で行うのがよいですか？

[m.ts10806]

私はそれでよいと思います >基本はvalidateメソッドを使うか、Validatorインスタンスを使う方法

[m.ts10806]

「こうじゃなきゃいけない」という方法はないので、まずはリファレンスにも載っている内容で要件を実現ができるのを確認できたらそれで進めていくのが良いと思います。ゴリゴリに自分で手続き的に処理を書いたりするのではなく、フレームワークの機能を使っているのであればまずは問題ないかと。

[kriht]

わかりました。ありがとうございます。

Answer 1

基本的にはどちらでも構わないと思いますが、規模やコードスタイルによって左右するかな（？）と思われます。

フォームリクエストを作り、POSTリクエストがコントローラに渡される前にバリデーションする

例えば データの更新処理（UPDATE） , データの新規登録（POST） 等 フォーム内容が同じ場合があったとすると、
バリデーション内容の更新が合った際にフォームリクエストを変更するだけでどちらの変更も適応できます。

逆の場合に、 UPDATE , POST の双方に差異が発生した場合には、フォームリクエストを増やすか、インスタンス作成で行うなどの手間が出来ます。

コントローラ内でValidatorクラスのインスタンスを使う

こちらは汎用的に利用できると思いますが、リダイレクトの処理や上記のような場合は記述部分が増えますし、
コントローラ内部で処理が増える（そもそもService等でカバーしなければいけないが・・・）と可読性が低下します。

結論：処理的にはどちらでもOKで、規模や可読性の向上・ソースコードの省略などなどでどちらかを使用すれば良いかなと思います。

個人的な意見でしたが、参考になれば幸いでございます。

Comments

[kriht]

参考になりました。ありがとうございます。