Pisanie funkcji - optymalizacja Opcje

[andrzejb]

czy taki sposob pisania fnkcji jest poprawny

[PHP] pobierz, plaintext 
<?php
function codeSig($code,$user_id){
		if(@mysql_query("INSERT INTO codes_sig (code, user_id, created) 
							VALUES ('{$code}','{$user_id}',NOW()")){
		return 1;
		exit;
							}
							return 0;
 
	}
?>
[PHP] pobierz, plaintext 

Tak wiec funkcja jest wykonywana , sprawdzamy warunek jezeli poprawny zworc 1 i zakoncz funkcje. Jezeli warnek sie nie spelni zwroc 0. Jakies sugestje ? uwagi ?

[dtb]

mozna to zrobic duzo prosciej:

[PHP] pobierz, plaintext 
<?php
function codeSig($code,$user_id){
return @mysql_query("INSERT INTO codes_sig (code, user_id, created) VALUES ('{$code}','{$user_id}',NOW()");
}
?>
[PHP] pobierz, plaintext 

Ten post edytował dtb 11.11.2006, 12:12:57

[dr_bonzo]

Kod dtd jest wlasciwy:
1. exit jest niepotrzebne -- i tak nigdy nie zostanie wykonane
2. php to nie C -- uzyj true/false do okreslania prawidlowosci wykonania funkcji zamiast 1/0
3. manual:

Dla innych [dla INSERT -- dr_bonzo] zapytań SQL mysql_query() zwraca TRUE lub FALSE informując czy zapytanie zakończyło się sukcesem czy też nie.

Odradzam uzywania malpy (@) -- lepiej uzyc w kodzie produkcyjnym error_reporting( 0 );
a w czasie tworzenienia aplikacji E_ALL zeby widziec gdzie popelniasz bledy.

[tes]

2. php to nie C -- uzyj true/false do okreslania prawidlowosci wykonania funkcji zamiast 1/0

Bez sensu, w C także można zwracać wartości logiczne.

[dr_bonzo]

w C także można zwracać wartości logiczne.

No tak. Pogooglowalem i od C99 mozna. Nie jestem tak dobry w C jak w php i gdy chcialem uzyc true/false to mi nie dzialalo. Dopiero w C++ bylo ok.


Dalej -- juz sam nie wiem jak to jest z booleanami :/

[andrzejb]

tak wiec dalej

[PHP] pobierz, plaintext 
<?php
function codeTake($code,$type='sms'){
		if(mysql_query("UPDATE codes SET modfied=NOW(), left=0 WHERE code='{$code}'")){
			return 1;
			exit;
		}else{
			return mysql_query("UPDATE codes SET modfied=NOW(), left=left-1 WHERE code='{$code}'");
			exit;
		}
 
		return 0;		
	}
?>
[PHP] pobierz, plaintext 

z tym pierwszym warunkiem już chyba nic nie zrobie ?

[nospor]

@andrzejb ale na grzyba ktos ma sie trudzic i mowic uwagi, jak ty olewke na calego i tak robisz?
Patrz co ci napisano wczesniej:

exit jest niepotrzebne -- i tak nigdy nie zostanie wykonane

A ty co na to? A ty na to: "spadaj koles i tak sie nie znasz a ja zrobie po swojemu".

Pozatym Twoja konstrukcja jest bez sensu:
Jesli nie powiedzie sie zapytanie to wykonaj je jeszcze raz. Gdzie sens i logika?
Tak wiem, ze zapytania te roznią sie tym kawalkiem: left=left-1, ale to nic nie zmienia, gdyz pierwszy update zwroci ci tylko false gdy jest blad zapytania (pomijam juz brak polaczenia do bazy), wiec jesli pierwsze sie nie wykona to nie wykona sie rowniez i drugie, ktore skladniowo jest identyczne

[andrzejb]

przyznam blad logiczny wagi o exicie niezauwazylem

[PHP] pobierz, plaintext 
<?php
function codeTake($code,$type='sms'){
		if($type=='sms'){
			return mysql_query("UPDATE codes SET modfied=NOW(), left=0 WHERE code='{$code}'");
		}else{
			return mysql_query("UPDATE codes SET modfied=NOW(), left=left-1 WHERE code='{$code}'");
		}
		return false;		
	}
?>
[PHP] pobierz, plaintext 

to jest tylko stare jadro i tak ta czesc ma zmieszany model z kontrolerem ale to chodzi o mozliowsc przeniesienia kod do aplikacji struktralnych

czmu exit nie zostanie wykonany ? przeciez w takim razie powzyszy kod bez wzgledu na zapytanie i tak zwroci 0 bo na koncu zostanie wykonany return 0 ?

Ten post edytował andrzejb 12.11.2006, 11:02:33

[dtb]

gdy uzywasz return to funkcja przestaje byc wykonywana, np. funkcja:

[PHP] pobierz, plaintext 
<?php
function dupa() {
echo '1';
return true;
// od tej pory funkcja przestajem byc wykonywana, bo juz zwrocila jakas wartosc.
echo '0';
return false;
}
?>
[PHP] pobierz, plaintext 

wydrukuje tylko 1 i zwroci true

Ten post edytował dtb 12.11.2006, 13:53:46