प्रारंभिक रिटर्न बनाम सकारात्मक नेस्ट अगर बयान

Question

यहाँ कुछ काल्पनिक कोड नमूना है:

if (e.KeyCode == Keys.Enter) 
{ 
    if (this.CurrentElement == null) { 
     return false;} 

    if (this.CurrentElement == this.MasterElement) { 
     return false;} 

    if (!Validator.Exist (this.CurrentElement)) { 
     return false;} 

    if (!Identifier.IsPictureElement (this.CurrentElement)) { 
     return false;} 

    this.FlattenObjects(this.CurrentElement); 
} 

वी.एस.

if (e.KeyCode == Keys.Enter) 
{ 
    if (this.CurrentElement != null) { 

     if (this.CurrentElement != this.MasterElement) { 

      if (Validator.Exist (this.CurrentElement)) { 

       if (Identifier.IsPictureElement (this.CurrentElement)) { 

        this.FlattenObjects(this.CurrentElement);}}}}}} 

} 

कौन सा आपको लगता है पठनीयता, रखरखाव, आदि के मामले में बेहतर है?

इसके अलावा दूसरा उदाहरण कोष्ठक के विभिन्न उपयोग के माध्यम से अलग-अलग स्वरूपित किया जा सकता है।

Answer 1

प्रारंभिक रिटर्न अधिक पढ़ने योग्य हैं।

जब भी आपको किसी विधि के अंदर घोंसले के चार या पांच से अधिक स्तर मिलते हैं, तो उस विधि को दोबारा करने का समय आता है।

एक एकल if एक || खंड के साथ कभी कभी अधिक पठनीय हो सकता है:

if (this.CurrentElement == null 
|| this.CurrentElement == this.MasterElement 
|| !Validator.Exist(this.CurrentElement) 
|| !Identifier.IsPictureElement(this.CurrentElement)) 
    return false; 

Answer 2

मुझे लगता है कि मुझे यह पसंद है लिखना होगा:

if (this.CurrentElement == null OR this.CurrentElement == this.MasterElement OR ...) return false; 

Answer 3

पहला उदाहरण हर तरह से बेहतर है। यह आसान है, और पढ़ने के लिए आसान है। कुछ लोग कहते हैं कि प्रत्येक समारोह में एक ही वापसी बिंदु होना चाहिए; यह उदाहरण स्पष्ट रूप से दिखाता है कि वे लोग गलत क्यों हैं। आदि

if (this.CurrentElement == null) return false; 

यह और भी आसान है, और भी पढ़ने के लिए आसान बनाता है:

पुनश्च व्यक्तिगत तौर पर मैं उन सभी ज़रूरत से ज़्यादा कर्ली कोष्ठक को दूर करेंगे।

Answer 4

मैं कहूंगा कि पहले पठनीयता और रखरखाव के लिए बेहतर है। हालांकि, मैं शायद इसे इस तरह कुछ लिखूंगा।

if (e.KeyCode == Keys.Enter) { 
    if(this.CurrentElement == null || 
     this.CurrentElement == this.MasterElement || 
     !validator.exists(this.CurrentElement) || 
     !identifier.isPictureElement(this.CurrentElement)) 
    { 
     return false; 
    { 
    else 
    { 
     this.flattenObjects(this.CurrentElement); 
    } 
} 

Answer 5

यह देखते हुए कि दूसरे उदाहरण में "गलत" सभी रास्ते के लिए बदले है लेकिन यह अंतर्निहित बजाय घोषित क्यों बस सभी रिटर्न परोक्ष झूठी नहीं और केवल एक शर्त यह है कि अद्वितीय है के लिए परीक्षण है?

यह किसी के स्टाइल दिशानिर्देशों का उल्लंघन कर सकता है लेकिन तर्कसंगत रूप से सबसे संक्षिप्त है।

if(e.KeyCode == Keys.Enter 
&& this.CurrentElement != null 
&& this.CurrentElement != this.MasterElement 
&& Validator.Exist (this.CurrentElement)      
&& Identifier.IsPictureElement (this.CurrentElement)) 
    this.FlattenObjects(this.CurrentElement);