Weisen Sie der Variablen das Ergebnis der Einzeloperationsmethode zu, bevor Sie sie zurückgeben oder einfach zurückgeben?

Question

bevorzugen Sie Schreibmethode mit sofortiger Rückgabe des Ergebnisses, um es zuerst der Variablen zuzuordnen? Um es klar zu sagen, betrachten wir nur die Situation, die unten dargestellt ist, wenn keine anderen Operationen im Methodenkörper ausgeführt werden.

Erste Option:

private List<Integer> getIdsOfUsersLoggedWithinLastHour() { 
    return userDAO.getUsersLoggedWithinLastHour().stream().map(User::Id).collect(Collectors.toList()); 
} 

Zweite Option:

private List<Integer> getIdsOfUsersLoggedWithinLastHour() { 
     List<Integer> ids = userDAO.getUsersLoggedWithinLastHour().stream().map(User::Id).collect(Collectors.toList()); 
     return ids; 
    } 

IMHO keine Notwendigkeit besteht, aufgrund der Verfahrensoperationen zu Variablen zugewiesen werden, wenn Sie mit ihm nichts zu tun und nur zurück. Der Rückgabetyp wird in Methodensignatur deklariert, so dass Sie ihn einfach überprüfen können. Die Codesauberkeit ist ungefähr gleich. Sie verwenden einige zusätzliche Speicher- und CPU-Operationen (vielleicht nicht so wichtig heute aber es ist immer etwas Zusätzliches). Ein Freund von mir behauptet, dass das Initialisieren von Variablen besser für das Debuggen und weitere Code-Entwicklung ist, aber ich denke, es ist überflüssig und kann getan werden, wenn Sie das wirklich brauchen, nicht 'für die Zukunft'. Was ist deine Meinung?

Answer 1

Ich weise das Ergebnis der Variablen nur dann zu, wenn ich eine gewisse Logik damit machen muss, bevor ich den Wert der Methode zurückgebe. Ansonsten verwende ich Codierungsstil aus dem ersten Beispiel - macht Code prägnant.

zweiter Code-Stil kann manchmal während der Debug-Prozesses helfen, aber Sie sollten diese IMHO

auch nicht begehen, wenn Sie zweites Beispiel inspizieren mit jedem gängigen statischen Analyse-Tool wäre es ein Ergebnis wie "local variable is redundant" geben, das ist mehr Grund, diese Vars in realen Projekten zu beseitigen.

Guter Artikel zu diesem Thema Design: http://www.yegor256.com/2015/09/01/redundant-variables-are-evil.html

Aber Sie müssen verstehen, dass es nur eine gängige Praxis, niemand kann Sie nicht redundante Variablen machen verwenden - Code arbeitet mit ihnen richtig, es ist nur eine Frage der Geschmack wie viele andere Design-Prinzipien

Answer 2

In vielen Fällen plump ich für die zweite Option und in dem genannten Fall würde ich fast sicher.

Damit können Sie einen Haltepunkt auf return ids; platzieren, der beim Debuggen hilfreich sein kann. (Die Überprüfung des Inhalts eines CPU-Registers ist eine Option in Low-Level-Sprachen wie C, aber ich denke nicht, dass für JVM-basierte Sprachen verfügbar ist).

Eine Reihe von Funktionen, die einen anonymen temporären Stapel zurück auf den Stapel zurückbringen, können schwer zu debuggen sein.

Ich würde mir keine Sorgen machen über irgendeinen Overhead, den die zweite Wahl einführt: das sollte optimiert werden, und eine zusätzliche Referenzvariable zu deklarieren ist wahrscheinlich kein Leistungsengpass, auch wenn sie nicht optimiert wurde. (Beachten Sie, dass in C++ es wird über einen Prozess Named Rückgabewert Optimierung genannt optimiert werden und ich glaube, dass Java nachziehen würde.) Denken Sie daran, dass Code in der Lage zu debuggen und warten ist ebenso wichtig wie Eleganz; was auch immer das in diesem speziellen Fall bedeutet.