सबसे अच्छा स्थिर कोड विश्लेषक है इसका मतलब है इस सटीक कारण के लिए इंडेंटेशन का। घोंसले के ऐसे उच्च स्तरों के साथ सभी तार्किक मामलों को संभालना बहुत मुश्किल हो जाता है।
क्या यह फ़ंक्शन के शीर्ष पर एक बड़ी गांठ में सभी त्रुटि स्थितियों की जांच करने की सामान्य नौसिखिया त्रुटि है?
यदि ऐसा है, तो हो सकता है कि आप कोड के लेखक को इसे भारी घोंसले के निर्माण के बजाए यदि बयानों के अनुक्रम में बदलना चाहें।
if(error1) {
/* report error 1 and exit */
}
if(error2) {
/* report error 2 and exit */
}
if(error3) {
/* report error 3 and exit */
}
...
यह बहुत आसान है एक विशिष्ट त्रुटि के बजाय एक सामान्य "कुछ का बुरा" बयान के बारे में जानकारी प्रदान करने के अनुरूप कोड का परीक्षण और यह भी बनाता है।
स्रोत
2010-01-25 09:51:54
मैं कैसे इस कोड refactor करने में दिलचस्पी है ... –
मैं क्यों नहीं देख पा रहे हैं यह बुरा है, यह – harryovers
पर जिस समस्या पर काम कर रहा है उसे हल करने का सबसे अच्छा तरीका हो सकता है, इससे कोई फर्क नहीं पड़ता कि आप अपने 'अगर' की व्यवस्था कैसे करते हैं। – Kobi