Операторы сравнения на php что делать?

Question

[Habitat Horsewithyou]

Потерялся вовсе если оставлять только одно условие то работает все хорошо
с двумя ни в какую

".($_SESSION['login']!==$row['id_name'] or empty($_SESSION['login'])?
"<div>то что нужно показать</div>"
:null)."

$row['id_name'] это поле выгружено из базы данных

Comments

[AlikDex]

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

[Habitat Horsewithyou]

AlikDex: думаю так понятнее?

[AlikDex]

Habitat Horsewithyou: нет конечно. Это попрежнему нечитаемое дерьмо. Да и не мешало бы всю строчку, а не этот кусок постить. Как пример:

<?php if ($_SESSION['login']!==$row['id_name'] or empty($_SESSION['login'])): ?>
    <div>то что нужно показать</div>
<?php endif; ?>

Само условия я бы переписал чутка иначе:

if (empty($_SESSION['login']) || $_SESSION['login'] !== $row['id_name'])

Т.е. сначала проверить, пустая ли переменная в сессии(или существует ли такая вообще), и только потом сравнивать значение.

[Habitat Horsewithyou]

AlikDex: то что я написал почему то я думал это есть сокращение if

[AlikDex]

Habitat Horsewithyou: Все верно, сокращение. Но оно порой не очень уместно. Вот в вашем случае это совсем неуместно, т.к. сильно мешает читабельности кода. Уважаемым местным донам не пристало разбираться в этом дерьме, чтобы ответить на вопрос. О чем, как я полагаю, свидетельствует отсутствие ответов.

[Habitat Horsewithyou]

AlikDex: услышал, благодарю за ответ пошел все таки попробую что нибудь сделать

Answer 1

[Habitat Horsewithyou]

if (empty($_SESSION['login']) || $_SESSION['login'] == $row['id_name'])