Ritorno anticipato da un costruttore di Scala

Question

Sto scrivendo il costruttore per la mia classe "principale". La prima cosa che fa è chiamare un metodo per usare commons-cli per analizzare la riga di comando. Se il metodo parseOptions restituisce false, si è verificato un errore e il costruttore deve uscire.

ho provato a scrivere il seguente codice

if (!parseOptions(args)) return 

ma il compilatore si lamenta che ho una "definizione di un metodo di fuori dichiarazione di ritorno".

corto di chiamare System.exit(1) o invertendo la booleana (e mettendo tutto il resto della mia logica all'interno della dichiarazione if, c'è qualche modo per tornare "presto" da un costruttore?

Suppongo che avrei potuto avere il parseOptions metodo di gettare un IllegalArgumentException e cattura che nel mio Main oggetto.

Grazie.

Answer 1

Esiste un modo per tornare "presto" da un costruttore

No. Ma nel tuo caso suona come una cattiva progettazione, in ogni caso.

Se il metodo parseOptions restituisce false, si è verificato un errore

In questo caso il costruttore dovrebbe lanciare un'eccezione, non restituisce normalmente.

Answer 2

Non provare a fare un ritorno/prematuro presto, questo rende il vostro codice più difficile più complessa, dal momento che gli effetti collaterali di ritorno possono essere h ardire per capire. Usa invece un'eccezione per segnalare che qualcosa non va.

È possibile utilizzare require nel costruttore. Questo non torna. Ma sembra che lanciare un'eccezione si adatti meglio alla sua situazione.

come 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

Un costruttore deve sempre completare completamente o interrompere (lanciare un'eccezione). Qualsiasi altra cosa lascia il tuo oggetto "a metà costruito" e quindi impossibile ragionare.

Se nel tuo caso, l'oggetto è valida anche se parseOptions fallito, allora si può cambiare la condizione e continua:

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