[PHP] operator trójkowy Opcje

[Fred1485]

Witam,

Czy pisząc kod jestem zobowiązany w pewnych blokach kodu używać zwykłego ifa czy też skróconego czy to zależy tylko ode mnie (wiem może pytanie głupie ale dla mnie skrócony if to coś bardzo przydatnego, kod się wtedy robi dla mnie bardziej czytelny w pewnych momentach) Jak wy to rozpatrujecie u siebie?

[Turson]

Jeżeli nie wykorzystujesz nigdzie pola klasy to nie ma sensu go inicjalizować

[Janusz1200]

Jeżeli nie wykorzystujesz nigdzie pola klasy to nie ma sensu go inicjalizować

To zdaje się należy do dobrych praktyk (inicjowanie zmiennych choćby nullem) a poza tym przydaje się szczególnie w PHP, gdy ktoś ma przejąć kod i nie wie, co i po co ta zmienna. Z tym, że zmienna musi być oczywiście gdzieś w metodzie użyta, jak nie jest, to po co ją deklarować?


Ale Autorowi chodziło chyba o to, że ni z gruchy ni z pietruchy wprowadza w prywatniej metodzie

$this :

Kod

$this->_inactive_link = str_replace($pattern, $replace, $this->_params['inactive']);
return $this->_inactive_link;

zamiast

Kod

$inactive_link = str_replace($pattern, $replace, $this->_params['inactive']);
return $inactive_link;

(czy lepiej - od razu

Kod

return tr_replace($pattern, $replace, $this->_params['inactive']

- po co ta zmienna $inactive_link? )

Otóż takie $this stosowane nieświadomie jest bez sensu i szkodliwe. Świadome zresztą też jest złe. Gdzie enkapsulacja? Gdzie unikanie efektów ubocznych? Ale mi się zdarza (IMG:style_emoticons/default/smile.gif)

Ten post edytował Janusz1200 25.04.2015, 21:03:33