1

Мои настройки php и настройки php на хостинге не ругаются если использовать записьif ($_POST['specialist_id']) {...}.

Хорошему программисту на php лучше использовать if (isset($_POST['specialist_id'])) {...}или не возбраняется писать просто if ($_POST['specialist_id']) {...}?

2
  • Если точно уверены, что такая переменная есть, то можно и без isset(), в противном случае лучше перестраховаться и использовать, что бы ошибки не сыпались. 10 сен 2017 в 23:12
  • 2
    рекомендую всегда использовать isset 10 сен 2017 в 23:13

5 ответов 5

4

isset проверяет на существование переменную или элемент массива. А сам if проверяет уже значение. Это совершенно разные конструкции. Рекомендуеться проверять и на наличие переменной и само значение.

0
2

Рекомендую всегда использовать isset() в данном контексте.

7
  • 1
    так, и что будет если не использовать isset в данном случае? 11 сен 2017 в 6:01
  • @АлексейШиманский по правильному будет варнинг "такого ключа нет". Но это ж пхп, а значит все зависит от версии пхп и настройки.
    – vitidev
    11 сен 2017 в 6:14
  • @vitidev доказательства можно? 11 сен 2017 в 6:15
  • @АлексейШиманский тут sandbox.onlinephpfunctions.com вбейте код $arr=array('foo'=>'bar');print($arr['foo2']); и выполните.
    – vitidev
    11 сен 2017 в 6:20
  • 3
    @АлексейШиманский $_POST точно такой же массив, как и другие.
    – vitidev
    11 сен 2017 в 6:22
2
+-------------------+-------------+-------------------+
|$var               | isset($var) | if($var)          |
+-------------------+-------------+-------------------+
|не задана          | false       | false + E_NOTICE  |
+-------------------+-------------+-------------------+
|приводится к false | true        | false             |
|[],"",false,0,"0"  |             |                   |
+-------------------+-------------+-------------------+
|приводится к true  | true        | true              |
+-------------------+-------------+-------------------+
1

Для полного эквивалента вашего условия нужно проверять не только что переменная задана, но и что значение у нее есть:

if (!empty($_POST['specialist_id'])) {
    // эквивалент isset($_POST['specialist_id']) && $_POST['specialist_id']
}

Другим вариантом может быть отключить ругань в вашем варианте:

if (@$_POST['specialist_id'])) {

}

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

2
  • забавно, но доки говорят что empty() фактически является точным эквивалентом конструкции !isset($var) || $var == false ............ Вопрос: зачем проверять, что переменная задана и еще вдобавок empty, который делает тоже самое? 11 сен 2017 в 6:10
  • 1
    Вы отрицание перед empty() не заметили наверное.
    – sanmai
    11 сен 2017 в 6:11
-2

Если в $_POST не будет ['key'], то есть $_POST['key'] не будет установлен, будет выведен NOTICE, а если сделать isset($_POST['key']) то даже, если этот ключ не будет определен, NOTICE не будет.
В отличии от empty конструкция isset поддерживает оператор and so on, по этому мы можем использовать такой синтаксис: isset($_POST['key'], $_POST['key2']), чего нельзя делать с empty.

Проверку на то, что переменная или ключ массива существует, нужно делать только в том случае, если вы не уверены, что этой переменной или ключа массива может не быть.
Это относится к тем областям. где переменная или ключ массима создается динамически или может быть удален в ходе.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.