Laravel に よる カスタムバリデーション実装

Question

前提・実現したいこと

Laravel 5.7 で、カスタムバリデーションの実装に関してです。
以下のルールでバリデーション掛けたいと思っています。
※ ソースは適当です

<form>
項目：A
<input type="text" name="a">

項目：B
<input type="text" name="b">

</form>

項目：A・Bは通常必須属性ではない。
項目：A ・B共に入力がない場合、どちらかに値が存在しないといけない。

抜粋ですが以下のようなバリデートルールをクロージャーで実装してみました

// 項目：Bに対するバリデーションルールです
[
  function( $attribute, $value, $fail ) {
     
    if ( empty( $this->getValue( 'a' ) ) && empty( $this->getValue( $attribute ) ) ) {
      return $fail( '項目：Bの入力が必要です' );
    }
  }
];

正常に動きません。原因は分かっています。

• getValue 上記形式だと使えない
• request オブジェクトを引き渡す方法が分からない
• 項目：Bが未入力だとそもそもバリデートが動かない

こういった場合、リファレンスを見て Rule オブジェクトを実装するか？Request オブジェクトを実装するか？
という情報も見てみたのですが、どの方法が良い方法なのかがいまいちわかりません。

required_with、required_without 等も要件とちょっと違う挙動になります。

具体的にこう実装したって方いらっしゃないでしょうか？
仕様自体はそう珍しいものではないので、むりくりやればいいんでしょうが、
FWのお作法に則った実装が出来れば知りたいです。

Answer 1

今回の目的にはカスタムバリデーションなどを用意するまでもなく、

[
    'a' => 'required_without:b',
    'b' => 'required_without:a',
]

で良いのではないかと思います。

なお参考までに未入力でも検査を行うようなルールを追加するには、Validator::extendImplicitを使います。マニュアル「バリデーション」の「暗黙の拡張」を見てください。

Comments

[ShintaroIshida]

リファレンスを読んで色々と試したつもりだったのですがテストが足りなかったようです。
提示してもらったルール定義で想定通りの挙動を確認しました。
もう少し読んだ内容をソースで確認する事をちゃんとしようと思います。

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

Answer 2

このフォームに対して

<form action="/" method="POST">
    @csrf

    <input type="text" name="a">
    <input type="text" name="b">

    @foreach($errors->all() as $error)
        {{ $error }}
    @endforeach

    <button type="submit">Submit</button>
</form>

方法1

Rule::requiredIf()を使うこと。

// 上に
use Illuminate\Validation\Rule;

// クラス内に
public function store(Request $request)
{
    $request->validate([
        'a' => Rule::requiredIf(!$request->b),
        'b' => Rule::requiredIf(!$request->a),
    ]);

    return $request->all();
}

方法2

aとbをマージして、マージした項目を必須にすることでどうでしょうか？

public function store(Request $request)
{
    $request->merge([
        'a_and_b' => $request->a . $request->b,
    ]);

    $request->validate([
        'a_and_b' => 'required'
    ]);

    return $request->all();
}

Comments

[ShintaroIshida]

今回のルールを繰り返し使うのであればRuleクラスの実装も視野にいれていたのですが単体要件だったので見送っています。request パラメータをマージして比較参照するという手もまったく考え付かなかったので、挙動の勉強になりました。ありがとうございます。

今回は、単体要件であったのと既存のルールで実装出来るよと提示頂いたchrg さんをベストアンサーとさせて頂きました。ご了承ください。

Rule実装時に是非とも参考にさせて頂きます。
回答ありがとうございました。