Eine bedingte Anweisung in Java umschreiben

Question

Sagen Sie, wenn ich den Code unten habe, bestimmt es im Grunde, dass eine Bedingung übereinstimmt, und weisen Sie dann den booleschen Wert zu, dann führen Sie einige Codes aus. Dann wird eine Ausnahme ausgelöst, wenn der Wert booleanValue false ist. Was passiert, wenn ich möchte, dass eine Ausnahme sofort ausgelöst wird, wenn der Wert booleanValue false ist, ohne dass die restlichen Codes ausgeführt werden? Wenn ich nur die zweite bedingte Anweisung in die erste gebe, wird es doppelte Codes geben. Bitte zeigen Sie mir eine kluge Möglichkeit, dies zu tun (ich habe den Code so verändert, dass er wie meine tatsächlichen Codes aussieht).

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

Wie wäre es mit der Beseitigung der booleschen Variable? Sie könnten Ihren Code wie folgt umschreiben:

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

, die leichter in die Augen sieht, aber solche Dinge sind Geschmackssache ...

Extra-Vorteil: Wenn die Ausnahme endet in einem Stapel-Trace-up Sie wissen, was die Bedingung war, weil Sie zwei verschiedene throw-Anweisungen haben. Das kann das Debugging ein wenig beschleunigen.

Answer 2

Statt

booleanValue = anotherMethod(); 

Sie einfach

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

Auf wirft eine allgemeine Ausnahme schreiben würde - nicht. Der Grund dafür, dass Sie eine Ausnahme auslösen, besteht darin, dem Anrufer mitzuteilen, dass etwas Außergewöhnliches aufgetreten ist. Es ist fast immer nützlich, ihnen zu sagen, was sonst, weder der Anrufer noch Sie können etwas dagegen tun.

Answer 3

Unter der Annahme, die beiden some codes... unterschiedlich sind, werden Sie wahrscheinlich tun wollen:

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

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

Wenn sie die gleiche die if(someCondition) sind nicht erforderlich

---

Wenn (hypothetisch) Sie haben statisches Analysewerkzeug, das keine ternären Ausdrücke zulässt, können Sie die ersten Zeilen durch folgende ersetzen:

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

Answer 4

Die offensichtliche Lösung ist:

boolean booleanValue = false; 

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

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

... aber ich habe nichts dagegen das if(!booleanValue) throw Exception(); Bit wiederholen, weil es wahrscheinlich ein konzeptionell anderen Grund, dass Sie die Ausnahme sind zu werfen. (Sie könnten z. B. in Ihrer Ausnahmebedingung eine bessere Fehlermeldung angeben.)

Answer 5

Es ist alles sehr subjektiv, vielleicht?

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

Answer 6

Ihre beste Lösung ... wäre

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); 
} 

Und Sie sollten schauen auf sie nicht als Code zu duplizieren, weil, wenn Sie eine Ausnahme werfen wollen, dann würde die Erwartung, dass der Grund für die Ausnahme wäre in jedem Fall anders und daher sollte jeweils eine andere Ausnahme oder eine andere Nachricht übergeben werden.

Ich nehme an, Sie möchten wissen, welche Bedingung ausgeführt wurde und anschließend fehlgeschlagen, weil alles, was Sie zurück bekommen, ist ein Boolean von Ihrem ... Methode nennt den Grund für den Fehler wird wahrscheinlich nicht in diesem offensichtlich sein Szenario.

Answer 7

Das riecht irgendwie ... wie in "Code-Geruch". Ein Rückgabewert wird in eine Ausnahme übersetzt. Es scheint, dass, wenn der Aufrufer someMethod und anotherMethod geschrieben hat, die Lösung darin besteht, diese Methoden neu zu schreiben und die Ausnahme von diesen Methoden zu werfen, anstatt einen Rückgabewert zu verwenden. Aber das ist nur, wenn der Programmierer Zugriff auf den Code hat. Wenn es sich um einen API-Aufruf eines Drittanbieters handelt, muss die Übersetzung möglicherweise stattfinden.