Почему не выполняется условие WHERE?

Question

[Denis]

if ($promo == 'NEWUSER'){
	if($bd->Query("SELECT COUNT(*) FROM users_nykfageubf WHERE p = '$p' AND promo_new=1")){
		$bd->Query("UPDATE users_nykfageubf SET promo_new = 0 WHERE p = '$p'");
		$bd->Query("UPDATE users_nykfageubf SET balance=balance+10 WHERE p = '$p");
	} else {
		echo '<div class="alert-msg"><a href="#" class="close-alert"><i class="fa fa-times"></i></a><p>Вы уже использовали промокод.</p></div>';
       }
}

Присутствует такой фрагмент кода. Реализуется использование промокода. По замыслу, у пользователя $p проверяется, использовал ли он промокод promo_new (1 - нет, 0 - да). Если нет, то баланс увеличивается на 10 и promo_new = 0; Проблема лишь в том, что в не зависимости от того, какое значение стоит в promo_new, пользователь неограниченное кол-во раз может использовать промокод и получить +10 на баланс. Не понимаю почему, ведь присутствует WHERE и всё чётко указано.

Comments

[Дмитрий]

А за один апдейт нельзя сразу и баланс поменять и метку обновить?

[Denis]

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

[Дмитрий]

Ваш query возвращает число из коунт запросов?

[Denis]

Дмитрий, да.

[Дмитрий]

Denis, покажите, мне кажется там просто bool и условие всегда верно

Answer 1

[Slava Rozhnev]

Потому что

$bd->Query("SELECT COUNT(*) FROM users_nykfageubf WHERE p = '$p' AND promo_new=1"))

всегда возвращает true - запрос выполнен успешно, а не количество найденных строк

Кстати два запроса следует объединить в один:

$bd->Query("UPDATE users_nykfageubf SET balance=balance+10, promo_new = 0 WHERE p = '$p' AND promo_new=1");

а в качестве провереи использовать более быстрый запрос:

$stmt = $bd->Query("SELECT 1 FROM users_nykfageubf WHERE p = '$p' AND promo_new=1 LIMIT 1");
if ($stmt->rowCount() > 0) {
    ...........
}

Comments

[FanatPHP]

и дать сообразительным пятиклассникам способ неограниченно пополнять себе баланс

[Slava Rozhnev]

FanatPHP, сообразительность нужно поощрать, глупость наказывать

[Denis]

Спасибо за ответ.
$stmt->rowCount()
Не сработал, но я заменил на
mysqli_stmt_num_rows($stmt)
И всё получилось.

[Slava Rozhnev]

Denis, Отлично, занчит у Вас за фасадом mysqli а, не PDO. На Вашем месте я бы mysqli_stmt_num_rows тоже спрятал за фасадом что бы было так:

if ($bd->rowCount($query) > 0) {

}

Answer 2

[Rsa97]

По хорошему, все три запроса надо объединить в один, переписать с плейсхолдерами и проверять mysqli::$affected_rows или PDOStatement::rowCount после запроса. Так можно избежать состояния гонки.

Comments

[Дмитрий]

Меня смущает что это $bd, а не $db. И там под капотом может быть обычный mysql

[BorLaze]

Дмитрий, $db - от database, a $bd - от bazadannyh :-)))