Teilungserklärung und Zuordnung = Good Practice?

Question

Da ich ein begeisterter Benutzer von NetBeans bin, erhalte ich immer wieder Vorschläge, meine Variablendeklarationen und Zuweisungen aufzuteilen, wenn ich eine neue Variable einführe. Als superschnelles Beispiel aus der Spitze von meinem Kopf, lassen Sie sich diese Java Linie nehmen:

String someInput = JOptionPane.showInputDialog(null, "- Enter something: "); 

gegen:

String someInput; 
someInput = JOptionPane.showInputDialog(null, "- Enter something: "); 

NetBeans letztere zu bevorzugen scheint (würde ich nicht über andere IDEs wissen, obwohl). Obwohl es eindeutig in beide Richtungen funktioniert, was würde, wenn überhaupt, als "gute Praxis" betrachtet werden? Oder ist es nur eine Frage der persönlichen Vorliebe?

(Offensichtlich ist die zwei bedeutet eine zusätzliche Zeile aufzuteilen, die Sie gerade in einem kombiniert haben könnte, aber das ist nebensächlich.)

Answer 1

Es gibt keinen Grund, die Erklärung und die Zuordnung zu teilen, wenn Sie gerade gehen um sie in aufeinanderfolgenden Zeilen zu haben. Ich würde sie nur aufteilen, wenn die Zuweisung konditional wäre oder wenn sie in einen separaten Codeblock gehen müsste (wie ein try/catch, oder wenn die Zuweisung in einen Konstruktor geht usw.).

Answer 2

Ich glaube nicht, dass es eine große Sache ist. Wählen Sie einen Stil, den Sie mögen, und bleiben Sie dabei (wenn Sie in einem Team sind, hilft es, wenn jeder seinen Code ähnlich auslegt).

Ich persönlich bevorzuge es, meine Variablen direkt zum Zeitpunkt der Deklaration zu initialisieren. Aus diesem Grund würde ich den ersteren gegenüber dem letzteren bevorzugen.

Answer 3

Ein allgemeines Muster, das auf die frühe statisch typisierte Programmierung zurückgeht, besteht darin, alle oben genannten Variablen am oberen Ende des Blocks zu deklarieren und anschließend diesen Werten zuzuweisen.

Mit diesem gesagt, solange Sie in der Lage sind, die Absicht Ihres Codes den Menschen, mit denen Sie daran arbeiten, oder sich selbst in einem Jahr klar zu kommunizieren, sollte es eigentlich keine Rolle spielen.

Lassen Sie uns auf den Aufbau von Programmen unserer traditionellen Haltung ändern: Statt dass unsere Hauptaufgabe einen Computer anzuweisen, sich vorzustellen, was uns eher Konzentrat zu tun, lassen den Menschen zu erklären, was wir wollen Computer zu tun. - Donald Knuth

Answer 4

Code Complete, die von den meisten der Programmierung Gemeinschaft angesehen wird, schlägt in seiner Seite 241, dass als eine Frage der defensiven Programmierung können Sie entweder jede Variable initialisieren, wie sie deklariert ist (erste Option) oder initialisieren Sie es in der Nähe des Ortes, an dem es zum ersten Mal verwendet wird (Ihre zweite Option).

Nichtsdestoweniger schlägt das Buch vor, dass die zweite Option besser ist, denn wenn Sie die erste Option verwenden, hat sich Ihre Variable möglicherweise zwischen ihrer Deklaration und ihrer Verwendung geändert.

In Ihrem Fall ist der Unterschied zwischen Deklaration und Verwendung im zweiten Fall eine einzelne Zeile. Daher gelten die Bedenken des Buches nicht. Nichtsdestoweniger sind seine Bedenken berechtigt und dies könnte der Grund sein, warum die Designer von Netbeans diesen Tipp aufgenommen haben (obwohl sie ihn überstrapaziert haben).

Answer 5

Netbeans schlägt vor, diese als es Codierung beschleunigen kann, zum Beispiel diese Zuordnung prüfen

final String myString = "somevalue" 

Sie dann entscheiden, dass tatsächlich der Wert auf etwas ein von zu der konvertieren boolean

final String myString; 
if(something) { 
    myString = "somevalue" 
} else { 
    myString = "someothervalue" 
} 

hängt Anderenfalls können Sie den Code-Hinweis verwenden, um Deklaration und Zuweisung zu verschütten, und die Vorlage if...else, um den Code in sehr wenigen Schritten zu generieren.

Im Allgemeinen, wenn Sie bedingungslos zuweisen, gibt es keinen Grund, Deklaration und Zuweisung zu teilen.

Answer 6

Dies war eine Option, die zu NetBeans irgendwo zwischen 7.0 und 7.3 hinzugefügt wurde. Wenn Sie nicht diesen Hinweis mag, kann es durch unchecking geändert werden: Extras> Optionen> Editor> Hinweise> Vorschläge> Split Erklärung