2013-03-06 8 views
11

Netbeans wydaje się sugerować, że zbyt wiele zadań jest bardzo złe i powinno zostać zmienione.W PHP zbyt wiele przypisań uważa się za złe? Jeśli tak, dlaczego?

tj

$foo = ' bar '; 
$foo = trim($foo); 

powinny lepiej kodowane jako

$foo = ' bar '; 
$trimmed_foo = trim($foo); 

Czy to dopuszczalne? Jeśli tak, dlaczego? Wiem, że mogę wyłączyć ten typ podpowiedzi w ustawieniach, ale tylko sprawdzam, czy ktoś coś zauważył.

+0

To jest ciekawe pytanie. Myślę, że to naprawdę nie ma znaczenia, ale silnik PHP zaskoczył mnie wcześniej. –

+0

Dziękujemy za nieprzyjęcie odpowiedzi, które nie odpowiadają. –

Odpowiedz

5

Idealnie nazwa zmiennej powinna opisywać zawartość. Jeśli wielokrotnie przypisujesz tę samą zmienną, sugeruje ona, że ​​zawartość zmiennych nie jest dobrze zdefiniowana.

Dodatkowo, jeśli Twój kod ma coś takiego:

$foo = ' bar '; 
// some code, mybe 100 lines of it 
// now do something with $foo 

Co się dzieje, jeśli zaktualizować kod, aby dodać $foo = trim($foo); się powyżej? Łamiesz poniższy kod.

5

Te warianty nie mają znaczenia. Jeśli cokolwiek, pierwsze jest lepsze, ponieważ pozwala uniknąć zaśmiecania zakresu zmiennymi.

myślę co ostrzeżenie naprawdę oznacza to, że należy spróbować zrobić

$foo = trim(' bar '); 

bezpośrednio (lub cokolwiek $ foo jest naprawdę są ustawione w pierwszej kolejności), zamiast zapisywania go w tymczasowy. Oczywiście nie zawsze jest to możliwe.

0

IMHO, pierwsza metoda jest w porządku, aby uzyskać lepszą porównywalność drugiego.

Z drugiej strony, drugie podejście może być bardziej pomocne, jeśli potrzebujesz więcej zmiennych opisowych. Może to również pomóc, jeśli wiele osób pracuje nad tym samym projektem.

7

Ths to nowy ostrzeżenie, około połowy 2012.

Ponieważ stosowane trymowania przykład Zgaduję, że przeczytałeś ten https://blogs.oracle.com/netbeansphp/entry/several_new_hints, która próbuje wyjaśnić ostrzeżenie. Moja uwaga na to ostrzeżenie jest taka, że ​​próbuje ostrzec cię o przypadkowym ponownym użyciu zmiennej. Jako taki jest bardzo przydatny. W przypadku przycinania, który wydajesz, wydaje się nieco ciężki. Jeśli zmiana przypisania jest następująca, prawdopodobnie nie jest przypadkowa.

+1

Popraw Peter, czytałem stronę, którą zacytowałeś, to nie mówi dokładnie dlaczego jest źle. Powstrzymaj to od przypadkowego wystąpienia z dobrego powodu, aby go włączyć. – xelber

2

Jeśli nie podoba to wskazówkę, można po prostu wyłączyć go w Narzędzia -> Opcje -> Editor -> Porady -> PHP -> Niezmienne Zmienne

Powiązane problemy