リダイレクト先が異なるバリデーションをFormRequestでまとめたい

Laravel 5.6 でバリデーションを行う際、FormRequestを使って処理しようと考えています。

項目登録フォームは複数のページにありますが、バリデーション内容自体はほぼ同じです。
しかし、バリデーションエラーの際の戻り先にアンカー(#comment-formなど)をつけたいと思っているため、１つのFormRequestクラスにまとめられずにいます。

あるページでは「#comment-form」のアンカーが使われていて、別のページでは名前が違ったりアンカー自体使われていないページがあるなど、ページによって戻り先がやや異なります。

Validator::make() の形を使えば失敗時の遷移先は自由に変えられますが、rules や messages, attributes は共通なので使い回せたらいいと思っています。

FormRequest は$url->previous()の内容を読んでいるので、これを書き換えできれば実現できそうにも見えますが方法がわかっておらず、ただしいやりかたなのかも自身がありません。

一時的に遷移先を変えたいだけであっても、もととなる FormRequest クラスを extends し、failedValidation() か getRedirectUrl() をオーバーライドした FormRequest クラスをフォームごとに用意する他ないのでしょうか？

Illuminate\Foundation\Http\FormRequest

php
1    protected function failedValidation(Validator $validator)
2    {
3        throw (new ValidationException($validator))
4                    ->errorBag($this->errorBag)
5                    ->redirectTo($this->getRedirectUrl());
6    }
7
8    protected function getRedirectUrl()
9    {
10        $url = $this->redirector->getUrlGenerator();
11
12        if ($this->redirect) {
13            return $url->to($this->redirect);
14        } elseif ($this->redirectRoute) {
15            return $url->route($this->redirectRoute);
16        } elseif ($this->redirectAction) {
17            return $url->action($this->redirectAction);
18        }
19
20        return $url->previous();
21    }

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

回答2件

ベストアンサー

たくさんFormReqeustをつくる方法

共通のrules や messages, attributes だけ書かれたFormRequest(仮にBaseFormRequestとします）を作成し、BaseFormRequestの継承先でリダイレクト先だけ指定する、といった形が丁寧な気がします。

この方法の利点は仕様変更に比較的強い点です。
rules/messages/attributesが変更になった場合、BaseFormRequestを変更すればすべてを変更できますし、BaseFormRequestの継承先を変更すれば個別の修正も容易だからです。
デメリットはクラスが増えること、コード量が増えることでしょうか。

Validator::make()でやる方法

rules/messages/attributesだけ適当なクラスにまとめておいて共通化することも可能だと思います。
たとえば、以下のような感じです。

php
1class MyFormValidator {
2  static public $rules = [];
3  static public $messages = [];
4  static public $attributes = [];
5}

php
1$validator = Validator::make($request->all(), MyFormValidator::rules, MyFormValidator::messages, );

こちらの方がコード量は少ないと思いますが、仕様変更の際には新しいクラスをつくったりとちょっと面倒になるかと思います。

※ コードは動かしてないのでちょっとそのままだと動かないかもしれません。

リダイレクト先変更方法について

ついでですが、FormRequestでリダイレクト先だけ変更したいのでしたら、
メソッドをオーバーライドするのではなくてプロパティをオーバーライドすれば大丈夫なはずです。

以下、参考にしてみてください。
https://qiita.com/rinkun/items/8b43fff78e4acb91941e

投稿2018/05/15 02:13

YamajiDaisuke

総合スコア215

丁寧な方法はYamajiDaisukeさんがご説明されているので、雑な方法を提案してみます。

■　FormRequest　１つで乗り切りたい

フォームリクエストのファイルを一つだけ作ります。
その中で、エラー時の戻り処理をオーバーライドし、
本来の戻り先URLから、追加したいアンカーを追記してみる

php
1namespace App\Http\Requests;
2
3use Illuminate\Foundation\Http\FormRequest;
4use Illuminate\Contracts\Validation\Validator;
5use Illuminate\Http\Exceptions\HttpResponseException;
6
7class MyFormRequest extends FormRequest
8{
9
10/*　message、rules など共通の設定部分は省略*/
11
12    protected function failedValidation( Validator $validator )
13　　　　$redirect_url = $this->getRedirectUrl();
14
15        //自動で戻される想定のURLを判定基準にしてみる
16        switch($redirect_url){
17            case 'a':
18                $redirect_url .= '#a';
19                break;
20            case 'b':
21                $redirect_url .= '#b';
22                break;
23            default:
24                $redirect_url .= '#default';
25                break;
26        }
27　　　　throw (new ValidationException($validator))
28                    ->errorBag($this->errorBag)
29                    ->redirectTo($redirect_url);
30    }
31}