Vorzeitige Rückkehr von einem Scala-Konstruktor

Question

Ich schreibe den Konstruktor für meine "Haupt" -Klasse. Das erste, was es tut, ist eine Methode aufzurufen, Commons-CLI zu verwenden, um die Befehlszeile zu analysieren. Wenn die Methode parseOptions false zurückgibt, ist ein Fehler aufgetreten und der Konstruktor sollte beendet werden.

versuchte ich

if (!parseOptions(args)) return 

den folgenden Code schreiben, aber der Compiler beschwert sich, dass ich eine „Return-Anweisung außerhalb Methodendefinition“ haben.

Kurz von System.exit(1) oder Umkehren der boolean Aufruf (und der ganze Rest meiner Logik innerhalb der if Anweisung setzen, ist es eine Möglichkeit, zurückzukehren „früh“ von einem Konstruktor?

Ich glaube, ich das haben könnte parseOptions Methode ein IllegalArgumentException werfen und dieses Objekt in meinem Main fangen.

Dank.

Answer 1

ist es eine Möglichkeit, zurückzukehren "früh" von einem Konstruktor

Nein, aber in Ihrem Fall klingt es wie schlechtes Design, sowieso.

Wenn die ParseOptions Methode false zurückgibt, ist ein Fehler

In diesem Fall tritt der Konstruktor sollte eine Ausnahme auslösen, die normalerweise nicht zurück.

Answer 2

Dont versuchen, eine frühe/vorzeitige Rückkehr zu tun, das macht den Code schwerer komplexe, da die Nebenwirkungen der Rückkehr sein können h ard zu verstehen. Verwenden Sie stattdessen eine Ausnahme, um anzuzeigen, dass etwas nicht stimmt.

Sie können require im Konstruktor verwenden. Dies kommt nicht zurück. Aber es scheint, als ob das Werfen einer Ausnahme tatsächlich besser zu seiner Situation passt.

Wie in:

class MyTest(
private var myValue: Int){ 

    require(myValue > 0) // Connected to constructor 

} 

defined class MyTest 

scala> val x = new MyTest(10) 
x: MyTest = MyTest@49ff4282 

scala> val y = new MyTest(-10) 
java.lang.IllegalArgumentException: requirement failed 
     at scala.Predef$.require(Predef.scala:133) 

Answer 3

Ein Konstruktor sollte immer entweder vollständig vervollständigen oder abbrechen (eine Ausnahme auslösen). Alles andere lässt dein Objekt "halb konstruiert" und somit unmöglich zu verstehen.

Wenn in Ihrem Fall das Objekt gültig ist, selbst wenn ParseOptions ausgefallen ist, dann können Sie den Zustand ändern und weiter:

if (parseOptions(args)) { 
    // rest of constructor 
}