Laravel8 2つのカラムのバリデーション方法

Question

選手登録フォームを作成しています。
選手登録フォームからデータ登録する際に未定義であるという下記のエラーがでております。

Call to undefined method Illuminate\Contracts\Validation\Rule::unique()

##実現したいこと1
テーブルの「team_owner_id」で絞り込んでから「player_no」をuniqueとしてバリデーションしたい。
例：team_owner_idが1、player_noが10が既に登録済みとした場合
team_owner_idが1、player_noが10の登録はできない。
team_owner_idが2、player_noが10の登録はできる。

##テーブル　playersテーブル
team_owner_idは、別で作成済みのteam_ownerテーブルとリレーションしています。

###前提
選手登録をするteam_ownerは、ログインをしておりAuth::id()とteam_owner_idの数字は同じである。
選手登録をするcreate.blade.phpは、他のteam_owner_idを選択することはできない。

public function store(Request $request)
    {
        $request->validate([
            'player_no' => [new Player_no_check],

        ]);

        $player = new Player();
        $player->team_owner_id = Auth::id();
        $player->player_no = $request->player_no;
        $player->player_name = $request->player_name;
        $player->save();
        // dd($player);
        return redirect()
            ->route('team_owner.players.index')
            ->with([
                'message' => '選手登録をしました。',
                'status' => 'info'
            ]);
    }

※下記のバリデーションは自分が分かりやすくなるように「player_no」のみ指定しています。

public function passes($attribute, $value)
    {
        return [
            'player_no' => Rule::unique('players')->where(function ($query) {
                return $query->where('team_owner_id', 1);
            })
        ];
    }

##参考にしたサイト
参考サイト1

参考画像：Laravelmanualより

##試してみたこと1
参考サイトなどをみて、「Player_no_check.php」のようにバリデーションルールを作成したつもりでしたが
記述が間違っているようで未定義となっております。

ここからがどうして良いのかわからず質問にいたりました。

どのようにするとバリデーションを希望のバリデーションを通すことができますでしょうか。

どうぞよろしくお願いいたします。

##現在でているエラー内容
修正後の記述

namespace App\Rules;

// use Illuminate\Contracts\Validation\Rule;
use Illuminate\Validation\Rule;
use App\Models\Team_owner;
use App\Models\Player;



class Player_no_check extends Rule
{
    /**
     * Create a new rule instance.
     *
     * @return void
     */
    public function __construct()
    {
        //
    }

    /**
     * Determine if the validation rule passes.
     *
     * @param  string  $attribute
     * @param  mixed  $value
     * @return bool
     */
    public function passes($attribute, $value)
    {
        dd($attribute, $value);
        return [
            'player_no' => Rule::unique('players')->where(function ($query) {
                return $query->where('team_owner_id', 1);
            })
        ];
    }

    /**
     * Get the validation error message.
     *
     * @return string
     */
    public function message()
    {
        return 'The validation error message.';
    }
}

Error
Object of class App\Rules\Player_no_check could not be converted to string

##試してみたこと2

下記の記述を試したところ「syntax error, unexpected ';', expecting ']'」が表示されました。

 public function passes($attribute, $value)
    {
        return [
            'player_no' => Player::where('player_no',$value)
                   ->where('team_owner_id', 1)
                   ->doesntExist();
        ];
    }

下記に変更をすると「syntax error」はなくなりましたが記述が合っているか自信がありません。

public function passes($attribute, $value)
    {
        
        return [
            'player_no' => Player::where('player_no', $value)
                ->where('team_owner_id', 1)->doesntExist(), //「;」を「,」に変更
        ];
    }

public function store(Request $request)
    {
        $request->validate([
            'player_no' => [new Player_no_check],

        ]);

        $player = new Player();
        $player->team_owner_id = Auth::id();
        $player->player_no = $request->player_no;
        $player->player_name = $request->player_name;
        $player->save();
        // dd($player);
        return redirect()
            ->route('team_owner.players.index')
            ->with([
                'message' => '選手登録をしました。',
                'status' => 'info'
            ]);
    }

この状態で選手の新規登録をおこないました。

結果としては、登録がなんでもできてしまいバリデーションが効いていないような感じになっています。
現在のteam_owner_idが1、player_noを10として何度も登録ができてしまう状態です。

Answer 1

すみませんでした。
詳しく調べた上、元のコードには問題ありません。
変更するべきコードはここだと思います。

 public function passes($attribute, $value)
    {
        return Player::where('player_no',$value)->where('team_owner_id', 1)->doesntExist();
    }

変更前のであってます。

use Illuminate\Contracts\Validation\Rule;

class Player_no_check implements Rule

Comments

[seven_7]

お返事ありがとうございます。
質問欄に試してみたこと2を追記いたしました。

教えていただいているのに、まだ躓いております。
申し訳ございません。

下記の記述の意味は、Playerモデルをまずplayer_noで絞り、その中からteam_owner_idが1のものがなければtrue、あればfalseの意味で理解しています。

この通り動けば確かにいけると思っておりますが、普通に登録ができてしまう状態です。
なぜ登録ができてしまうのかを模索中です。

```
return [
'player_no' => Player::where('player_no', $value)
->where('team_owner_id', 1)->doesntExist(),
];
```

[skys215]

登録できたplayerのteam_owner_idも同じですか？

[seven_7]

お返事ありがとうございます。
はい、team_owner_idも同じになります。
(今回のteam_owner_idは1)

team_ownerはログインをしておりまして、登録時は自分のteam_owner_idでしか登録ができない想定でいます。

現在は、team_owner_idが1の人がplayer_no10をなんども登録できてしまうということになります。

[skys215]

回答を更新しました。passesはtrue/falseを返さなけらばいけません。arrayを返してたから、いつまでもtrueとなって登録ができてしまいました。

[seven_7]

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

も、も、もしかしたらできたかもしれません！

public function passes($attribute, $value)
{

$player = Player::where('player_no', $value)
->where('team_owner_id', Auth::id());

return $player->doesntExist();
}

上記のように記述をしたところ、team_owner_id「1」の人がplayer_no「10」の2回目を登録しようとしたらバリデーションがかかりエラーメッセージが表示されました。
他のplayer_noの場合はエラーがでずちゃんと登録することができました。

また、team_owner_id「2」の人のplayer_no「10」は登録ができまして、同じくもう1回「10」の登録はできずエラーが表示されました。

一応いまのところ希望の動作になったかと思われます。

[seven_7]

skys215様
この度は、たくさんお時間をとっていただきありがとうございました。
いろいろと確認して一応希望の動作となりました。

個人的になかなか先が見えない中で、skys215様のご助言のおかげでなんとか
希望の動作ができました。

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

Answer 2

Illuminate\Contracts\Validation\Ruleではなく、

use Illuminate\Validation\Rule;

に変えて試してください。

Comments

[seven_7]

お返事ありがとうございます。

Player_no_checkのuse文を変更しました。
変更前：use Illuminate\Contracts\Validation\Rule;
変更後：use Illuminate\Validation\Rule;

エラー内容
App\Rules\Player_no_check cannot implement Illuminate\Validation\Rule - it is not an interface
Ruleの実装ができない？というようなエラーがでてしまいました。

Ruleの実装にあたりもしかすると基本的な記述が足りていないのでしょうかね。
引き続き考えてみます。

お忙しいところ申し訳ございませんが、他にわかることなどありましたらよろしくお願いたします。

[skys215]

class Player_no_check implements Ruleではなく、class Player_no_check extend Ruleに変えてみてください。

[seven_7]

お返事ありがとうございます。

とりあえず記述変更をして試した結果のみの返信となります。

変更後：class Player_no_check extends Rule

エラー内容：Class App\Rules\Player_no_check cannot extend from interface Illuminate\Contracts\Validation\Rule
拡張ができないエラーとなりました。

むむーなかなか難しいですね。

[skys215]

すみません。二つとも変える必要があります。
変更前：use Illuminate\Contracts\Validation\Rule;
変更後：use Illuminate\Validation\Rule;
変更前：class Player_no_check implements Rule
変更後：class Player_no_check extends Rule

[seven_7]

お返事ありがとうございます。

先程変更をして試してみました。
use Illuminate\Validation\Rule;
class Player_no_check extends Rule

エラー内容：オブジェクトが文字列に変換できない
Object of class App\Rules\Player_no_check could not be converted to string

初めてみるエラーでしたので、どういったエラーなのか調べ中ではあります。
私が簡単なフォームの登録・保存・編集・更新・削除がやっとできるようになった初心者レベルですので理解には時間がかかりそうです。

似たような事例のサイトをいくつも見て理解を深めようとしておりますが、バリデーション部分だけでもなかなか奥が深いですね。

[skys215]

エラーはどのファイルが吐き出したのですか？
もしvendor/フォルダーじゃなかったら、コードをお願いします。

[seven_7]

お忙しい中お返事ありがとうございます。

質問に「現在でているエラー内容」を追記いたしました。

[seven_7]

エラーは新規登録(create.blade.php)に入力をして、登録ボタンを押したあとに追記をさせていただいたエラーが表示されました。

[skys215]

いや、これは...vendor\laravel...フォルダーですね。
それともエラーのスクショをアップできますか？

[seven_7]

仰っていることを理解できず申し訳ございません。
今、私が言っているエラー画面を質問欄に更新しました。

他にエラー出力される場所があるということでしょうか。