[PHP]Czy tak ma wygląać funkcja? Opcje

[-TreleMorela-]

Witam.
Czy tak ma wyglądać funkcja sprawdzająca czy dany rekord istnieje w bazie?

[PHP] pobierz, plaintext 
    public function sprawdz($sql) {
		$result = mysql_num_rows($sql, $this->connection);
		if ($result >0){
			$istnieje = '1';
		}
		else{
			$istnieje = '0';
		}
        return $istnieje;
    }
[PHP] pobierz, plaintext 

[konrados]

Nie wspominając o tym, że jest przeskomplikowana bo wystarczyłoby:

[PHP] pobierz, plaintext 
return $result >0?return 1:return 0;
[PHP] pobierz, plaintext 

to czym jest zmienna $sql?. Co tam dajesz? Powinien być rezultat funkcji mysql_query, tu masz przykład: http://www.w3schools.com/php/func_mysql_num_rows.asp (niestety php.net dzisiaj nie chce działać).

Ten post edytował konrados 9.06.2012, 16:13:11

[-TreleMorela-]

[PHP] pobierz, plaintext 
$sql = query($zapytaniel)
[PHP] pobierz, plaintext 

Jeśli tak może być to będzie działać

[PHP] pobierz, plaintext 
return $result >0?return 1:return 0;
[PHP] pobierz, plaintext 

[konrados]

Ja tam nie wiem, to było pytanie czy stwierdzenie? I co to za funkcja "query"?

Ten post edytował konrados 9.06.2012, 16:42:23

[-Gość-]

[PHP] pobierz, plaintext 
return $result->0?return 1:return 0;
[PHP] pobierz, plaintext 

Nie działa, biała strona jest.

[Mephistofeles]

Oczywiście, że nie działa.
To forma skróconego ifa, w tym wypadku po zadziałaniu warunku dostaniesz return return 1 / return return 0.
Poprawnie powinno być

[PHP] pobierz, plaintext 
return ($result > 0) ? 1 : 0
[PHP] pobierz, plaintext 

Masz włączone raportowanie błędów?

W zasadzie to mogłoby być nawet tak:

[PHP] pobierz, plaintext 
return ($result > 0);
[PHP] pobierz, plaintext 

Ten post edytował Mephistofeles 9.06.2012, 16:53:55

[konrados]

No to żeś z siebie wydusił kupę informacji.

Wciąż nie wiem co to jest za funkcja "query".

Poza tym ten kawałek kodu, który dałem nie jest rozwiązaniem problemu - ja tylko wspomniałem, że można prościej.

Wciąż nie wiem też, czym jest zmienna "sql" którą przekazujesz do metody "sprawdz".

Jak na razie ja napisałem więcej od Ciebie, jak tak dalej pójdzie, to raczej nie otrzymasz tu odpowiedzi.

@Mephistofeles: też fakt, za mało kawy dzisiaj:)

Ten post edytował konrados 9.06.2012, 16:55:06