2013-03-06 4 views
11

Netbeans scheint zu viele Zuweisungen zu empfehlen ist ziemlich schlecht und sollte geändert werden.In PHP, werden zu viele Zuweisungen als schlecht angesehen? Wenn ja warum?

heißt

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

Sollte besser codiert als

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

Ist das akzeptabel? Wenn ja warum? Ich weiß, dass ich diesen speziellen Tipp-Typ in der Einstellung ausschalten kann, aber nur überprüfen, ob jemand etwas dafür gefunden hat.

+0

Dies ist eine interessante Frage. Ich stelle mir vor, dass es eigentlich egal ist, aber die PHP-Engine hat mich schon einmal überrascht. –

+0

Danke, dass Sie keine Antworten annehmen, die nicht beantwortet werden. –

Antwort

5

Idealerweise sollte der Variablenname den Inhalt beschreiben. Wenn Sie wiederholt derselben Variablen zuweisen, schlägt dies vor, dass der Inhalt der Variablen nicht gut definiert ist.

Außerdem, wenn Ihr Code so etwas wie dieses hat:

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

Was ist, wenn Sie den Code aktualisieren geschieht $foo = trim($foo); oben zu addieren? Sie brechen den folgenden Code.

5

Diese Varianten machen keinen Unterschied. Wenn überhaupt, ist das erste besser, weil es den Bereich mit Variablen nicht überfüllt.

Ich denke, was die Warnung wirklich bedeutet, ist, dass Sie versuchen sollten,

$foo = trim(' bar '); 

direkt zu tun (oder was auch immer $ foo ist wirklich in erster Linie eingestellt wird), statt es in einem temporären Speicherung. Natürlich ist das nicht immer möglich.

0

IMHO, ist der erste Weg in Ordnung für mehr Leistung im Vergleich zum zweiten.

Auf der anderen Seite kann der zweite Ansatz hilfreicher sein, wenn Sie mehr beschreibende Variablen benötigen. Auch das kann mehr helfen, wenn mehrere Personen an demselben Projekt arbeiten.

7

Ths ist eine neue Warnung, circa Mitte 2012

Da Sie die Trimmung Beispiel verwendete ich nehme an, dass Sie diese https://blogs.oracle.com/netbeansphp/entry/several_new_hints gelesen haben, die die Warnung zu erklären versucht. Ich nehme diese Warnung an, weil sie versucht, Sie vor versehentlicher Wiederverwendung einer Variablen zu warnen. Als solches ist es sehr praktisch. In der Ordnung, die Sie geben, scheint es ein bisschen schwer angeschlagen. Wenn die Neuzuweisung auf der folgenden Zeile ist, ist es wahrscheinlich nicht zufällig.

+1

Richtig Peter, ich habe die von dir zitierte Seite gelesen, sie sagt dir aber nicht genau, warum es schlecht ist. Hör auf, es aus einem guten Grund zufällig zu tun, um es eingeschaltet zu lassen. – xelber

2

Wenn Sie nicht diesen Hinweis mögen, können Sie es in Werkzeuge einfach deaktivieren -> Optionen -> Editor -> Tipps -> PHP -> Immutable Variablen

Verwandte Themen