[php] Funkcje, return FALSE Opcje

[Jawor]

Witam.

Mam pytanie ktora definicja funkcji jest poprawna/lepsza/szybsza.
Jezeli pod kazdym wzgledem (oprocz ilosci kodu) jest to to samo napiszcie ktory sposob Wy wolicie.

[PHP] pobierz, plaintext 
<?php
 
  function nazwa($ile)
{
 
	if ($ile == '1')
  {
	// kilka instrukcji
	return TRUE;
  }
	else
  {
	return FALSE;
  }
 
}
 
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
  function nazwa($ile)
{
 
  if ($ile != '1') { return FALSE; }
 
  // kilka instrukcji
  return TRUE;
 
}
 
?>
[PHP] pobierz, plaintext 

Pozdrawiam
J.

Ten post edytował Jawor 23.01.2008, 21:49:29

[Grzyw]

Mogę się mylić, ale są one równorzędne. Warunki "równości" i "różności" wymagają od parsera identycznej analizy. A bloki instrukcji w obydwóch przypadkach są już identyczne.

[Cotter]

Podobnie jak przedmówcy wydaje mi się że są równoważne pod względem wydajnościowym, ale głowy nie dam.

Jeżeli chodzi o preferowany przeze mnie sposób to jest to zdecydowanie rozwiązanie drugie. Dlaczego? Ponieważ kod jest w mniejszej liczbie nawiasów oraz sprawdzanie w ten sposób pewnych warunków koniecznych wydaje mi się po prostu bardziej czytelne. Dzieli to w pewien sposób zadania funkcji na obsługiwane w pierwszej kolejności sytuacje wyjątkowe i sytuację normalną obsługiwaną kiedy żaden wyjątek nie zajdzie.

[.radex]

Osobiście preferuję drugi sposob. Kiedy pod if/for/while masz tylko jedną linijkę kodu, nie musisz dawać klamer (dla czytelności tylko tabulator). W kwestii kodu nie ma to znaczenia, choć w drugim jest raczej bardzo mała różnica (taka, że na pewno jej nie zauważysz).

Ten post edytował radex_p 24.01.2008, 15:28:28

[Jawor]

Ok. Dzieki wielkie za wasze opinie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)