Как правильно вернуть ошибку из if?

Question

[Николай Стешенко]

Есть код призванный проверять данные пришедшие из формы с данными находящимися в базе. Все работает только не могу вернуть ошибку, если данные из базы и данные из формы не совпадают. Как правильно решить задачу?

public function Emptyinvate($invait)
	{
		 $token = Invaite::all()->first();
	
      if($invait == $token->invaite_token){
        return $invait; 
      }else{
		  return responсe(['error' => ['message' => 'Приглашение не существует']]);  
    }
}
	
    protected function create(array $data)
    {
		$invait = $this->Emptyinvate($data['invaite_token']);
        return User::create([
            'name' => $data['name'],
            'email' => $data['email'],
			'invaite_token' => $invait,
            'password' => Hash::make($data['password']),
        ]);
    }
}

Comments

[JhaoDa]

Ты реально так код пишешь, по три пустых строки отступов?

[Сергей delphinpro]

С именованием беда.
Слово правильно пишется так: invite
Emptyinvate — принято использовать верблюжью нотацию: emptyInvite, возможно даже лучше так - inviteIsEmpty

Answer 1

[Сергей delphinpro]

Вот этот раздел внимательно прочитайте и все вопросы отпадут: https://laravel.com/docs/8.x/validation

Ru: https://github.com/russsiq/laravel-docs-8.x-ru/blo...

Comments

[Сергей delphinpro]

protected function create(array $data)
{
    Validator::make($data, [
        // Проверка всех других полей формы
        'name'     => '...',
        'email'    => '...',
        'password' => '...',
        // Проверка приглашения
        'invite_token' => [
            'required',
            function ($attribute, $value, $fail) {
                $token = Invaite::firstOrFail();
                if ($value == $token->invaite_token) {
                  $fail('Приглашение не существует');
                }
            },
        ]
    ])->validate();
    
    return User::create([
        'name'          => $data['name'],
        'email'         => $data['email'],
        'invaite_token' => $data['invaite_token'],
        'password'      => Hash::make($data['password']),
    ]);
}

[Николай Стешенко]

Ничего не получилось пробовал по разному. просто регистрирует даже если вожу просто строку.

/**
     * Get a validator for an incoming registration request.
     *
     * @param  array  $data
     * @return \Illuminate\Contracts\Validation\Validator
     */
    protected function validator(array $data)
    {
        return Validator::make($data, [
        // Проверка всех других полей формы
            'name' => ['required', 'string', 'max:255'],
            'email' => ['required', 'string', 'email', 'max:255', 'unique:users'],
            'password' => ['required', 'string', 'min:8', 'confirmed' ],
        // Проверка приглашения
        'invite_token' => [
            'required', 'string', 'invaite_token', 'max:32', 'unique', 
            function ($data, $fail) {
                $token = Invaite::firstOrFail();
                if ($this->invaite_token == $token->invaite_token) {
                  $fail('Приглашение не существует');
                }
            },
        ]
    ]);
	
	
    }

    /**
     * Create a new user instance after a valid registration.
     *
     * @param  array  $data invaite_token
     * @return \App\User
     */
    protected function create(array $data)
    {
        return User::create([
            'name' => $data['name'],
            'email' => $data['email'],
	    'invaite_token' => $data['invaite_token'],
            'password' => Hash::make($data['password']),
        ]);
    }

[Сергей delphinpro]

Николай Стешенко, вы в методе create() не вызываете валидатор

protected function create(array $data)
    {
        $this->validator($data)->validate();
        return User::create([

[Сергей delphinpro]

В списке правил 'invaite_token' - нет такого правила.
Для unique таблицу нужно указать (как в мыле сделали)
Но unique здесь не нужен, вы же не добавляете инвайт в таблицу инвайтов.
И выдергивание первого попавшегося инвайта из таблицы вызывает сомнения.
Тут скорее нужно проверить наличие инвайта в таблице

$token = Invite::where('invite_token', $value)->first();
if (!$token) {
  $fail('Приглашение не существует');
}

[Николай Стешенко]

Валидатор игнорит проверку. Не важно, что я туда пишу правильный инваит или просто строку регистрирует.

protected function validator(array $data)
    {
	 return Validator::make($data, [
        // Проверка всех других полей формы
            'name' => ['required', 'string', 'max:255'],
            'email' => ['required', 'string', 'email', 'max:255', 'unique:users'],
            'password' => ['required', 'string', 'min:8', 'confirmed'  ],
             
		// Проверка приглашения 
        'invite_token' => [
            'required', 'string', 'max:32',  
            function ($data, $fail) {
                $token = Invite::where('invite_token',  $data['invaite_token'])->first();
				if (!$token) {
				  $fail('Приглашение не существует');
				}
            },
        ]
    ]);
	
	
    }

    /**
     * Create a new user instance after a valid registration.
     *
     * @param  array  $data invaite_token
     * @return \App\User
     */
    protected function create(array $data)
    {
		
	$this->validator($data)->validate();
        return User::create([
            'name' => $data['name'],
            'email' => $data['email'],
			'invaite_token' => $data['invaite_token'],
            'password' => Hash::make($data['password']),
        ]);
    }