if-else 구조

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

x, y, z, q에 대해 몇 가지 가정을 할 수 있습니까? 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(); 

을 할 경우 열거 형을 좋아하지 않음 State는 Interface가 될 수 있고 X to Q는 클래스를 구현할 수 있습니다. 이 경우 이점은 동일한 if else 구문을 여러 번 사용하는 것입니다. 일부 codelines 나중에는

if(x) 
    do_the_next_thing_with_X(); 
... 

로 시작하는 것이 말하거나 또 다른 기능으로 열거를 확장하고 단락의 목적을 물리 칠 것이다, 하나의 호출

state.doTheNextThing();