バリデーションbetweenがうまく動いてくれない[ Laravel ]

Question

バリデーションルールが思う通りに動いてくれない。

環境

• PHP 7.2
• Laravel 5.3

期待する動き

送られてきた'quantity'データが1~12の数字であることをバリデーションする。

コードと挙動

セレクトボックスで送った値をバリデーションしています。
1,10,11,12の数字の時は通り、2~9の時のみバリデーションエラーがはかれます。

バリデーションエラーメッセージ
The quantity must be between 1 and 12 .

FormRequest

 public function rules()
    {
        return [
           "quantity" => 'numeric | between:1,12 | required',
        ];
    }

Contoroller

 public function add(CartFormRequest $req){
        $this->cartService->add_cart($req);
        return view('cart.add');
    }

blade

      <select name="quantity">
            <option value="1">1</option>
            <option value="2">2</option>
            <option value="3">3</option>
            <option value="4">4</option>
            <option value="5">5</option>
            <option value="6">6</option>
            <option value="7">7</option>
            <option value="8">8</option>
            <option value="9">9</option>
            <option value="10">10</option>
            <option value="11">11</option>
            <option value="12">12</option>
        </select>

よろしくお願い致します。

###　補足
between1,9 でバリデーション かけた時、12でも通ってしまっています。

Comments

[m.ts10806]

バージョンはマイナーバージョンまで記載願います。
PHP7でも.0と.1ではサポート状況変わりますので

[zushi0905]

修正致しました。ご指摘ありがとうございます。

[m.ts10806]

あと、「クラス名」「ファイル名」より「php」のように言語名にされたほうがコードハイライトもつきます。
また、Requestも含めてコード全体の提示は可能ですか？
なるべくコピペで再現確認できるようにしてもらえると助かります。
想像でコードを埋めると都合がいいように書いてしまうので正しく再現確認できませんので。

[退会済みユーザー]

| で分断したあとトリムなんてしてくれないと思うがどうなんだろう?

[zushi0905]

申し訳ないです。不勉強なものでトリムとはなんでしょうか？
また具体的にはどのようにしたらいいのでしょうか？
ご教授願いたいです。

[m.ts10806]

https://www.php.net/manual/ja/function.trim.php
つまり余計な空白を取り除くこと。
ドキュメントでは空白なしで書かれているのでそれに倣っておくべきとは思います。

[zushi0905]

空白消したところ通りました！ありがとうございます！！！
ベストアンサーをあげたいんでできれば回答の方にもう一度投稿お願いします！
明後日までになければ自己解決法で記入しておきます！

[m.ts10806]

了解です。
ちょっと根拠込みで回答調整して投稿します。

Answer 1

文字列として処理されてるんじゃないんかねえ
digits_betweenを使うと幸せになれそう
無理にbetween使わずとも、おとなしくmin、max使うとか、出来る方法で実装してさっさと次に進んでしまうのもありかもね

Comments

[zushi0905]

今回は1~12ってことなんですよ...。digits_between使うと1〜99はできますけど1~12って実装方法が思いつかないんです。何か方法があったらご教授ください。

min,maxで1回やったんですが、2〜9の数字入れた時にmaxの方のバリデーション エラーの12以下の数字入れろって言われてしまいました。2~9が12以上として扱われている可能性があるんですが、numeric通っている時点で数値として入っていると思いますし、正直どん詰まりですね。

カスタムバリデーション組んでみる感じで次は試してみます。

[hentaiman]

mixmaxでもダメなんだ、そしたら分からない
そしたらカスタムバリデーションの方がいいですね
こっちもemailバリデーション使おうとしたけどまともに動作しなかった事あるし、バグなのかもね（ドキュメント上ではエラーになるはずのものが通った）

[zushi0905]

割り切ることができました、ありがとうございます。他の方法を模索してみます。

[退会済みユーザー]

integer でなく、なぜnumericを使うんだろう…（独り言）

[hentaiman]

ついで言うと、数値程度ならregexを使えばいい　今のところ正規表現ぐらいはまともに動作してる
直接function書いて単純なifでもいいと思うけど、見た目的に。

[zushi0905]

確かにintegerの方がいいですね。

Answer 2

質問修正依頼にて、確認してもらおうと思っていた内容で解決されたようなので質問者さんのご依頼により回答として投稿します。

Valiadtionの項より、サンプルコードでも 'title' => 'required|unique:posts|max:255',のように詰めて書かれているので、倣うべきと思います。
そもそもフレームワークのコア機能的に、詰められるのが前提で組まれているか何かで、正しく各ルールやパラメータが認識されていない可能性があります。

どのようにバリデーションルールを分解しているかは、Validationのコアソース”/src/Illuminate/Validation/ValidationRuleParser.php”を読んでいくとなんとなく見えてくるとは思います。
|でexplodeするところとか,でexplodeされるところとか、一応、trimもされているようですね。
バリデーション処理のメイン部分#validateBetween()とか追ってみるといいかもしれません。
var_dump()など、デバッグ仕込めばそれぞれのパラメータの値とかも見ることができます。

Comments

[zushi0905]

ありがとうございます。時間がある時に自分でもコア部分を読みに行きたいと思います。

[m.ts10806]

そうですね。コア部読み込みは勉強にはなるけどかなり複雑にさまざまな要素が絡み合っていてピンポイントな情報を見つけ出すのは骨がおれます。
まずは「そういうもの」と飲み込んでドキュメントの書き方に倣うのが吉です