Sollten Methoden mit dem Rückgabetyp void eine return-Anweisung verwenden?

Question

Ich weiß, dass es Zeiten gibt, wenn return; verwenden, können einen nützlichen Zweck in Java, wie in Bewachungs dienen:

public void foo(Bar bar) { 
    if(bar == null) 
     return; 

    // bar is not null, go ahead and do stuff with it 
} 

Aber was ist nur das Ende einer Methode mit Rückgabetyp erreicht void? Zum Beispiel

public void printMenu() { 
    System.out.println("Print out some boilerplate info here, line 1."); 
    System.out.println("Print out some boilerplate info here, line 2."); 
    System.out.println("Print out some boilerplate info here, line 3."); 

    return; 
} 

Anders als reine Stil Vorlieben, gibt es Gründe für oder gegen einschließlich return;? Wenn ja, was sind sie?

EDIT: Nun, das wurde schnell beantwortet. Um die 15 Antworten zusammenzufassen: "Nein."

Answer 1

Vielleicht sind Sie nach Codezeile bezahlt?

Ansonsten gibt es wirklich keinen Grund, am Ende eine leere Rückkehr zu setzen.

Answer 2

In Ihrem Beispiel ist die "Rückkehr" am Ende eine Frage des persönlichen, Team- oder Organisationsstils. Ich persönlich bevorzuge eine explizite Rückkehr.

Answer 3

Rein stilbasierte Frage, macht absolut keinen Unterschied (vielleicht eine zusätzliche ASM-Anweisung, aber wen interessiert das?). Tun Sie, was immer Sie sich wohler fühlen, oder befolgen Sie die Konvention, die zuvor im Code festgelegt wurde.

Answer 4

Ich vermeide sie, mich selbst. Es ist nur eine nutzlose Codezeile. In der Tat hat PMD eine Regel, die nach solchen unnützen return Aussagen überprüft.

Answer 5

Ich sehe keinen Grund, auch aus einer Stilperspektive, am Ende eine dangling Rückkehr zu haben. Schließlich wissen Sie, dass es zurückkehren wird, weil es dort eine Endklammer gibt ...

Answer 6

Ich denke, anders als persönliche Vorliebe, gibt es keinen Unterschied. Der erste Fall, ich denke, es ist legitim. Andernfalls sollten Sie die Anweisungen in eine große if-Anweisung packen.

Das letzte Beispiel die Rückkehr ist überflüssig.

Answer 7

Ich sage, mache das nie. Return-Anweisungen in void Funktionen dienen nur dazu, die Logik der Anweisung zu brechen. Wenn Sie dies tun, dann senden Sie eine verwirrende Aussage an die Leser Ihres Codes, wird man versucht zu denken, dass Sie vielleicht einige wenn Aussage, dass Sie vergessen haben. Immer auf Lesbarkeit achten.

Answer 8

Ich würde es nur von einem Konsistenzpunkt vermeiden. Es ist einfacher, es nie zu sagen, als sich daran zu erinnern, es immer hinzuzufügen.

Answer 9

Es dient keinem Zweck, aber wenn Sie es tun möchten (weil jeder in Ihrem Team es tut oder aus irgendeinem Grund), können Sie es tun.

Answer 10

Eine Idee von structured programming war:

Jede Routine genau einen Einstiegspunkt haben sollte und genau einen Ausgangspunkt.

Wenn Sie diese Richtlinie abonnieren, gibt die return-Anweisung die einzige Möglichkeit an, die Routine zu beenden.

In der Praxis ist, dass die Politik nicht Code klarer machen, und es seit den 1970er Jahren meist ignoriert. Wenn Sie mehrere Rückgabeanweisungen in anderen Routinen zulassen, sollten Sie Null-Return-Anweisungen zulassen, wenn dies am sinnvollsten ist.

Answer 11

Ich denke, dass unnötige Aussagen sind nur Lärm, also würde ich nicht, dass die Rückkehr am Ende hinzufügen. Davon abgesehen würde ich am Anfang der Methode Returns hinzufügen, wenn etwas nicht meinen Anforderungen entspricht oder, noch besser, ich würde IllegalArgumentException-Ausnahmen auslösen.