Bessere Java-Methode Syntax? Früh oder spät zurückkehren?

Question

Duplizieren:Should a function have only one return statement?

Oft Sie vielleicht eine Methode, die zahlreiche Bedingungen überprüft und gibt einen Status (boolean sagen jetzt lässt). Ist es besser, eine Flagge zu definieren, setzen Sie ihn während des Verfahrens, und senden Sie es am Ende:

boolean validate(DomainObject o) { 
    boolean valid = false; 
    if (o.property == x) { 
    valid = true; 
    } else if (o.property2 == y) { 
    valid = true; 
    } ... 
    return valid; 
} 

oder ist es besser/richtigere einfach zurück, sobald Sie die Methode der Ergebnisse wissen?

boolean validate(DomainObject o) { 

    if (o.property == x) { 
    return true; 
    } else if (o.property2 == y) { 
    return true; 
    } ... 
    return false; 
} 

Jetzt könnte es offensichtlich versuchen/fangen Blöcke und alle anderen Arten von Bedingungen, aber ich denke, das Konzept ist klar. Meinungen?

Answer 1

Wenn es eine Methode ist, die Sie Tausende Male anrufen werden, dann ist frühzeitige Rückkehr besser, um eine [etwas] erhöhte Leistung zu erreichen.

Wenn nicht, dann würde ich eine späte Rückgabe bevorzugen, da es die Lesbarkeit verbessert.

Denken Sie daran, dass Programmierer in der Regel mehr Zeit mit Lesen verbringen als mit dem Schreiben von Code. Daher ist alles, was Sie tun können, um die Lesbarkeit zu verbessern, sicherlich willkommen.

Answer 2

Für mich ist das eine Art von diesen religiösen Kriegsthemen mit keiner richtigen Antwort. Das Argument gegen eine frühe Rückkehr läuft im Wesentlichen darauf hinaus, dass die Anzahl der möglichen Pfade durch den Code, die theoretisch nur einen Punkt enthält, wo eine Funktion austreten kann, reduziert wird, was zumindest theoretisch die Fehlerwahrscheinlichkeit verringert. Mein persönlicher Stil ist es, in Situationen, in denen es Sinn macht, früh zurückzukehren, und in Situationen, in denen es Sinn macht, auf eine Rückmeldung zu verzichten, dies zu tun.

Answer 3

Persönlich mag ich die zweite Methode besser. Es ist für mich einfacher und klarer, aber ich weiß, dass es Leute gibt, die nur eine einzige Rückkehr in einer Funktion haben müssen.

Answer 4

Wie bei den meisten Codierungsstilen ist es wirklich eine Frage der Präferenz, aber guard clauses werden von vielen als eine Best Practice angesehen.

Answer 5

Wenn Ausnahmen nicht Teil des Bildes sind, ziehe ich es vor, sofort zurückzukehren, wenn ich kann.

Es kann leicht sein, die Flag-Variable falsch zu verwalten und ich bin gegen Flag-Variablen im Allgemeinen. Wenn Sie nicht zurückkommen, könnte ein Betreuer denken, dass weitere Arbeit geleistet werden könnte (wenn die Methode lang ist).

Answer 6

Ehrlich gesagt denke ich, es hängt von der Situation ab. Persönlich benutze ich beide, und ich entscheide, basierend auf welcher wird den Code klarer und einfacher zu lesen.

Wenn Sie stark verschachtelte if-Anweisungen (oder irgendeine andere Steuerstruktur) und es kann verwirrend sein, dann würde ich in den Anweisungen geben

Sie nicht zu viel Sorgen darüber, was ‚best practice‘ in dieser Fall, da es wichtiger ist, dass der Code klar und einfach zu verstehen ist. Verwenden Sie, was sich für die Situation richtig anfühlt.

Answer 7

Ich ziehe es vor, früh zurückzukehren und tiefe Verschachtelung zu vermeiden. Dies ist insbesondere wahr gleich zu Beginn der Methode: Testen Sie alles, was einfach ist, und raus (oder werfen Sie eine Ausnahme), wenn Sie dies wirklich früh tun können.

Wenn es mitten in einer Methode ist, ist es eher ein Urteilsspruch.

Bitte beachte, dass ich Ihr Beispiel Refactoring würde sofort eine einzige verwenden if:

boolean validate(DomainObject o) {  
    if (o.property == x || o.property2 == y) { 
    return true; 
    } ... 
    return false; 
} 

Ich weiß, das nur ein Spielzeug Beispiel war, aber mein Punkt ist, dass es immer lohnt, für mehr Möglichkeiten suchen, zu vereinfachen Ihr Code :)

Answer 8

Für diesen Fall ziehe ich:

boolean validate (DomainObject o) { 
    if (o.property == x || 
     o.property2 == y || 
     ...) { 
      return true; 
    } else { 
      return false; 
} 

Im allgemeinen ich mag frühe Rückkehr verwenden, um Fehlerbedingungen zu behandeln und Rückkehr am Ende um berechnete Ergebnisse zurückzugeben.

Answer 9

Es gibt zwei Faktoren, die sich gegenseitig beeinflussen.

Der erste Faktor ist die einfache Fehlersuche. Wenn Sie sofort zurückkehren (wie in Ihrem zweiten Code-Snippet gezeigt), wird es manchmal schwierig, eine große Funktion zu debuggen, da es schwierig ist, diese Return-Anweisungen zu finden, besonders wenn sie versehentlich dort abgelegt wurden.

Der zweite Faktor ist die einfache Implementierung. Wenn Sie die grundlegende Korrektheit von Argumenten zu Beginn der Funktion überprüfen und es vor dem Ende der Funktion einen langen Codeabschnitt gibt, müssen Sie möglicherweise den gesamten Code in eine Bedingungsschleife einfügen. Wenn Sie das nicht tun, könnte das Argument irgendwann für eine lange Berechnung verwendet werden, was Zeitverschwendung wäre, weil es letztendlich sowieso abgelehnt würde.

So könnte die Antwort so aussehen:

If the function is small, 
     save the return status in a variable and return at the end. 
else 
     return immediately. 

Answer 10

Das einzige Mal, wenn ich Sie auf jeden Fall zurückkehren sollte nicht zu früh sagen kann, ist, wenn man nicht so leicht jede Rückkehr in einem einzigen Bildschirm sehen können (was auch immer das Standard könnte für Leute sein, die an der gleichen Code-Basis arbeiten), sollten Sie zumindest Kommentare hinzufügen, die anzeigen, dass die Funktion früh zurückkehren kann, wenn es eine vorzeitige Rückgabe gibt.

Das einzige Mal, dass ich sollte auf jeden Fall früh sagen würde zurückkehren, wenn Ihr Code wie folgt aussieht ...

boolean valid = true; 
if(condition1) { 
    valid = false; 
} 
if(valid) { 
    ... 
    if(condition2) { 
     valid = false; 
    } 
} 
if(valid) { 
    ... 
    if(condition3) { 
     valid = false; 
    } 
} 
... (etc) 

Wenn Sie sich in beiden Situationen finden, aber ... Sie wahrscheinlich Refactoring sein sollte die Funktion.