[php] optymalny zapis Opcje

[Crozin]

@fifi209, @kapuch: Chodziło o sytuację gdzie zależy nam tylko na inkrementacji wartości zmiennej, np.:

[PHP] pobierz, plaintext 
for (...; ...; ++$i)
   if () {
     ++$j;
   }
[PHP] pobierz, plaintext 

W tych przypadkach nie ma realnej różnicy.

Szczerze to jestem zdziwiony jak długo się ten wątek ciągnie, więc tam na zakończenie (oby)
1) Mikroptymalizacja z reguły jest pozbawiona sensu w "zadaniach wysokiego poziomu" - to odnośnie pre/postinkrementacji, używania bloków if/else lub ?: itp.
2) Kod powinno się pisać czytelny. Jeżeli jeden zapis jest czytelniejszy od drugiego to można z niego skorzystać. A to kiedy coś jest czytelne... no tyle to już chyba potraficie sami ocenić?

[S3L41N]

No dobrze, ale są jakieś konkretne wytyczne kiedy używać jednego zapisu, a kiedy drugiego?
Których z zapisów jest bardziej poprawny, czytelny i zalecany w następujących sytuacjach?
Sytuacja 1:

[PHP] pobierz, plaintext 
//1.1
$syshelpers = file_exists($syspath = trim(FRAMEWORKSDIR.'/'.HELPERSDIR, '/')) ? array_slice(scandir($syspath), 2) : array();
//1.2
if (file_exists($syspath = trim(FRAMEWORKSDIR.'/'.HELPERSDIR, '/')))
     $syshelpers = array_slice(scandir($syspath), 2);
else
     $syshelpers = array();
//1.3
$syshelpers = array();
if (file_exists($syspath = trim(FRAMEWORKSDIR.'/'.HELPERSDIR, '/')))
     $syshelpers = array_slice(scandir($syspath), 2);
[PHP] pobierz, plaintext 

Sytuacja 2:

[PHP] pobierz, plaintext 
//2.1
$type = (substr($type, -1) == 'y') ? substr($type, 0, -1).'ies' : $type.'s';
//2.2
if (substr($type, -1) == 'y')
     $type = substr($type, 0, -1).'ies';
else
     $type = $type.'s';
[PHP] pobierz, plaintext 

Sytuacja 3:

[PHP] pobierz, plaintext 
//3.1
return (isset($this->$key) OR isset($this->data[$key])) ? TRUE : FALSE;
//3.2
if (isset($this->$key) OR isset($this->data[$key]))
     return TRUE;
return FALSE;
//3.3
if (isset($this->$key) OR isset($this->data[$key]))
     return TRUE;
else
     return FALSE;
[PHP] pobierz, plaintext 

Sytuacja 4:

[PHP] pobierz, plaintext 
//4.1
return (isset($this->$key) ? $this->$key : (isset($this->data[$key]) ? $this->data[$key] : NULL));
//4.2
return (isset($this->$key) ? $this->$key :
           (isset($this->data[$key]) ? $this->data[$key] :
           NULL));
//4.3
if (isset($this->$key))
     return $this->$key;
elseif (isset($this->data[$key])
     return $this->data[$key];
else
     return NULL;
//4.4
if (isset($this->$key))
     return $this->$key;
else if (isset($this->data[$key])
     return $this->data[$key];
else
     return NULL;
//4.5
if (isset($this->$key))
     return $this->$key;
if (isset($this->data[$key])
     return $this->data[$key];
return NULL;
[PHP] pobierz, plaintext 

Sytuacja 5:

[PHP] pobierz, plaintext 
//5.1
return isset(self::$instance) ? self::$instance : self::$instance = new self;
//5.2
if (isset(self::$instance))
     return self::$instance;
else
     return self::$instance = new self;
//5.3
if (isset(self::$instance))
     return self::$instance;
return self::$instance = new self;
//5.4
if (!isset(self::$instance))
     self::$instance = new self;
return self::$instance;
[PHP] pobierz, plaintext 

Które z nich, a może jakieś inne są najbardziej poprawne?

[Crozin]

No ileż można - który zapis wydaje Ci się najbardziej przejrzysty? Wariant pierwszy, drugi lub trzeci - używaj który Tobie najbardziej pasuje. Mnie pasuje czwarty Tobie drugi - używaj sobie drugiego. Nie ma żadnych wytycznych - to tylko forma zapisu!

[S3L41N]

No ok, to poprostu napisz, które zapisy Ty byś wybrał...

[attimo]

Ja też wole 4. może i wdurgim kod jest krótszy prawie 4 krotnie, ale jeśli musiałbym tam coś zmieniac to rozszyfrowanie tego zajeło by mi wiecej czasu dlatego wole już wiecej linijek ale czytelniej choc slowo czytelniej jest subiektywne i dla kogos innego sytuacja 2 moze byc czytelniejsza.

[S3L41N]

attimo widzę, że chyba nie wczytałeś się w posta. Jest pięć sytuacji z różnymi możliwościami rozwiązań (kolejne rozwiązania dla poszczególnych sytuacji są wypunktowane w komentarzach).

[Crozin]

#1

[PHP] pobierz, plaintext 
$syspath = trim(FRAMEWORKSDIR.'/'.HELPERSDIR, '/');
$syshelpers = file_exists($syspath) ? array_slice(scandir($syspath), 2) : array();
[PHP] pobierz, plaintext 

#2

[PHP] pobierz, plaintext 
$type = substr($type, -1) == 'y' ? substr($type, 0, -1).'ies' : $type.'s';
[PHP] pobierz, plaintext 

#3

[PHP] pobierz, plaintext 
return isset($this->$key) || isset($this->data[$key]);
[PHP] pobierz, plaintext 

#4

[PHP] pobierz, plaintext 
if (isset($this->$key)) {
     return $this->$key;
} elseif (isset($this->data[$key]) {
     return $this->data[$key];
}
 
return NULL;
[PHP] pobierz, plaintext 

#5

[PHP] pobierz, plaintext 
if (!isset(self::$instance)) {
     self::$instance = new self;
}
 
return self::$instance;
[PHP] pobierz, plaintext 

Tylko nie wiem po co to podaję. Naprawdę tak ciężko ocenić co dla siebie samego jest wygodne?

Ten post edytował Crozin 3.08.2010, 13:57:03