[MySQL][PHP]Niedziałająca funkcja zliczająca rekordy Opcje

[Matt23]

Witam, stworzyłem taką oto funkcję

[PHP] pobierz, plaintext 
<?
function ile_rekordow($tabela, $pole, $wartosc, $pole2 = 0, $wartosc2 = 0, $inne = 0){  //pobiera nazwę tabeli, pole nr 1 do sprawdzenia, wartosc dla pola nr 1, pole nr 2, wartosc dla pola nr2, i parametr "inne"
include("/../config.php");
if($pole2 == 0 && $wartosc2 == 0 && $inne == 0){  //jeśli podane zostały tylko `tabela`, pole i 'wartosc' dla pola
 
	$miejsce = mysql_fetch_assoc(mysql_query("SELECT COUNT(*) FROM $tabela WHERE $pole = '$wartosc'"));
 
	return $miejsce; 
	mysql_close();
	exit;
}
if($pole2 != 0 && $wartosc2 != 0 && $inne == 0){ //jeśli podane są wszystkie pola prócz "inne"
 
	$miejsce = mysql_fetch_assoc(mysql_query("SELECT COUNT(*) FROM $tabela WHERE $pole = '$wartosc' AND $pole2 = '$wartosc2'"));
 
	return $miejsce; 
	mysql_close();
	exit;
 
}
if($pole2 !=0 && $wartosc2 != 0 && $inne == 1){  //jeśli wszystkie pola, a inne ma wartość 1
 
	$miejsce = mysql_fetch_assoc(mysql_query("SELECT COUNT(*) FROM $tabela WHERE $pole = '$wartosc' AND $pole2 != '$wartosc2'"));  //!= przy drugim sprawdzerniu
 
	return $miejsce;
	mysql_close();
	exit;
}
}
?>
[PHP] pobierz, plaintext 

Niestety funkcja nie działa. Próbuję ją wykorzystać między innymi w taki sposób:

[PHP] pobierz, plaintext 
$ist_user_haslo = ile_rekordow('admins', 'nick', $login, 'haslo', $haslo);
$ist_user = ile_rekordow('admins', 'nick', $login, 'haslo', $haslo, 1);
if($ist_user[0] == 1) {
    $status="wrong_password";
    alert('Logowanie nieudane.<br> Sprawdź pisownię nicku oraz hasła.');
    logi($login, $status);
    exit;
}
 
if  ($ist_user_haslo[0] == 0) {
 
    $status="user_doesnt_exist";
    alert('Logowanie nieudane.<br> Nie ma takiego użytkownika.');
    logi($login, $status);
    exit;
}
if($ist_user_haslo[0] == 1){
 
    $status="logged";
    logi($login, $status);
 
[PHP] pobierz, plaintext 

Proszę o wskazanie moich błędów.

[SmokAnalog]

Moim zdaniem jeśli funkcja nazywa się ile_rekordow(), to powinna zwracać finalną liczbę, a nie tablicę. W Twoim przypadku zamiast mysql_fetch_assoc zastanowiłbym się czy nie lepiej użyć mysql_result, która nie zwraca tablicy, ale wartość właściwą.

Co do SQL Injection, to w programowaniu chodzi też o to, żeby pisać kod z myślą o przyszłości. Twoja funkcja może się przydać w Twoich przyszłych projektach i lepiej, żeby była odporna na wszystko. Kiedyś możesz zapomnieć, że ona ma taką wadę i wtedy klops (IMG:style_emoticons/default/smile.gif)

Fajnie, że się uczysz!