Co jest złego w tym warunku? Opcje

[Bilbosz]

Witam. Potrzebuję porównać 2 dane z tabeli w warunku. Problem polega na tym, że warunek nie działa jak trzeba.

[PHP] pobierz, plaintext 
$exp = $m -> query('SELECT exp FROM gracze WHERE id = '.$_SESSION['id'].' LIMIT 1');
$exp_max = $m -> query('SELECT exp_max FROM gracze WHERE id = '.$_SESSION['id'].' LIMIT 1');
 
if(mysqli_num_rows($exp) > mysqli_num_rows($exp_max)){
    $m -> query('UPDATE gracze SET exp = 0 WHERE id = '.$_SESSION['id'].' LIMIT 1');
    $m -> query('UPDATE gracze SET poziom = poziom + 1 WHERE id = '.$_SESSION['id'].' LIMIT 1');
}
[PHP] pobierz, plaintext 

[trueblue]

Obydwa zapytania zwracają taką samą ilość rekordów (0 lub 1) i porównujesz ilość, a nie wartości.
Pobierz wartości (jednym zapytaniem) i porównaj. Możesz też zrobić porównanie bezpośrednio w zapytaniu.

[Bilbosz]

Obydwa zapytania zwracają taką samą ilość rekordów (0 lub 1) i porównujesz ilość, a nie wartości.
Pobierz wartości (jednym zapytaniem) i porównaj. Możesz też zrobić porównanie bezpośrednio w zapytaniu.

Jestem w tym zielony. Jakoś tak to powinno wyglądać

[PHP] pobierz, plaintext 
//kolejny poziom
$z = $m -> query('SELECT exp, exp_max FROM gracze WHERE id = '.$_SESSION['id'].' LIMIT 1');
 
while($w = $z -> fetch_assoc()){
if($w['exp'] >= $w['$exp_max']){
    $m -> query('UPDATE gracze SET exp = 0 WHERE id = '.$_SESSION['id'].' LIMIT 1');
    $m -> query('UPDATE gracze SET poziom = poziom + 1 WHERE id = '.$_SESSION['id'].' LIMIT 1');
}
}
[PHP] pobierz, plaintext 

[trueblue]

1. while jest niepotrzebne, bo pobierasz 1 rekord.
2. Wystarczy jeden UPDATE.

[borabora]

z tego co widzę to przy każdym graczu masz tę samą wartosć w kolumnie exp_max. Chyba, że w innym miejscu to updejtujesz... pytanie: jaki jest w tym sens? gracz A może potrzebować innej ilości expa to przejścia z poziomu 1 na poziom 2 niż gracz B?

[Bilbosz]

Akurat pobierałem 2 rekordy ale już działa. Tutaj $w['$exp_max'] przez przypadek dodałem znak dolara
Dzięki!

[borabora]

[SQL] pobierz, plaintext 
UPDATE gracze WHERE id='.$_SESSION['id'].' AND exp>exp_max SET poziom=poziom+1,exp=0
[SQL] pobierz, plaintext 

zamiast 3 zapytań możesz użyc jednego tak jak już było wspomniane wyżej.
To, że obydwa sposoby będą działać, nie zmienia faktu, że rozwiązanie jest dziwaczne.