[MySQL][PHP] Sprawdzanie czy zapytanie zwróciło wynik? Opcje

[--Ja--]

Witam!

Mam taki fragment kodziku:

[PHP] pobierz, plaintext 
 
$pyt = "SELECT * FROM oczekujace WHERE \"login\" = '$login' AND \"haslo\" = '$haslo' AND \"email\" = '$email'";
$odp = mysql_query($pyt);
 
if($odp)
{
$query2 = "INSERT INTO users VALUES ('$login', '$haslo', '0', '$email')";
$result2 = mysql_query($query2);
} 
else 
{
print "Błędne dane. Skontaktuj się z administratorem w dziale <a href='kontakt.html'>kontakt</a>.";
exit;
}
[PHP] pobierz, plaintext 

Odpowiada on za aktywację konta (link jest wysyłany mailem, a następnie metodą GET pobieram z niego dane do zmiennych $login, $haslo i $mail). Aktywacja konta działa. Jednakże, gdy w linku wpiszę błędne dane, np. login 'Ciapek', to ono się i tak doda do tabeli 'users', mimo, że nie zostało wcześniej zarejestrowane. Zadaniem pierwszej instrukcji warunkowej, było blokowanie tego, jednakże nie działa.

Męczę się nad tym już 30 min, rozwiązania nie znalazłem. Jedyne co mi zostało, to prosić was o pomoc, więc... "HELP!" (IMG:style_emoticons/default/wink.gif) .

[peter13135]

albo

[SQL] pobierz, plaintext 
SELECT count(*) ...
[SQL] pobierz, plaintext 

i sprawdzaj pierwszy element tablicy zwróconej przez mysql_fetch_array
albo sprawdzaj w tej sposób:

[PHP] pobierz, plaintext 
if(mysql_num_rows($odp))
[PHP] pobierz, plaintext 

[--Ja--]

albo

[SQL] pobierz, plaintext 
SELECT count(*) ...
[SQL] pobierz, plaintext 

i sprawdzaj pierwszy element tablicy zwróconej przez mysql_fetch_array
albo sprawdzaj w tej sposób:

[PHP] pobierz, plaintext 
if(mysql_num_rows($odp))
[PHP] pobierz, plaintext 

Nie działa.

[bastard13]

Po pierwsze to zapisałbym to twoje zapytanie tak:

[PHP] pobierz, plaintext 
$pyt = "SELECT * FROM oczekujace WHERE login = '$login' AND haslo = '$haslo' AND email = '$email'";
[PHP] pobierz, plaintext 

ponieważ ten cudzysłów jest trochę mylący może wskazywać na wartość, a nie na nazwę kolumny.

Po drugie wyrażenie w ifie:

[PHP] pobierz, plaintext 
if(mysql_query($pyt))
[PHP] pobierz, plaintext 

po zrzutowaniu na boolean, zazwyczaj zwróci true, bo:

For SELECT, SHOW, DESCRIBE, EXPLAIN and other statements returning resultset, mysql_query() returns a resource on success, or FALSE on error.

Więc tak jak napisał ci peter13135, powinieneś użyć:

[PHP] pobierz, plaintext 
if(mysql_num_rows($odp))
[PHP] pobierz, plaintext 

Dodatkowo dodaj sobie przed tym ifem (do celów debugowania):

[PHP] pobierz, plaintext 
var_dump(mysql_num_rows($odp)); //będziesz wiedział ile wierszy wyciąga
var_dump(mysql_fetch_assoc($odp)); //jeżeli coś wyciąga, to zobaczysz co
[PHP] pobierz, plaintext 

[lukaskolista]

A nie lepiej

[PHP] pobierz, plaintext 
$result = mysql_query("...");
if (mysql_fetch_row($result) !== FALSE)
{
 
}
[PHP] pobierz, plaintext 

btw. cos sie znacznik quote zepsul, bierze w cytata calego posta zamiast fragmentu (przynajmniej w tym przypadku tak bylo)

Ten post edytował lukaskolista 9.08.2011, 07:14:47

[--Ja--]

Po pierwsze to zapisałbym to twoje zapytanie tak:

[PHP] pobierz, plaintext 
$pyt = "SELECT * FROM oczekujace WHERE login = '$login' AND haslo = '$haslo' AND email = '$email'";
[PHP] pobierz, plaintext 

ponieważ ten cudzysłów jest trochę mylący może wskazywać na wartość, a nie na nazwę kolumny.

Po drugie wyrażenie w ifie:

[PHP] pobierz, plaintext 
if(mysql_query($pyt))
[PHP] pobierz, plaintext 

po zrzutowaniu na boolean, zazwyczaj zwróci true, bo:


Więc tak jak napisał ci peter13135, powinieneś użyć:

[PHP] pobierz, plaintext 
if(mysql_num_rows($odp))
[PHP] pobierz, plaintext 

Dodatkowo dodaj sobie przed tym ifem (do celów debugowania):

[PHP] pobierz, plaintext 
var_dump(mysql_num_rows($odp)); //będziesz wiedział ile wierszy wyciąga
var_dump(mysql_fetch_assoc($odp)); //jeżeli coś wyciąga, to zobaczysz co
[PHP] pobierz, plaintext 

Zadziałało, ten zapis musiał nie być poprawny. Temat można zamknąć.