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

Question

[Сергей]

Какой метод лучше и почему?

if ( !empty($arResult) && !isset($arResult) )
if ( $arResult != "" )

Answer 1

[FanatPHP]

Если выбирать из этих двух
То второй метод лучше. Он хотя бы иногда будет работать.
В отличие от первого,
Который просто безумный треш.

А вот почему - ты сам должен ответить. Прочитав документацию по использованным тобой функциям.
Там же - сюрприз! - ты найдешь и ответ на свой вопрос. Настоящий ответ, а не то жалкое блеянье, которое тебе напишет здесь очередной мимо проходивший школьник.

Comments

[Сергей]

Посмотрел в документации, empty включает в себя проверку $arResult != "", но с другой стороны там и 0 считается пустой переменной, действительно треш.

[FanatPHP]

Сергей: ты не понял. Треш - это твой первый пример. А проверка работает так, как положено.

[FanatPHP]

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

[Сергей]

Если переменная не пуста, то вернет true. В чем проблема? Много где видел так используют

[Александр Зеленин]

Сергей: if ( !empty($arResult) && !isset($arResult) )
если непустая и не существует

[FanatPHP]

Я не спрашивал, где это используют. Я спросил, ПРОБОВАЛ ли ты сам использовать это выражение. И да - после этого я хотел попросить тебя произнести это выражение словами, что уже сделал Александр Зеленин: как ты считаешь, есть ли смысл в такой проверке?

[Сергей]

FanatPHP: Признаю ошибся, if ( !empty($arResult) && isset($arResult) ) надо так. Откуда у вас столько ненависти к людям? или вы считаете своим долгом грубить и тыкать каждому кто спрашивает совета? Если вы не можете ответить на вопрос по существу, то зачем вообще отвечаете? От вас только и слышно "Шивили извилиной своей", "Смотри в документации". Вы говорили другому человеку "Тебе стоит взять взять моратиорий на ответы, годика на два. И сам не будешь позориться, и тостер.ру станет выглядеть чуть менее убого, чем сейчас.". Мое мнение это вам нужно взять мараторий, ведь именно из-за таких как вы можно считать сервис убогим. Люди приходят за ответом, а слышат лишь оскорбления в свой адрес.

[Сергей]

Александр Зеленин: Спасибо, действительно ошибся в написании

[FanatPHP]

Сергей: очередной анекдот из серии "мама он меня сукой обозвал" :) Тебя я пока ещё не скорблял. Я всего лишь задал тебе вопрос, в корректной форме. То, что ты не смог на него ответить - это твоя проблема, а не моя ненависть :) В любом случае, то что ты исправил совсем бессмысленный код - уже хорошо. Но проблемы остались . Этот новый код все равно бессмысленный. Сможешь догадаться - почему? :)

[Сергей]

FanatPHP: А вы вопрос читали, или тупо здесь потролить собираетесь? Я бы не задавал его, зная ответ. Если не один из методов не подходит, напишите почему, дайте свое профессиональное мнение об этом. Хотя у вас врятли это получиться, вы же не делаете бесплатно ничего

[FanatPHP]

Сергей: тебя даже троллить не надо, у тебя само подгорает. Свободен

[Сергей]

FanatPHP: Ладно, извините за беспокойство

Answer 2

[TekVanDo]

все зависит от того что вы хотите проверить.
empty - делает приведение типов, т.е. если внутри arResult - пустой массив или например 0, то !empty($arResult) == false
$arResult != "" работает только для строк.

Answer 3

[Mikhail Osher]

Когда надо проверить, что переменная инициализирована и отлична от null - isset.
Когда надо проверить, что переменная не пустая - empty.
empty принимает только один аргумент, в отличии от isset.

<?php
if (isset($a)) {
    /* code */ 
}
if (isset($a, $b)) {
    /* code */
}
if (isset($a['a']['b'], $b, $c[0][1])) {
    /* code */
}

Answer 4

[Тимофей]

Судя по $arResult ты имеешь дело с битриксом. Если это так, то $arResult за тебя уже инициализирован. В любом случае, проверку в пыхе стоит делать более осмысленно, а именно исходить из того, какой тип значения ты хочешь получить. Если у тебя, скажем, строка, то можно обойтись простейшим методом, которым я чаще всего пользуюсь сам:

if (isset($arResult['somevar']) && $arResult['somevar']) { ... }

Здесь ты для начала проверяешь, есть ли вообще такая переменная (или ключ в массиве), а потом смотришь, приводится ли она к false (а, как известно, пустая строка приводится, но, например, 0 тоже приводится, что нужно понимать). Потом уже в коде ты скорее всего где-то неявно приведешь переменную к строке, то есть, например, число 100500 будет приведено к строке '100500'.
Для более строгой проверки можно использовать функции, подобные is_int, is_array и т.д., а так же instanceof. Еще, если не знаешь, то почитай про операторы == и === (!=, !==) - тоже по этой теме.
Вообще, сравнение значений в условиях нестрогой типизации - достаточно обширная тема.

Comments

[Сергей]

Про битрикс вы угадали, но просто скопировал переменную, что ближе была (лень было писать:). Но вопрос не именно к нему. Понял, что нужно исходить из вида переменой и что в ней хранится. Спасибо за ответ.

[FanatPHP]

рекомендую открыть для себя empty()

[Тимофей]

FanatPHP: Можно и empty воспользоваться. Пожалуй, в приведенном мной коде empty сделает ровно то же, что и мое условие. Но когда нужна проверка типа, лучше isset(...) && is_int(...), например.

[FanatPHP]

Так я и говорюб про проверку выше. А про проверку типа isset(...) && is_int(...) я ни разу в жизни не слвшал. Да и ты только сейчас ее из пальца высосал

[Тимофей]

FanatPHP: Ну а какой еще способ для проверки типа ты знаешь?

[FanatPHP]

Мальчик мой. Самооценку поднимают не вопросами, а ответами. Напиши сначала осмысленный ответ, я прочту и решу, заслужил ли ты право задавать мне вопросы.

[Тимофей]

FanatPHP: А ты смешной)

[Сергей]

Тимофей: не обращай внимание, это местный клоун, который позиционирует себя как эксперт.