Riscrivere un'istruzione condizionale in Java

Question

Dire se ho il codice seguente, è fondamentale determinare che una condizione sia abbinata e quindi assegnare il valore booleano, quindi eseguire alcuni codici. Quindi lanciare un'eccezione se il valore booleano è falso. Cosa succede se voglio che venga lanciata un'eccezione immediatamente se il valore booleano è falso senza eseguire il resto dei codici? Se metto la seconda istruzione condizionale nella prima, ci saranno dei codici duplicati. Per favore mostrami un modo intelligente per farlo (ho modificato il codice in modo che assomigli ai miei codici reali).

boolean booleanValue = false; 
Permission value; 

if (someCondition) { 
    value = getPermission_1(); 
    booleanValue = someMethod(value); 
    useValue_1(value); 
} 
else { 
    value = getPermission_2(); 
    booleanValue = anotherMethod(value); 

    useValue_2(value); 
} 

if (!booleanValue) { 
    throw Exception(); 
} 

Answer 1

Come eliminare la variabile booleana? Si potrebbe riscrivere il codice come questo:

if (someCondition) { 
    if (!someMethod()) { 
    throw new Exception(); 
    } 
    some codes... 
} 
else { 
    if (!anotherMethod()) { 
    throw new Exception(); 
    } 
    some codes... 
} 

che sembra più facile ai miei occhi, ma queste cose sono una questione di gusto ...

vantaggio in più: Se l'eccezione finisce in una pila-trace sai qual è stata la condizione perché hai due diverse dichiarazioni di lancio. Questo potrebbe velocizzare un po 'il debug.

Answer 2

Invece di

booleanValue = anotherMethod(); 

si dovrebbe semplicemente scrivere

if(!someMethod()) 
    throw new SomeException(); 

Su gettando un Eccezione generica - non lo fanno. Il motivo per cui stai facendo un'eccezione è dire al chiamante che si è verificato qualcosa di eccezionale. È quasi sempre utile dir loro cosa, altrimenti né il chiamante né tu puoi fare qualcosa a riguardo.

Answer 3

Supponendo che i due some codes... sono diverse, probabilmente vuole fare:

boolean booleanValue = someCondition ? someMethod() : anotherMethod(); 
if(!booleanValue) { 
    throw new Exception(); 
} 

if(someCondition) { 
    // some code 
} else { 
    // some code 
} 

Se sono le stesse della if(someCondition) non è necessario

---

Se (ipoteticamente) si dispone di un strumento di analisi statica che non consente le espressioni ternarie, è possibile sostituire le prime righe con:

boolean booleanValue; 
if(someCondition) { 
    booleanValue = someMethod(); 
} else { 
    booleanValue = anotherMethod(); 
} 

Answer 4

La soluzione più ovvia è:

boolean booleanValue = false; 

if (someCondition) { 
    booleanValue = someMethod(); 
    if(booleanValue){ 
     //some codes... 
    } 
} 
else { 
    booleanValue = anotherMethod(); 
    some codes... 
} 

if (!booleanValue) { 
    throw Exception(); 
} 

... ma non mi dispiace ripetere il bit if(!booleanValue) throw Exception();, perché è probabile che un motivo concettualmente diverso che si sta gettando l'eccezione. (Ad esempio, potresti fornire un messaggio di errore migliore nella tua eccezione)

Answer 5

È tutto molto soggettivo, forse?

boolean booleanValue = aBoolean; 
if (someCondition) { 
    if (!someMethod()) { 
     throw new SomeException(); 
    } 
    some codes... 
} else { 
    if (!anotherMethod()) { 
    throw new AnotherException(); 
    } 
    some other codes... 
} 

Answer 6

La soluzione migliore sarebbe ...

if (someCondition) { 
    value = getPermission_1(); 

    if (!someMethod(value)) { 
    throw new SomeException(); 
    } 

    useValue_1(value); 
} 
else { 
    value = getPermission_2(); 

    if (!anotherMethod(value)) { 
    throw new AnotherException(); 
    } 

    useValue_2(value); 
} 

e non si deve guardare su di essa come la duplicazione di codice, perché se si vuole lanciare un'eccezione quindi l'aspettativa è che la ragione per la l'eccezione sarebbe diversa in ogni caso e quindi un'eccezione diversa o un messaggio diverso dovrebbe essere passato in ogni caso.

Presumo che tu voglia sapere quale condizione è stata eseguita e successivamente fallita, perché tutto quello che stai tornando è un booleano dal tuo ... Il metodo chiama il motivo dell'errore probabilmente non sarà evidente in questo scenario.

Answer 7

Questo tipo odora ... come in "codice odore". Un valore di ritorno viene tradotto in un'eccezione. Sembra che se il chiamante ha scritto someMethod e anotherMethod, allora la soluzione è di riscrivere quei metodi e lanciare l'eccezione da quei metodi invece di usare un valore di ritorno. Ma questo è solo se il programmatore ha accesso al codice. Se si tratta di una chiamata API di terze parti, suppongo che la traduzione potrebbe dover avvenire.