स्पष्टता के लिए समारोह से बूल के बजाय वापसी enum?

Question

इस के समान है:

.NET: bool vs enum as a method parameter

लेकिन कुछ स्थितियों में एक समारोह से एक bool लौटने चिंताओं।

उदा। समारोह जो bool रिटर्न:

public bool Poll() 
    { 
     bool isFinished = false; 

     // do something, then determine if finished or not. 

     return isFinished; 
    } 

इस तरह उपयोग किया:

 while (!Poll()) 
     { 
      // do stuff during wait. 
     } 

इसके बुला संदर्भ क्या bool पोल से लौटे से स्पष्ट नहीं() का अर्थ है। कुछ तरीकों से यह स्पष्ट हो सकता है कि "पोल" फ़ंक्शन का नाम बदलकर "IsFinished()" रखा गया था, लेकिन विधि थोड़ा सा काम करती है, और (आईएमओ) वास्तव में यह नहीं दर्शाती कि वास्तव में कार्य क्या करता है। "Isfinished" जैसे नाम गुणों के लिए भी अधिक उपयुक्त लगते हैं। एक और विकल्प इसका नाम बदल सकता है जैसे: "PollAndReturnIsFinished" लेकिन यह सही नहीं लगता है।

तो एक विकल्प एक enum वापस करने के लिए हो सकता है। उदाहरण के लिए:

public enum Status 
    { 
     Running, 
     Finished 
    } 

    public Status Poll() 
    { 
     Status status = Status.Running; 

     // do something, then determine if finished or not. 

     return status; 
    } 

इस तरह कहा जाता है:

 while (Poll() == Status.Running) 
     { 
      // do stuff during wait. 
     } 

लेकिन यह overkill की तरह लगता है। कोई विचार?

Answer 1

एक विधि एक क्रिया की तरह पढ़ किया जाना चाहिए, और bool Poll() विधि का परिणाम भ्रामक है, और यह शायद यही कारण है कि इसका इस्तेमाल करने की अजीब लगता है।

// you wrote. 
while(!Poll()) 
{ 
    // still waiting .. do something. 
} 

जब मैं पहली बार अपने कोड को पढ़ने के लिए, मैंने सोचा कि यह कहा जबकि (सिस्टम है) नहीं मतदान, कुछ करते हैं?

लेकिन यह वास्तव में कहता है ... मतदान, और यदि मतदान समाप्त नहीं होता है तो हम प्रतीक्षा करते समय कुछ करते हैं।

आपका enum संस्करण कॉल के अर्थशास्त्र को बदलता प्रतीत होता है, लेकिन बेहतर के लिए, यही कारण है कि लोग इसे पसंद करते हैं। जबकि मतदान() अभी भी चल रहा है, हम प्रतीक्षा करते समय कुछ करें।

सबसे अधिक पढ़ने योग्य कोड जीतता है।

Answer 2

यदि आपके पास 2 से अधिक राज्य हैं, तो enum का उपयोग करें, अन्यथा बस bool का उपयोग करें।

संपादित करें:

अपने उदाहरण के रूप में, आप आसानी से दोनों के उपयोग, यदि आवश्यक हो तो कर सकते हैं।

public bool IsRunning { get {return Poll() == Running; }} 
public bool IsFinished { get {return Poll() == Finished; }} 

Answer 3

सभी कोडों में से पहला कोड पढ़ने के लिए है, और आपके मामले में enum संस्करण बूल संस्करण से अधिक पठनीय है।

संपादित करें: आप आसानी से अन्य स्थितियां जोड़ सकते हैं अगर आप की जरूरत है

enum संस्करण के अन्य लाभ है। उदाहरण के लिए Error की तरह।

Answer 4

मैं .Net सम्मेलन का पालन करता हूं कि बूलियन गुणों को "है" के साथ उपसर्ग किया गया है और बूलियन विधियों को "कोशिश करें" (या "है" जहां उचित हो) के साथ उपसर्ग किया गया है।

आपके मामले में मुझे लगता है कि समस्या "पोल" नाम में है। यह बताते हुए विधि का नाम दें कि यह क्या कर रहा है या मतदान कर रहा है। जैसे TryDoSomething()

Answer 5

मैं जो कुछ करने की कोशिश कर रहा हूं उसे पढ़ और पुनः पढ़ता हूं। मेरे लिए, पोल() का मतलब यह होना चाहिए कि आप किसी चीज़ के लिए मतदान कर रहे हैं। फिर यदि स्थिति अभी भी चल रही है तो मैं स्थिति की जांच करूंगा।

मेरा परिणाम कोड इस तरह दिखेगा।

while (myObject.Poll() && myObject.IsRunning) 
{ 
    // myObject successfully poll it .. 
    // and is successful 
    // do more things here... 
} 

नोट: IsRunning एक गेटर/सेटर है। क्या यह स्पष्ट होगा?