編集履歴

質問編集履歴

試してみたこと2を追記。エラー画像削除

2021/12/08 10:39

投稿

seven_7

スコア7

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -124,7 +124,7 @@
-##試したみたこと
+##試してみたこと1
 参考サイトなどをみて、「Player_no_check.php」のようにバリデーションルールを作成したつもりでしたが
@@ -260,14 +260,106 @@
+##試してみたこと2
-C:\xampp\htdocs\fifa_app\vendor\laravel\framework\src\Illuminate\Validation\ValidationRuleParser.php:114
+下記の記述を試したところ「syntax error, unexpected ';', expecting ']'」が表示されました。
-エラー画像
-今私がエラーがでましたと言ってみている画像は下記になります。
+```Player_no_check.php
-![イメージ説明](2fdcf7fae30261ac43cb9fb5020ba446.jpeg)
+ public function passes($attribute, $value)
+    {
+        return [
+            'player_no' => Player::where('player_no',$value)
+                   ->where('team_owner_id', 1)
+                   ->doesntExist();
+        ];
+    }
+```
+下記に変更をすると「syntax error」はなくなりましたが記述が合っているか自信がありません。
+```Player_no_check.php
+public function passes($attribute, $value)
+    {
+        return [
+            'player_no' => Player::where('player_no', $value)
+                ->where('team_owner_id', 1)->doesntExist(), //「;」を「,」に変更
+        ];
+    }
+```
+```PlayerController
+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として何度も登録ができてしまう状態です。

エラー画像を追加12/8　13：07

2021/12/08 10:39

投稿

seven_7

スコア7

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -262,128 +262,12 @@
 C:\xampp\htdocs\fifa_app\vendor\laravel\framework\src\Illuminate\Validation\ValidationRuleParser.php:114
-```エラー場所コード
+エラー画像
- * @return mixed
-     */
-    protected function prepareRule($rule)
-    {
-        if ($rule instanceof Closure) {
+今私がエラーがでましたと言ってみている画像は下記になります。
-            $rule = new ClosureValidationRule($rule);
-        }
-        if (! is_object($rule) ||
-            $rule instanceof RuleContract ||
-            ($rule instanceof Exists && $rule->queryCallbacks()) ||
-            ($rule instanceof Unique && $rule->queryCallbacks())) {
-            return $rule;
-        }
-        return (string) $rule; //ここでエラーがでている
+![イメージ説明](2fdcf7fae30261ac43cb9fb5020ba446.jpeg)
-    }
-    /**
-     * Define a set of rules that apply to each element in an array attribute.
-     *
-     * @param  array  $results
-     * @param  string  $attribute
-     * @param  string|array  $rules
-     * @return array
-     */
-    protected function explodeWildcardRules($results, $attribute, $rules)
-    {
-        $pattern = str_replace('*', '[^.]*', preg_quote($attribute));
-        $data = ValidationData::initializeAndGatherData($attribute, $this->data);
-```

現在進行中の内容を追記12/8

2021/12/08 04:07

投稿

seven_7

スコア7

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -141,3 +141,249 @@
 どうぞよろしくお願いいたします。
+##現在でているエラー内容
+修正後の記述
+```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
+```
+C:\xampp\htdocs\fifa_app\vendor\laravel\framework\src\Illuminate\Validation\ValidationRuleParser.php:114
+```エラー場所コード
+ * @return mixed
+     */
+    protected function prepareRule($rule)
+    {
+        if ($rule instanceof Closure) {
+            $rule = new ClosureValidationRule($rule);
+        }
+        if (! is_object($rule) ||
+            $rule instanceof RuleContract ||
+            ($rule instanceof Exists && $rule->queryCallbacks()) ||
+            ($rule instanceof Unique && $rule->queryCallbacks())) {
+            return $rule;
+        }
+        return (string) $rule; //ここでエラーがでている
+    }
+    /**
+     * Define a set of rules that apply to each element in an array attribute.
+     *
+     * @param  array  $results
+     * @param  string  $attribute
+     * @param  string|array  $rules
+     * @return array
+     */
+    protected function explodeWildcardRules($results, $attribute, $rules)
+    {
+        $pattern = str_replace('*', '[^.]*', preg_quote($attribute));
+        $data = ValidationData::initializeAndGatherData($attribute, $this->data);
+```