Ist das eine geniale Sache oder einfach nur schlechter Code?

Question

Dies ist ein Code-Snippet, das ich in einem Open-Source-Java-Template-Projekt gefunden habe.

Hat jemand eine Ahnung, wofür dieses Konstrukt gut sein kann?

final public Expression Expression() throws ParseException { 
    Expression exp; 
    exp = OrExpression(); 
     {if (true) return exp;} 
    throw new Error("Missing return statement in function"); 
    } 

Answer 1

Dies ist schlampig Code. Der tatsächliche Effekt ist einfach return OrExpression();.

Es gibt eine Ausrede, die ich für Code wie folgt akzeptiere: Es gibt viele andere Methoden wie es, die einem ähnlichen Muster folgen, aber komplexer sind, so dass die nutzlosen Bits hier nicht nutzlos sind. Dies ist jedoch ein extremes Beispiel.

Jedoch gibt es fast keine Entschuldigung für die throw new Error("Missing return statement in function");. Der Java-Compiler verweigern kompilieren eine Methode, deren Rückgabetyp nicht void ist und die einen Code-Pfad hat, der das Ende erreicht (nicht return, throw, oder geben Sie eine Endlosschleife). Der einzige Grund, dies wäre eine gute Idee, wenn die Absicht war, unvollständigen Code zu kompilieren - dh zur Laufzeit statt zur Kompilierzeit fehlschlagen, so dass andere Teile des Codes verwendet oder getestet werden können - aber selbst dann anwenden zu allen Methoden ist eine schlechte Idee.

Answer 2

Dies ist ein automatisch generierter Code von JavaCC.