[PHP][PDO] Rejestracja Opcje

[Max Damage]

Cześć, czytałem trochę dziś o PDO i postanowiłem użyć go w skryptach. Nie jestem jednak pewny czy dobrze wszystko zrozumiałem i napisałem, więc dobrze by było gdyby ktoś to sprawdził i poprawił, bądź pokazał jak napisać to lepiej. Przykładowy kod: rejestracja.

[PHP] pobierz, plaintext 
function rejestruj($nick,$haslo,$haslo2,$email){
	$lacz=lacz();
	if(strlen($nick)<6)
		throw new Exception('Za krotki nick.');
	$wynik=$lacz->prepare("SELECT nick FROM user WHERE nick=:nick");
	$wynik->bindValue(':nick',$nick,PDO::PARAM_STR);
	$ilosc=$wynik->execute();
	if(count($ilosc) > 0) 
		throw new Exception('podany nick juz istnieje.'); 
	if(strlen($haslo)<6)	
		throw new Exception('Za krotkie haslo.');
	if($haslo!=$haslo2)
		throw new Exception('Hasla nie sa takie same.');
	if(!preg_match("/^[a-zA-Z0-9.-]+@[a-zA_Z0-9.-]+[a-zA-Z0-9.-]+$/",$email)) 
		throw new Exception('Adres email jest niepoprawny');
	$wynik=$lacz->prepare("SELECT email FROM user WHERE email=:email");
	$wynik->bindValue(':email',$email,PDO::PARAM_STR);
	$ilosc=$wynik->execute();
	if(count($ilosc) > 0)
		throw new Exception('podany email juz istnieje.');
	$sql="INSERT INTO user (nick,haslo,email) VALUES (:nick,:haslo,:email)";
	$wynik=$lacz->prepare($sql);
	$wynik->bindValue(':nick',$nick,PDO::PARAM_STR);
	$wynik->bindValue(':haslo',$haslo,PDO::PARAM_STR);
	$wynik->bindValue(':email',$email,PDO::PARAM_STR);
	$wynik->execute();
	if($wynik)
		echo 'Rejestracja pomyslna.';
}
[PHP] pobierz, plaintext 

Z tego co wyczytałem używając bindValue nie muszę już używać na zmiennych innych funkcji typu: stripslashes itp., ponieważ ona sama przed tym zabezpiecza. Może ktoś dokładniej sprecyzować jak to jest?

[erix]

Daaawno było na przedszkolu cokolwiek z wyjątkami jako struktury kontrolne, na plus.

Nie jestem jednak pewny czy dobrze wszystko zrozumiałem i napisałem, więc dobrze by było gdyby ktoś to sprawdził i poprawił, bądź pokazał jak napisać to lepiej. Przykładowy kod: rejestracja.

A nie działa?

Z tego co wyczytałem używając bindValue nie muszę już używać na zmiennych innych funkcji typu: stripslashes itp., ponieważ ona sama przed tym zabezpiecza. Może ktoś dokładniej sprecyzować jak to jest?

Zgadza się.

[Max Damage]

To, że działa nie znaczy że jest dobrze : )
Przede wszystkim nie bardzo wiem jak mam zliczać ilość zwróconych wierszy w zapytaniu. Fakt, że w tamtym kodzie to działało i pokazywało, że np. taki nick jest już zajęty. Ale np. w tym fragmencie kodu już działa źle. Loguje każdego jak leci : )

[PHP] pobierz, plaintext 
function logowanie($nick,$haslo)
{
	$lacz=lacz();
	$sql="SELECT * FROM user WHERE nick=:nick AND haslo=:haslo";
	$wynik=$lacz->prepare($sql);
	$wynik->bindValue(':nick',$nick,PDO::PARAM_STR);
	$wynik->bindValue(':haslo',$haslo,PDO::PARAM_STR);
	$ilosc=$wynik->execute();
	if(count($ilosc)>0) 
		return true;
	else 
		return false;
}
[PHP] pobierz, plaintext 

[zegarek84]

[PHP] pobierz, plaintext 
function logowanie($nick,$haslo)
{
	$lacz=lacz();
	$sql="SELECT * FROM user WHERE nick=:nick AND haslo=:haslo";
	$wynik=$lacz->prepare($sql);
	$wynik->bindValue(':nick',$nick,PDO::PARAM_STR);
	$wynik->bindValue(':haslo',$haslo,PDO::PARAM_STR);
	$ilosc=$wynik->execute();
	if(count($ilosc)>0) 
		return true;
	else 
		return false;
}
[PHP] pobierz, plaintext 

w zapytaniu daj jeszcze limit 1 to szybciej zwróci wynik jeśli znajdzie takiego urzytkownika a i tak ma być jeden...
po wykonaniu zapytania [$wynik->execute();] obiekt $wynik ma w odpowiedzi jakiś wynik...
i np.:
$wynik=$wynik->fetch(); jeśli bedziesz miał zalogowanego urzytkownika zwróci Ci tablicę asocjacyjną oraz odpowiednio pola numeryczne kolejno do asocjacyjnej - ja u siebie akurat mam na tablicy asocjacyjnej ale patrząc na Twoje zapytanie bedzie lepiej skożystać z numerycznej i wracając do tego co napisałem wcześniej:

[PHP] pobierz, plaintext 
$wynik->execute();
$wynik=$wynik->fetch(); 
return isset($wynik[0])?true:false;
[PHP] pobierz, plaintext 

[erix]

Sposób z wyciąganiem jednego rekordu przy sprawdzaniu są dane/nie jest tu najlepszy.

Ale jeśli chcesz najszybciej sprawdzać ilość rekordów z zapytania, to wykonaj dodatkowe FOUND_ROWS - poczytaj, co do czego. Swoją drogą, lepiej dla Ciebie najpierw zwalidować formularz i dopiero wtedy katować bazę. ;]

[Cysiaczek]

A nie lepiej po prostu samo COUNT(id)?

[SQL] pobierz, plaintext 
SELECT COUNT(id) AS number_of_users FROM users WHERE nick=:nick AND haslo=:haslo
[SQL] pobierz, plaintext 

[Max Damage]

Brakuje mi tu po prostu funkcji typu mysql_num_rows(), ale wyczytałem wczoraj, że wcale nie jest ona tak dobra jak się z początku wydaje. W sumie mogę, więc zliczać ilość zwróconych wierszy na kilka sposobów:

[PHP] pobierz, plaintext 
 
$lacz=lacz();
$sql=$lacz->query("SELECT SQL_CALC_FOUND_ROWS * FROM user");
$sql2=$lacz->query("SELECT FOUND_ROWS() AS total");
$wynik=$sql2->fetch();
echo 'Suma: '.$wynik['total'].'<br/>';
$sql2->closeCursor();
 
$ilosc=0;
while($tab=$sql->fetch()){
	echo $tab['nick'].'<br/>';
	echo $tab['haslo'].'<br/>';
	echo $tab['email'].'<br/>';
	$ilosc++;echo '<br/>';
}
echo 'Suma: '.$ilosc.'<br/>';
$sql->closeCursor();
 
echo '<br/>';
$sql4=$lacz->query("SELECT *,COUNT(id) AS suma FROM user");
foreach($sql4->fetchAll() as $wynik){
	echo $wynik['nick'].'<br/>';
	echo $wynik['haslo'].'<br/>';
	echo $wynik['email'].'<br/>';
}
echo 'Suma: '.$wynik['suma'];
$sql4->closeCursor();
[PHP] pobierz, plaintext 

I znów nie pasuje mi kilka rzeczy:
Podobno nie da się wykonać drugiego zapytania bez zamknięcia pierwszego za pomocą closeCursor(). A ja mam dwa zapytania pod rząd i to działa. Więc jak to w końcu jest?
Chciałbym aby zapytanie oprócz tego że zlicza mi wybrane rekordy, zwracało mi także inne pola. Niby mógłbym wykonać dwa zapytania: jedno z COUNT, a drugie pobierające wszystkie rekordy, ale jak to zrobić w jednym zapytaniu? Teraz wyświetla mi tylko dane z pierwszego wiersza (nie wiem dlaczego) i oczywiście sumę, a ja chciałbym dane z wszystkich wierszy.

[erix]

Podobno nie da się wykonać drugiego zapytania bez zamknięcia pierwszego za pomocą closeCursor(). A ja mam dwa zapytania pod rząd i to działa. Więc jak to w końcu jest?

:

PDOStatement::closeCursor() frees up the connection to the server so that other SQL statements may be issued, but leaves the statement in a state that enables it to be executed again.

Niby mógłbym wykonać dwa zapytania: jedno z COUNT, a drugie pobierające wszystkie rekordy, ale jak to zrobić w jednym zapytaniu? Teraz wyświetla mi tylko dane z pierwszego wiersza (nie wiem dlaczego) i oczywiście sumę, a ja chciałbym dane z wszystkich wierszy.

Wyciąga tak dlatego, gdyż count jest funkcją agregującą. Hmm, a found_rows Ci nie pasuje...?

[Max Damage]

Wyciąga tak dlatego, gdyż count jest funkcją agregującą. Hmm, a found_rows Ci nie pasuje...?

Może być, o count pytałem tak z ciekawości. Dzięki za pomoc.