Почему не работает OR в условии?

Question

[del993788]

Почему может не работать OR? Если оставить условие $query != 404 одним, то будет всё хорошо, но если у меня появляется OR, то ничего не работает

function is_eampty(&$var)
{
    return !($var || (is_scalar($var) && strlen($var)));
}

$query = $_GET['w'];


if (is_eampty($query) or $query != 404 or $query != 403){
$query = "404";
include 'error.tpl';
} else {
$query = $_GET['w'];
include 'error.tpl';
}

Comments

[Дмитрий]

Есть приоритет операций, or и || имеют разный вес приоритета, попробуйте написать

if(is_empty($var) || !in_array($var, [403, 404], true)) {}

[FanatPHP]

для каких входных данных не работает?
что ты вообще хочешь получить от этого кода?

[FanatPHP]

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

[Дмитрий]

FanatPHP, ну как минимум сократить проверку с трех до двух

[del993788]

Дмитрий, FanatPHP,
если query 403, то не выходит 403

[FanatPHP]

а откуда здесь вообще должно выйти 403?
Ты наверное имел в виду 404?
то есть тебе надо не != а ==?
"Если запрос пустой, или равен 404, или равен 403, то вернуть 404" - это тебе нужно?
Если нет, то словами сформулируй, так как у меня написано

[del993788]

FanatPHP,
если query не пустой или "не 404" или "не 403", то query становится 404,
иначе query остаётся GET

[FanatPHP]

Это утверждение не имеет смысла. "Не пустой" уже включает в себя любое множество, в том числе и "не 403".
Ты не пересказывай мне словами свое кривое условие, ты скажи что по условиям задачи надо сделать.

[del993788]

FanatPHP, я в пересказе ошибся. "если пустой"
is_eampty($query) - проверяет на пустоту.

У меня есть список кодов: 404, 403, 402, 401. Работать я буду только с ними. Мне нужно сделать так, чтобы в GET ничего своего нельзя было подсунуть. Если подставляется что-то иное - возвращается 404 код.

Answer 1

[FanatPHP]

Тебе нужен оператор AND, а не OR. Не работает у тебя сейчас потому, что как правильно написали Артур и AUser0 , условие $query != 404 or $query != 403 будет выполняться вообще всегда, поскольку если $query равно 404 то тогда не будет равно 403, и наоборот. А тебе нужно чтобы правильным был только один из вариантов. Это делает AND.

Кроме того, функция "is_eampty" в твоем примере вообще не нужна. Если переменная $query определена до условия, то тебе нужно только
if ($query != 404 AND $query != 403){
Если я правильно понял этот ужас, который написан у тебя в вопросе, то тебе надо

$query = $_GET['w'] ?? "";
if ($query != 404 AND $query != 403){
    $query = "404";
}
include 'error.tpl';

Answer 2

[Артур]

Это бессмысленное условие, оно всегда будет true

Answer 3

[AUser0]

С двумя этими OR у вас всегда будет выполняться код $query = "404";.
Потому что сработает или одно неравенство (когда $query==403), или другое неравенство (когда $query==404), или оба неравенства (когда $query==401).

P.S. Только непонятно, а что именно не работает-то? Код без ошибок, рабочий, в чём проблема-то?

Comments

[del993788]

если query 403, то не выходит 403

[AUser0]

Ну смотрите, если $query==403, то получается такой код:
if (is_eampty(403) or 403 != 404 or 403 != 403){
Вы ведь не будете спорить, что 403 не равно 404, да? Вот среднее условие и срабатывает.