यदि अन्यथा संरचना

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 (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

आप एक्स, वाई, जेड, क्ष बारे में कुछ मान्यताओं बना सकते हैं? ईजी उनमें से सिर्फ एक सच हो सकता है। की तुलना में आप एक राज्य

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 पसंद नहीं है राज्य एक इंटरफ़ेस हो सकता है और क्यू एक्स क्यू कक्षाओं को कार्यान्वित किया जा सकता है। यदि इस मामले में निर्माण होता है तो इस मामले में लाभ एक ही उपयोग में होते हैं। कुछ codelines बाद में आप

if(x) 
    do_the_next_thing_with_X(); 
... 

साथ शुरू होगा कहें या आप बस एक और समारोह के साथ अपने enum का विस्तार करने और एक ही कॉल

state.doTheNextThing();