क्या यह स्पष्ट रूप से राज्य सीमाओं के लिए हानिकारक है यदि अन्यथा बयानों में?

Question

(मैंने "अन्यथा रेंज" की खोज की लेकिन मेरे प्रश्न का उत्तर देने वाली किसी भी पोस्ट के साथ नहीं आया)।

अगर .. elseif .. किसी और कुछ (परस्पर अनन्य) पर्वतमाला में एक चर जा रहा है के आधार पर क्या करना है का चयन करने के का उपयोग करते समय, मैं स्पष्ट रूप से पर्वतमाला का उल्लेख करना चाहते:

int num = 55; 
String message; 

if (num >= 20) { 
    // update message variable 
    message = "Bigger or equal to than 20"; 
} 
else if (10 <= num && num < 20) { 
    // update message variable 
} 
else if (0 <= num && num < 10) { 
    // update message variable 
} 
else if (num < 0) { 
    // update message variable 
} 

System.out.println(message); 

लेकिन सभी पाठ्यपुस्तकों और व्याख्यान नोट्स मैं देख रहा हूँ इस तरह इस तरह के एक उदाहरण के बारे में:

int num = 55; 
String message; 

if (num >= 20) { 
    // update message variable 
    message = "Bigger or equal to than 20"; 
} 
else if (num >= 10) { 
    // update message variable 
} 
else if (num >= 0) { 
    // update message variable 
} 
else { 
    // update message variable 
} 

System.out.println(message); 

मुझे समझ में क्यों वे अंत में एक और को उपयोग सुनिश्चित करें (यानी यह सोच कर कि संदेश की तरह एक चर प्रारंभ नहीं हो सकता से जावा कम्पाइलर को रोकने के लिए, यदि वह चर एक आदिम था), लेकिन यह सब दिया गया पाठ्यपुस्तक और व्याख्यान नोट्स अन्य शैली दिखाते हैं, क्या मुझे अन्य सभी स्थितियों के लिए सीमाओं को स्पष्ट रूप से लिखने के कारण कोई समस्या है, जैसा कि मुझे करना है?

Answer 1

यदि मैं कुछ कोड बना रहा हूं जो कि किसी अन्य वाहन पर सफलतापूर्वक उतरने के लिए किसी प्रकार का वाहन चलाएगा तो मैं हमेशा स्पष्टता के लिए मोटा होगा। याद रखें कि कोड की एक पंक्ति एक बार लिखा जाएगा लेकिन सैकड़ों बार पढ़ा जा सकता है। आपका पहला स्निपेट 4am डिबगिंग सत्र के दौरान समझना बहुत आसान होगा।

आपका पहला तरीका वास्तव में बहुत स्पष्ट है और अगर कोई गलती रिफैक्टरोर सशर्त जांच को रोकता है तो बग पेश नहीं किए जाएंगे।

मैं हमेशा } else { के साथ समाप्त होता हूं, हालांकि आदर्श नियंत्रण किसी भी प्रकार के दावे के साथ आदर्श रूप से नहीं पहुंचता है।

Answer 2

अपने आप को दोहराना न करें।

अपने दृष्टिकोण के साथ, आप कोड में दो बार एक ही मूल्य दोहरा रहे हैं, अनावश्यक रूप से। इसमें कई नुकसान हैं, जिनमें शामिल हैं:

• यह मूल्यों में से किसी एक को बदलकर रखरखाव के दौरान त्रुटियों को पेश करना आसान बनाता है, लेकिन दूसरा नहीं।
• निष्पादन के दौरान यह एक अनावश्यक जांच करता है। यदि आप else if में हैं, तो आप पहले ही जानते हैं कि पूर्व शर्त गलत थी।
• कोड में कोई अतिरिक्त जानकारी जोड़ने के बिना लंबी है। यह समय लेता है पढ़ने के लिए, और लंबे समय तक निरीक्षण करने के लिए।

बाद में रखरखाव मूल्यों में से एक को बदल सकता है, लेकिन दूसरे नहीं, जिससे कुछ आश्चर्यजनक व्यवहार हो सकता है।

if (num >= 30) { 
    // update message variable 
} 
else if (10 <= num && num < 20) { // Whoops! 
    // update message variable 
} 
... 

आप प्रत्येक मूल्य के लिए निरंतर परिभाषित करके इसे टाल सकते हैं। लेकिन फिर आपका कोड अभी भी एक अनावश्यक जांच निर्दिष्ट कर रहा है।

final int HIGH = 20; 
final int MEDIUM = 10; 
final int LOW = 0; 

if (num >= HIGH) { 
    // update message variable 
} 
else if (MEDIUM <= num && num < HIGH) { // We already know num<HIGH 
    // update message variable 
} 
...