Sollte ich weniger lokale Variablen verwenden oder auf Prägnanz setzen?

Question

ich in letzter Zeit wurde über die Verwendung lokaler Variablen zum Beispiel denken:

Sollte ich es vorziehen, diese:

String firstString = classString.getFirstString(); 
String secondString = classString.getSecondString(); 

ClassGlobal.execute(firstString, secondString); 

oder diese;

Ich persönlich würde das spätere bevorzugen, da es prägnanter ist und zwei zusätzliche Instanzvariablen nicht benötigt. Aber ist es eine gute Sache zu tun? Zahle ich Code-Lesbarkeit für Prägnanz? Oder ist es nur ok, einen zu benutzen?

Answer 1

Wie für mich, sollten Sie Code Lesbarkeit Priorität vor der Minimierung der Anzahl der Variablen. Sie können diese Codezeile perfekt verstehen, aber für jeden Programmierer, der Ihren Code in der Zukunft wiederholt, wird er die Verwendung von lokalen Variablen schätzen (noch besser, wenn Sie gute "selbstdefinierende" Namen verwenden).

Answer 2

In diesem speziellen Fall spielt es keine Rolle. Wenn Ihre Zeile für sich allein lesbar ist, müssen die Variablen nicht getrennt werden. Sobald Sie jedoch anfangen, mehr Logik als ein Getter hinzuzufügen, würde ich dazu neigen, sie zu trennen. Betrachten Sie den folgenden Code ein:

ClassGlobal.execute(classString.getFirstString().toLowerCase().substring(0, 10), 
    classString.getSecondString().toLowerCase().substring(0, 10)); 

Diese Linie viel besser wäre, wenn Sie trennen firstString und secondString und ordnen sie aussagekräftige Namen. Wie bei den meisten Dingen im Code gibt es keinen "besten" Weg. Ich würde eher auf der Seite der Lesbarkeit irren.

Answer 3

Ich werde für die erste gehen. Es erhöht die Lesbarkeit. Bei der zweiten reduziert es definitiv einige Zuweisungsoperationen (!), Aber in den meisten Fällen ist dies eine vernachlässigbare Menge an Leistungsverbesserung. Also besser streng mit Lesbarkeit

ALSO, Sie sollten nicht über die vorzeitige Optimierung kümmern. Darüber hinaus machen Compiler heutzutage diese Art der Optimierung für Sie.

Answer 4

Wenn Sie Code erstellen können, indem Sie den Locals einen guten Namen geben, erstellen Sie die zusätzlichen Locals (auch nützlich, wenn Sie interaktiv debuggen müssen ...).

Wenn die Einheimischen keine Bedeutung hinzufügen würden, weil die Methoden, die aufgerufen werden, so gut benannt sind, und Sie eine ziemlich eiserne Fähigkeit haben, Unit Test statt Debug und Breakpoint .... gehen für die kompakte Notation kann besser fühlen. Aber in den meisten Fällen tendiere ich zur Kreation der Einheimischen.

Answer 5

Im Zweifelsfall nur daran erinnern, dieses Zitat von Coding Horror:

immer Code, als ob die Person, die den Code Aufrechterhaltung endet ein gewaltsamer Psychopath ist die, wo Sie leben weiß.

Die Bedeutung dieses Zitats ist "machen Sie es so einfach wie möglich, den Code zu verstehen und zu lesen." Später, wenn Sie das Programm in 5 Jahren weiter pflegen müssen, wird es für Sie einfacher, wenn Sie es so leserlich wie möglich machen. Das ist eine Win-Win-Situation für alle.

Answer 6

Die Lesbarkeit ist das Wichtigste. Versuchen Sie, Ihren Code in 1 Jahr zu lesen, wenn es nicht gut ist, und Sie werden die Antwort bekommen.