Почему не выводится текст echo?

Question

[sandul23]

Есть очень интересная проблема,решение к которой я подобрать, к сожалению, не смог(А,нет,есть одно,снести всё к чертям)

Есть вот такой код:

$UL = $user['level'] + 1;
$XP = $user['exp']; 
$Lvls = mysql_query('SELECT * FROM `levels` WHERE `Number` ='.$UL.';');
$Lvl = mysql_fetch_array($Lvls);
$exp_in = $Lvl['Exp'];
if($exp_in <= $XP){ 
echo('<div class="block center">Новый уровень!<br>Награда:'.$user['level'].' <img src="/images/coin.png"> </div>');
mysql_query("update `users` set `exp` = 0, `level` = `level` + 1, `coin` = `coin` + ".$user['level']." where `id` = ".$user['id']."");
}

Коротко о работе: при достижении определенной отметки опыта(вытаскивается из БД), у пользователя он обнуляется,прибавляется один к уровням,добавляется валюта. И самое главное:должна выводиться надпись об уровне. Запрос происходит,всё работает как надо,однако надпись не выводиться. Если поместить за пределы блока if,всё прекрасно работает. Если менять первое в условии,ничего не происходит. Однако если поменять значение второй переменной/заменить её например на простое число,надпись выводится. Также если ставить напрямую $user[exp],то надпись также не выводится. Я вообще не понимаю,как это может влиять на работу вывода.

Comments

[ThunderCat]

Если поместить за пределы блока if,всё прекрасно работает.

То есть какой отсюда вывод?

Попробуйте пользоваться функциями отладки, например var_dump(), и жить сразу станет проще.

[sandul23]

ThunderCat, вывод ошибок включен,0. Тем более работа с БД выполняется успешно,хотя функция находится в том же блоке.

[snake2]

Значит не выполняется условие

[Александр]

Перед if
var_dump($exp_in, $XP);
и пришлите результат сюда

[sandul23]

Александр, array(2) { [0]=> string(3) "150" [1]=> string(2) "15" }

[sandul23]

snake2, запрос sql выполняется,значения при условии меняются

[Rsa97]

sandul23, Вы показали только одну переменную, в которой массив. Что во второй переменной?

[sandul23]

Rsa97, какую ? Дайте название пожалуйста.

[Rsa97]

sandul23, Вам виднее, какую. В var_dump, который вас попросили показать, две переменные, а вы показали одну.

[sandul23]

Rsa97, как я понимаю,там две переменные в массиве. В прочем,данные те,которые должны быть.

[Rsa97]

sandul23, И какая из этих переменных какая? Если $exp_in = "150", a $XP = "15", то условие ($exp_in <= $XP) будет выглядесть как ("150" <= "15") и не будет выполняться.
И таки да, вы сравниваете строки, значит можете получить не тот результат, на который рассчитываете.

10 > 2 === true
"10" > "2" === false

[sandul23]

Rsa97, да,верно. Однако всё-таки есть одна большая странность: когда $XP становится больше, чем $exp_in, всё же условие срабатывает и запрос SQL проходит,изменяя значения.

[sandul23]

Есть новость: начинает работать при срабатывании условия,если поставить в этом if exit(); , но из-за этого исполнение обрывается и выводится только оно. В чём тогда дело ?

[Rsa97]

sandul23, А что в этом странного? Когда условие становится истинным, выполняется код, вложенный в ветку if.

[sandul23]

Rsa97, странно то,что при работе условия if, работа с БД происходит,а вывод текста без использования exit() нет

[sandul23]

ну и плюс то,что при том,что переменные имеют значения строк,судя по var_dump, сравнение работает.

[Rsa97]

sandul23, По сравнению строк - перепутал с JS. В PHP работает магия.

В случае, если оба операнда являются строками, содержащими числа или один операнд является числом, а другой - строкой, содержащей числа, то сравнение выполняется численно. Эти правила также применяются к оператору switch. Преобразование типа не происходит при сравнении === или !==, поскольку это включает сравнение типа, а также значения.

[sandul23]

Rsa97, PHP одна сплошная магия. Порою по логике работы похлеще,чем JS

[ThunderCat]

Rsa97,

В PHP работает магия.

Э?

let a = "5";
let b = 6;
if(a<b){console.log('same shit!')}

[Rsa97]

ThunderCat,
PHP: var_dump("5" > "20"); // bool(false)
JS: console.log("5" > "20"); // true

[ThunderCat]

Rsa97, спрашивается, и где тут еще магия? )) Хотя некоторая и разница в приведении типов есть, но даже хз где логика "логичнее" ))

[Rsa97]

ThunderCat, PHP магичнее. Если JS только на типы смотрит, то PHP ещё и внутрь строк лезет, чтобы определить, что эта строка содержит именно число.

Answer 1

[ThunderCat]

у вас там много конечно что не по уму, я так перечислю, а вы уже смотрите что делать (в идеале все исправить, конечно же).
1) mysql_query: Данный модуль устарел, начиная с версии PHP 5.5.0, и удалён в PHP 7.0.0. Используйте нормальный PDO или mysqli драйвер бд.

2) Используйте подготовленные выражения в запросах, а не дырявое решето вместо них...

3) Используйте нормальный стиль форматирования и именования переменных, читайте PSR для ознакомления с принятым форматированием кода. Все эти $XP, $exp_in, $Lvls... Кровь из глаз...

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

5) echo не требует скобок, хотя они и не влияют на его работу.

6) Если что-то работает не так, как вы ожидаете - 99% где то в переменных не то, что вы думаете. По этому var_dump() ваш лучший друг на ближайшие 2-3 года.

var_dump([$exp_in, $XP]);
if($exp_in <= $XP){ 
var_dump(1);
...
var_dump(2);
...

Тем более работа с БД выполняется успешно

Как вы это проверяете?

Comments

[Дмитрий]

я бы еще добавил к этому, называть переменные человеческим языком на английском, чтоб хотя бы было понятно, что к чему, ну и первым делом надо дпмпить условия и данные в условиях, молодеж не очень визуально понимает знаки больше и меньше

[sandul23]

Постараюсь учесть,однако вы игнорируете один фактор: если заместо $XP поставить просто $user['exp'], то всё равно не работает. Проверяю таким образом,что на страницу выводится количество опыта,уровень и т.д и они меняются,как нужно + в PMA видно,что значения меняются. Что-то не так конкретно с echo, или в буферизации дело, или в чём,я не знаю. Вы даёте действительно полезные советы,однако игнорируете некоторые факторы и основную проблему. Переменные не при чем.

Однако если поменять значение второй переменной/заменить её например на простое число,надпись выводится. Также если ставить напрямую $user[exp],то надпись также не выводится.

var_dump ничего не показывает. К слову,работает всё на php 5.4

[ThunderCat]

var_dump ничего не показывает.

Значит таких значений нет.

Что-то не так конкретно с echo, или в буферизации дело,

Возможно, это легко проверить, достаточно добавить exit(); после вывода. Будет понятно дело в буфере или в чем то другом. Опять же - готовить данные, потом работать с выводом, тогда не будет проблем с буферизацией. Когда каша в коде - невозможно работать с ошибками. Но сомневаюсь что дело в этом.

Однако если поменять значение второй переменной/заменить её например на простое число, надпись выводится.

Значит переменные не содержат то что должны, а буферизация отсутствует.

Также если ставить напрямую $user[exp]...

ну вот и проверяйте var_dump($user), смотрите какие значения и ключи там есть.

[sandul23]

ThunderCat, exit() решает проблему. Однако потом после этого ничего не выводится,но как я понимаю,это так и должно быть. В чём тогда дело,буфер ?

[ThunderCat]

sandul23, у вас же тут только кусок кода, откуда я знаю. Скорее всего буфер, хотя возможно ниже выводится хтмл который "не совместим" с этим выводом, например ниже выводится боди, и ваш див конечно же до него просто не виден.

Еще раз: Когда код хреновый, его и дебажить сложно. Вынеси ВСЕ вычисления и запросы в базу в начало файла, дай переменным осмысленные длинные названия, и ПОСЛЕ ТОГО КАК ВСЕ ДАННЫЕ ГОТОВЫ, выводи хтмл, и там вставляй свои переменные через <?=$variable?>, тогда и отладка и вывод будут на порядок проще.