cấu trúc if-else

Question

Tôi có các câu lệnh dài mà tôi sẽ gọi là x, y, v.v. ở đây. cấu trúc câu điều kiện của tôi đi như thế nà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 

Có một cách ngắn hơn để viết điều này? Cảm ơn

Answer 1

Điều này có vẻ khá rõ ràng đối với tôi (và rõ ràng là tốt).

Điều bạn có thể làm trước tiên là đánh giá x, y, z và q và lưu trữ chúng dưới dạng biến để bạn không phải thực hiện việc đó hai lần.

Answer 2

Có thể điều này dễ đọc hơn một chút. Nhưng bây giờ bạn sẽ thực hiện một kiểm tra thêm. Nếu đó không phải là mã quan trọng của sứ mệnh thì có thể bạn có thể sử dụng như sau:

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

Tôi không thấy vấn đề lớn với cách bạn viết ngay bây giờ. Tôi khuyên bạn nên sử dụng dấu ngoặc nhọn ngay cả đối với câu lệnh if-blocks. Điều này sẽ giúp bạn tránh được những sai lầm trong trường hợp bạn phải thêm nhiều dòng mã sau đó (và có thể quên thêm các dấu ngoặc nhọn sau đó). Tôi cũng thấy nó dễ đọc hơn. Mã này sẽ trông như thế này thì:

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

Tôi không giới thiệu sau đây, trên thực tế, tôi nghĩ rằng những gì bạn nhận được là tốt, nhưng:

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

Một biến thể mà tránh được nhiều kiểm tra và biểu thức logic phức tạp errorprone có thể là:

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

Bạn có thể đưa ra một số giả định về x, y, z, q không? e.G. chỉ một trong số họ có thể đúng. Hơn bạn có thể xem nó như là một nhà nước

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

    } 
} 

và trong mã của bạn, nơi bạn sử dụng câu lệnh if

bạn có thể gán một nhà nước và chỉ cần làm điều đúng

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

Trong trường hợp bạn không thích enums Nhà nước có thể là một giao diện và X để Q có thể được thực hiện các lớp học. Những lợi ích trong trường hợp này là trong nhiều cách sử dụng giống nhau nếu người khác xây dựng. Giả sử một số codelines sau đó bạn sẽ bắt đầu với

if(x) 
    do_the_next_thing_with_X(); 
... 

hoặc bạn chỉ có thể mở rộng enum của bạn với chức năng khác và làm cho một cuộc gọi duy nhất

state.doTheNextThing();