Как правильно проверять переменную в php?

Question

[therealvetalhidden]

В каких случаях и вообще стоит ли проверять переменную в таком виде?

if ($var) {}
if (!$var) {}

Ведь если переменная не инициализирована то возникает ошибка

Answer 1

[Сергей]

if ($var) {}
if (!$var) {}

Это если true или если false.
Если в $var будет ноль - первое условие уже не сработает. Поэтому так лучше не проверять.
Проверяйте так:

if ( isset($var) && $var == 'нужное значение') {
     //что-то делаем
}

Comments

[therealvetalhidden]

Сергей Илларионов: спасибо) а если точно уверен что переменная инициализирована и надо проверить true false или пустой не пустой массив итд то if ($var) {} такой вариант нормальный?

[Сергей]

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

[FanatPHP]

therealvetalhidden: этот ответ - полная ерунда, забудь его

[D']

therealvetalhidden: забей на каменты фаната, он тут местный клоун. Сергей тебе все правильно подсказал. Так же лучше использоваль специализированные функции (is_array, empty, is_null, etc..) для проверки конкретного результата.

[FanatPHP]

Denormalization: подгорает? Это хорошо :)

[therealvetalhidden]

Сергей Илларионов: спасибо)

[therealvetalhidden]

Denormalization: спасибо) то есть лучше всегда в коде использовать так
if (isset($var)) чем if ($var), if ($var) использовать только true false ?

[Сергей]

therealvetalhidden: Зависит от ситуации, все способы проверки хороши. Проверять еще по регулярке можно if( preg_match() ) Просто проверка if ($var) может вернуть не то, что Вы ожидаете. В $var к примеру у Вас цифра 0, если вы хотите узнать определена ли переменная, проверка if($var) вернет false, а там же ноль. Для этого надо смотреть таблицы приведения типов.

[D']

therealvetalhidden: я предпочитаю использовать специализированные функции, для проверки конкретного значения. Иначе "вдруг" может быть косяк из-за неверных данных.

Answer 2

[FanatPHP]

Аналогом
if ($var)
для случая, когда переменная может быть не инициализирована, является
if (!empty($var))

Тем не менее, следует учесть замечание OnYourLips, и делать так, чтобы переменные, источником которых является твой скрипт, всегда были определены.

Update.
Мда. Полез поискать тебе ответ для случая, когда 0 считается не пустым значением.

Если бы тогда пассажир по фамилии KorsaR-ZN написал тебе не феерически бредовую отсебятину,

empty - проверят именно на пустоту переменной, т.е возвращает true если переменная является пустой и false в противоположном случаи.

а объяснил, что
!empty($var)
всего-навсего является короткой записью
if (isset($var) && $var))
то сейчас у тебя вопроса не возникло бы.

Это к вопросу о том, почему давать плохие ответы плохо (для автора) но хорошо (для посещаемости). И объясняет, почему администрация специально разводит на сайте лохов.

Comments

[KorsaR-ZN]

Зачем приводить обрывочную цитату, я указал, что является пустотой в условиях для PHP.

Раз такой умный, что ж в рамках своего ответа в контексте того вопроса, не исправил ситуацию?!
А начал заливать, какой ты крутой, а все кругом ничтожны.

[FanatPHP]

KorsaR-ZN: обтекай молча.

[KorsaR-ZN]

FanatPHP: Давай еще... Сделай это. Тебя же это заводит!

[KorsaR-ZN]

Жаль ума не много, на нормальные комментария.

[therealvetalhidden]

FanatPHP: спасибо) то есть если переменные мои(источником которых является мой скрипт) они должны быть всегда инициализированы и тогда их нужно проверять if ($var) а не if (isset($var)) или if (!empty($var)) а если переменные не мои то проверять нужно так if (isset($var)) или if (!empty($var)) ?

[FanatPHP]

я пожалуй воздержусь от написания ответов тебе, пока не пойму, тролль ты, или просто дурак.

[therealvetalhidden]

FanatPHP: только начинаю изучать языки прогр, ты таким никогда не был, сразу родился гением?

[FanatPHP]

therealvetalhidden: ладно ок. меня просто бесит, что тебе эти два неуча по ушам ездят, не могут ответить на прямой вопрос, и начинают юлить "все зависит от ситуации!" Ну ясно же, что если так пишет, то сам ответа не знает. И начинает лепить всякий бред про is_array(). Ну ты читай хоть их ответы и критически оценивай. Если ты хочешь узнать, пустой массив или в нем есть элементы, if (is_array($array)) тебе НЕ ПОМОЖЕТ. Тебе нужна твоя изначальная проверка if ($array). Неужели так сложно сделать такой разбор, как я сейчас сделал? Тебе этот диванный теоретик про is_array, а ты ему - вот такую выкладку, "мне это не подходит!". А ты с одинаковым энтузиазмом впитываешь как правильные ответы, так и всякий бред. НУ ХОТЬ НЕМНОГО ОСМЫСЛИВАТЬ полученную информацию ты можешь? Понять ПРИНЦИП? или ты просто хочешь написать себе шпаргалок на все случаи жизни и тупо ими пользоваться? Это в программировании не раоботает. Это тебе надо в менеджеры по продажам - у них есть должностная инструкция с ответами на все вопросы. А в программировании надо немного думать. А ты рот открыл и повторяешь вслед за ними бред, "то есть лучше всегда в коде использовать так if (isset($var)) чем if ($var), "! Неужели ты до сих пор не понял, что это абсолютно РАЗНЫЕ проверки, которые не могут использоваться одна вместо другой? Что они не "лучше" одна другой, а просто разные?

[FanatPHP]

if (isset($var)) - это есть ли у тебя кошелек в кармане. а if ($var) - лежат ли в нем деньги. если ты хочешь проверить, взял ли с собой кошелек - то иссет. если хочешь проверить, есть ли в кошельке деньги - иф вар. Если хочешь проверить, есть ли деньги, но не уверен, что кошелек собой - иссет плюс иф вар - то есть емпти. ЧТО такого сложного в этой тройке? С какой стати ты слушаешь всякий посторонний бред про is_array, preg_match() и идиотское isset($var) && $var == 'нужное значение'. Неужели ты сам не видишь, что вот это, последнее не имеет отношения к твоему вопросу? Ты же ведь спрашивал про проверку на пустоту, а не конкретное значение? Неужели не можешь хоть немного подумать и отличить тупой ответ от нормального?

[FanatPHP]

Да, все правильно ты написал. когда заведомо иницализированы, то if ($var). И здесь как раз ДЛЯ ТЕВОЕЙ ПОЛЬЗЫ и сработает нотис "undefined variable" - РНР тебе сразу подскажет, если ты опечатался, выдав ошибку. Именно поэтому иссетами злоуптреблять не стоит - всегда надо дать возможность РНР просигналить тебе об обращении к несуществующей переменной. Пример: ты выводишь пользователю форму, он ее заполняет, и отправляет назад. Если я правильно помню, браузер в любом случае отправляет на сервер все типы полей, кроме чекбоксов. То есть, здесь каждое текстовое поле проверять по isset не нужно. Наоборот! Если какого-то из полей нет, и ты видишь нотис, это сигнал об ошибке! Ты либо забыл вписать это поле в форму, или опечатался, или тебе кто-то пытается послать подделанную форму. Во всех трех случаях обрабатывать форму не надо.

[FanatPHP]

впрочем я наверное зря все это пишу