Почему не работает проверка на пустоту?

Question

[Павел]

делаю запрос в БД, чтоб не описывать весь ответ, он просто выдает число, если оно есть то его выводит
проблема в том что в случае неудачного поиска, когда нет подходящих объектов для вывода числа , нечего не выводит, а нужно вывести "нет записей" или "0"
делаю такую проверку

if ($data['name'] == $value9['name']): ?>
                                        <?php if ($value9['cnt'] == "") {
                                            echo "0";
                                        } else {
                                            echo $value9['cnt'];
                                        } ?>

но не выводит 0 , вообще нечего не выводит , тоже и с empty($value9['cnt']) и $value9['cnt'] < 0
в чем может быть проблема?

Comments

[Adamos]

Поставьте второй строчкой <?php print_r($value9); ?> и смотрите сами, в чем проблема.
Детский сад, штаны на лямках.

[Павел]

Adamos, и что мне это даст? Там огромный массив всего что из бд пришло

[Adamos]

Павел, например, есть ли в этом массиве вообще поле cnt. От нас-то вы какого ясновидения по вашим данным и вашим скриптам ожидаете? Приведенный кусок - вообще ни о чем.

[Павел]

Adamos, так если его нет должна сработать empty($value9['cnt']) ? А если isset($value9['cnt']) то вообще всегда условие срабатывает

[Adamos]

Павел, вам таки надо работать или поговорить? У вас есть ваши данные и ваш код. Вы выдали нерелевантный кусок кода и предлагаете почтенной публике угадать, что не так в остальном коде и данных. Удачи...

[ThunderCat]

Павел,

и что мне это даст? Там огромный массив всего что из бд пришло

Или нет, не мы, не вы этого утверждать не можете, по этому первое что нужно сделать в случае "там должно быть..." - посмотреть что там реально есть. Неужели это так сложно?

Answer 1

[Stimulate]

echo (((isset($value9['cnt'])) AND ((int)$value9['cnt'] > 0)) ? $value9['cnt'] : '0');

Comments

[Ивор Барханский]

Меня тернарная запись if постоянно пугает до чёртиков своей нечитаемостью. Если разработка это не основной профиль, а сторонний доп. инструмент, то возвращаясь к этому раз в пару месяцев напрочь забываешь логику построения строки, тупишь, гуглишь, разбиваешь на компоненты...

Может есть человекопонятный cheatsheet на все случаи, или какой универсальный случай, чтобы распечатать толсто и на стену повесить перед глазами?

[Stimulate]

Игорь, честно говоря, не понимаю, чем эта запись может пугать, тем более в масштабах проектов настоящего времени) если в кратце - запиши себе "if ? true : false"

[ThunderCat]

Игорь, че там думать, оно же читается как текст: условие выполнено? да : нет;

[Ивор Барханский]

ThunderCat, на простых обучающих примерах может и да. А прямо посреди кода, когда условий штук пять, а true и false сами по себе являются сложными однострочниками... А если вложенные условия... А если во вложенностях присутствуют свои блоки с двоеточиями/знаками вопроса...

Вот эта экономия на трёх переносах строк в классическом if...else мне непонятна.

[Stimulate]

Игорь, тут уж надо для себя понимать, когда стоит пользоваться короткой записью, а когда - нет

[ThunderCat]

Игорь, так оно в основном и есть простой однострочник, там где буквы экономят на хирожопой логике это уже из области злых буратин. Оно в основном вывести сточку по условию, получить нолик при емпти и значение переменной при существовании подходит, для него даже в 7 дополнительный синтаксис укороченный есть - $a ?? 'b'; , так как это коммон юзе кейс.

Answer 2

[FanatPHP]

Надо не мучитЬся, а логически мыслить.

нИчего не выводит, а нужно вывести "нет записей" или "0"

Если условие if ($value9['cnt'] == "") не выполняется, это значит что в $value9['cnt'] не пустая строка или приравненное к ней значение.
Чтобы узнать, какое там зрачение на самом деле, надо сделать var_dump()

Comments

[Павел]

там где есть что то , то это стринг цифры , а если нет то там совсем пусто

[FanatPHP]

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

Answer 3

[ThunderCat]

1) Научитесь использовать var_dump, прям вот "в любой непонятной ситуации - делай вардамп".

2) использовать синтаксис if(): ... endif - моветон, он кроме того что избыточен, еще и не все иде его нормально воспринимают, и визуализация нарушается. Короче, совет дня - не используйте. А у вас он еще и не закрывается.

3) что вам дает открытие / закрытие тегов в каждой строке? Или это чисто эстетическое наслаждение - смотреть на кровоточащие глаза коллег, читающих ваш код?

4) if ($value9['cnt'] == "") это не проверка на пустоту, хотя в большинстве случаев оно будет работать, но это однозначно говнокод, который будет сыпать в лог варнинги, для таких проверок есть empty();

5) if ($data['name'] == $value9['name']) не будет работать, если $value9['name'] пустой, соответственно ВЕСЬ код внутри этой проверки работать не будет.

Comments

[Антон]

2) Но когда идет помесь из пыхи и хтмл, отделять условия, циклы таким образом куда нагляднее, нежели чем скобками, которые визуально поймать там "где нить внизу" уже тяжелее

[FanatPHP]

поправка. для "таких проверок" служит if (!$value9['cnt'])
empty() служит для проверок if (isset($value9['cnt']) && !$value9['cnt'])

[ThunderCat]

Антон,

которые визуально поймать там "где нить внизу" уже тяжелее

ИДЕ хорошо понимает "скобочные" блоки и правильно их сдвигает, а так же подсвечивает начало/конец блока для скобок, но не делает этого для альтернативного синтаксиса. Выбор очевиден.

[ThunderCat]

FanatPHP, проверка if(!$value9['cnt']) вообще приемлема ТОЛЬКО в случае если значение булево и заранее известно что оно точно есть (и точно булево). В данном случае это нифига не ясно, по этому empty().

[FanatPHP]

Нет. empty() не имеет никакого отношения к приведению типов. а только к проверке переменной на существование.
Я понимаю логику за этим утверждением, но она не верна. empty() неявно приводит типы точно так же, как это делает if($var) или if ($var == ""). То есть здесь просто добавляется бессмысленный вызов.

[ThunderCat]

FanatPHP, разница есть, и в данном случае логика весьма очевидна - переменная может не существовать, и может иметь значение приводимое к булевому. В случае не существующей мы, как минимум, поимеем варнинг, если не используем empty.

[FanatPHP]

Совершенно верно. Но это совсем другой вопрос, который не имеет отношения к проверке на пустое значение, которое у автора

[ThunderCat]

FanatPHP, не понимаю о чем спор. У автора неверно проверяется значение, по факту оно проверяет "что лежит в переменной", вместо "если есть переменная и не пустая ли она". Так проверять хеново во всех смыслах. В том числе и if($var) будет давать варнинг .

[FanatPHP]

Ну ладно, возможно мы просто по-разному поняли задачу автора.
просто empty надо применять не вообще всегда, а только если ззаведомо известно, что переменная может не существовать.