Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Co jest złego w tym warunku?
Forum PHP.pl > Forum > Bazy danych
Bilbosz
Witam. Potrzebuję porównać 2 dane z tabeli w warunku. Problem polega na tym, że warunek nie działa jak trzeba.

  1. $exp = $m -> query('SELECT exp FROM gracze WHERE id = '.$_SESSION['id'].' LIMIT 1');
  2. $exp_max = $m -> query('SELECT exp_max FROM gracze WHERE id = '.$_SESSION['id'].' LIMIT 1');
  3.  
  4. if(mysqli_num_rows($exp) > mysqli_num_rows($exp_max)){
  5. $m -> query('UPDATE gracze SET exp = 0 WHERE id = '.$_SESSION['id'].' LIMIT 1');
  6. $m -> query('UPDATE gracze SET poziom = poziom + 1 WHERE id = '.$_SESSION['id'].' LIMIT 1');
  7. }
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
Cytat(trueblue @ 12.03.2017, 15:11:20 ) *
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ćquestionmark.gif

  1. //kolejny poziom
  2. $z = $m -> query('SELECT exp, exp_max FROM gracze WHERE id = '.$_SESSION['id'].' LIMIT 1');
  3.  
  4. while($w = $z -> fetch_assoc()){
  5. if($w['exp'] >= $w['$exp_max']){
  6. $m -> query('UPDATE gracze SET exp = 0 WHERE id = '.$_SESSION['id'].' LIMIT 1');
  7. $m -> query('UPDATE gracze SET poziom = poziom + 1 WHERE id = '.$_SESSION['id'].' LIMIT 1');
  8. }
  9. }
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 tongue.gif
Dzięki!
borabora
  1. UPDATE gracze WHERE id='.$_SESSION['id'].' AND exp>exp_max SET poziom=poziom+1,exp=0

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.
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2024 Invision Power Services, Inc.