Undefined variable при проверке переменной?

Question

[jazzus]

Код

if (!(Hash::check($request->current_password, $user->password))) {
                  $errors['old_password_error'] = 'Старый пароль введен неправильно';
              }

              if(strcmp($request->current_password, $request->new_password) == 0){
                  $errors['passwords_are_same'] = 'Новый пароль должен отличаться от текущего';
              }

              if ($errors) {
                return response()->json([
                  'errors'=>[
                    'password_change'=>$errors
                    ]
                  ]);
              }else{
                $user['password'] = Hash::make($request->new_password);
                $user->save();
              }

Получаю
Undefined variable: errors
При этом я через if ($errors) else как раз и проверяю существование переменной. Понятно, что можно другими вариантами, интересует почему в данном примере ошибка?

Answer 1

[vism]

if ($errors) не проверяет существование переменной

Answer 2

[Кирилл Несмеянов]

При этом я через if ($errors) else как раз и проверяю существование переменной.

if - это не проверка на существование. if - это проверка на то, что существующая переменная соответствует boolean true. Почитайте, пожалуйста документацию по основам языка: php.net/manual/ru/control-structures.if.php

Comments

[jazzus]

я думал если переменная не существует это и есть false. С этой мыслью и проверял переменные.

Нагуглил такие варианты проверок:
Количеством

$errors = [];
if (count($errors) > 0)

isset
If (isset($errors))

Оба работают. Какой способ правильно использовать?

[Кирилл Несмеянов]

jazzus, зависит от того что требуется. Если в указанном примере, то ни тот, ни другой, т.к. для отображения глобальных ошибок (к в данном случае) достаточно выкинуть любое HTTP исключение, а в случае форм - они (ошибки) автоматом создаются на уровне валидаторов.

[jazzus]

Кирилл Несмеянов, со слов сложно конечно сразу вникнуть.. получается вот это

if (!(Hash::check($request->current_password, $user->password))) {
                  $errors['old_password_error'] = 'Старый пароль введен неправильно';
              }

              if(strcmp($request->current_password, $request->new_password) == 0){
                  $errors['passwords_are_same'] = 'Новый пароль должен отличаться от текущего';
              }

лучше перенести в реквест файл и проверять на стадии валидации?