if-else structure

Question

У меня есть эти длинные заявления, которые я буду называть x, y и т. Д. Здесь. структура моих условных операторов выглядит следующим образом:

if(x || y || z || q){ 
    if(x) 
     do someth 
    else if (y) 
     do something 

    if(z) 
     do something 
    else if(q) 
     do something 
} 
else 
    do smthing 

Есть лучше, более короткий путь, чтобы написать эту вещь? Спасибо

Answer 1

Это кажется мне совершенно понятным (и ясно, что хорошо).

Что вы можете сделать, это сначала оценить x, y, z и q и сохранить их как переменные, поэтому вам не нужно делать это дважды.

Answer 2

Возможно, это немного легче читать. Но теперь вы выполните один дополнительный чек. Если это не критически критический код, то, возможно, вы можете использовать следующее:

if (x) 
    do something; 
else if (y) 
    do something; 

if (z) 
    do something; 
else if(q) 
    do something; 

if !(x || y || z || q) 
    do something completely different. 

Answer 3

Я не вижу большой проблемы с тем, как вы пишете сейчас. Я рекомендую использовать фигурные скобки даже для одиночных утверждений if-блоков. Это поможет вам избежать ошибок, если вам придется добавить дополнительные строки кода позже (и, возможно, забудьте добавить фигурные скобки). Я считаю его более читаемым. код будет выглядеть следующим образом, то:

if (x || y || z || q) { 
    if (x) { 
     do something 
    } else if (y) { 
     do something 
    } 

    if (z) { 
     do something 
    } else if (q) { 
     do something 
    } 
} else { 
    do something 
} 

Answer 4

Я не рекомендую следующее, на самом деле, я думаю, что вы получили это хорошо, но:

s = true; 
if (x) { 
    do something; 
    s = false; 
} else if (y) { 
    do something; 
    s = false; 
} 
if (z) { 
    do something; 
    s = false; 
} else if (q) { 
    do something; 
    s = false; 
} 

if (s) { 
    so something; 
} 

Answer 5

Другой вариант, который позволяет избежать многократного проверки и errorprone сложные логические выражения могут быть:

boolean conditionhandled = false; 
if (x) { 
    do something 
    conditionhandled = true; 
} else if (y) { 
    do something 
    conditionhandled = true; 
} 

if (z) { 
    do something 
    conditionhandled = true; 
} else if (q) { 
    do something 
    conditionhandled = true; 
} 

if (!conditionhandled) { 
    do something 
} 

Answer 6

вы можете сделать некоторые предположения относительно х, у, г, д? e.G. только один из них может быть правдой. Чем вы могли видеть его в качестве государства

enum State { 
X{ 
    void doSomething(){ 
    doItTheXWay(); 
    } 
}, 
Y{ 
    void doSomething(){ 
    doItTheYWay(); 
    } 
}, 
Z{ 
    void doSomething(){ 
    doItTheZWay(); 
    } 
}, 
Q{ 
    void doSomething(){ 
    doItTheQWay(); 
    } 
}; 
    void doSomething(){ 

    } 
} 

и в вашем коде, где вы использовали, если заявления

можно присвоить состояние и просто делать правильные вещи

State state = getAState(); 
state.doSomething(); 

В случае, если вы не нравится enums State может быть интерфейсом, а X to Q может реализовывать классы. Преимущества в этом случае в многократном использовании той же конструкции if else. Скажем, некоторые codelines позже вы бы начать с

if(x) 
    do_the_next_thing_with_X(); 
... 

или вы могли бы просто расширить перечисление с помощью другой функции и сделать один единственный звонок

state.doTheNextThing();