विस्तार विधि में शून्य को संभालना

Question

मेरे पास स्ट्रिंग क्लास के लिए एक सरल विस्तार विधि है जो स्ट्रिंग से सभी गैर संख्यात्मक वर्णों को पट्टी कर देगी। तो अगर मेरे पास एक स्ट्रिंग है उदाहरण के लिए एक फोन नंबर जैसे "(555) 215-4444" यह इसे "5552154444" में परिवर्तित कर देगा। ऐसा लगता है:

public static string ToDigitsOnly(this string input) 
{ 
    Regex digitsOnly = new Regex(@"[^\d]"); 
    return digitsOnly.Replace(input, String.Empty); 
} 

मैं सोच रहा हूं कि यहां शून्य मूल्य को संभालने का सबसे शानदार तरीका क्या है? क्या इन मामलों में पालन करने के लिए एक सामान्य पैटर्न है, जैसे कि शून्य न हो जाने पर शून्य मूल्य वापस लौटाएं? ऐसा लगता है क्योंकि मैं स्ट्रिंग क्लास को विस्तारित कर रहा हूं, इसलिए मैं शून्य मानों को अनुमति देना चाहता हूं और एक बहस अपवाद नहीं फेंक सकता हूं (क्योंकि जब मैं इसका उपयोग करता हूं तो मैं वास्तव में बहस में गुजर रहा हूं ...)? लेकिन कुछ तर्क दे सकते हैं कि मुझे 'सामान्य' विधि की तरह अपवाद फेंकना चाहिए। आप यहां क्या सबसे अच्छा अभ्यास कर रहे हैं?

धन्यवाद!

Answer 1

आप कम से कम आश्चर्य की बात है के सिद्धांत का पालन कर सकते हैं: उपयोग पैटर्न LINQ में लागू:

public static string ToDigitsOnly(this string input) 
{ 
    if(input == null) 
      throw new ArgumentNullException("input"); 

    Regex digitsOnly = new Regex(@"[^\d]"); 
    return digitsOnly.Replace(input, String.Empty); 
} 

आप, विधि का उपयोग कर सकते हैं proposed by Jon Skeet। यह आपका चेक कम हो जाएगा बस

input.ThrowIfNull("input"); 

इसके अलावा जॉन एक अच्छा अनुभाग 10.2.4 गहराई में में सी # एक अशक्त संदर्भ पर एक विधि कॉलिंग, बोली है:

तुच्छता के लिए जाँच एक ईमानदार डेवलपर के रूप में, मुझे यकीन है कि आपके उत्पादन विधियां हमेशा कार्यवाही से पहले अपने तर्कों की वैधता की जांच करें। एक सवाल यह है कि स्वाभाविक रूप से इस quirky विस्तार विधियों की विशेषता से उत्पन्न होता है, जब पहले तर्क शून्य होता है (माना जाता है कि यह होना नहीं है)। क्या यह ArgumentNullException होना चाहिए, जैसे कि यह एक सामान्य तर्क था, या इसे होना चाहिए NullReferenceException, जो होता है यदि एक्सटेंशन विधि प्रारंभ करने के लिए एक उदाहरण विधि थी? मैं पूर्व की अनुशंसा करता हूं: यह अभी भी एक तर्क है, भले ही एक्सटेंशन विधि वाक्यविन्यास स्पष्ट नहीं करता है।

मैं के रूप में (और से मेरा व्यक्तिगत अनुभव) इस सिफारिश देखें: यह हमेशा अशक्त के लिए जाँच करने के लिए बेहतर है, विशेष रूप से स्थिर तरीकों के लिए और शून्य मान पर भरोसा करने की नहीं है। एक अपवाद केवल तभी होता है जब यह आपकी विधि का सही उद्देश्य है, उदाहरण के लिए ThrowIfNull या IsNullOrEmpty विस्तार विधियां।

Answer 2

जब तक आप व्यवहार को अच्छी तरह से संवाद नहीं करते हैं, तब तक वास्तव में कोई फर्क नहीं पड़ता है (ताकि अंतिम उपयोगकर्ता जानता है कि क्या उम्मीद करनी है)।

अपेक्षित व्यवहार को संवाद करने के लिए अंतर्निहित XML Documentation Comments का उपयोग करने पर विचार करें।

/// <exception cref="ArgumentNullException">argument is null.</exception> 
public string Example(string argument) 
{ 
    if (argument == null) 
     throw new ArgumentNullException(); 
    return argument.ToString(); 
} 

कई उदाहरण के लिए

MSDN देखें प्रलेखन:

• DateTime.ParseExact Method (String, String, IFormatProvider)
• Uri.FromHex Method

Answer 3

मान लीजिए मैं इस है:

class A 
{ 
    public void F() 
    { 
     //do stuff 
    } 
} 

अगर मैं तो चलाने निम्नलिखित कोड, क्या होता है?

A a = null; 
a.F(); 

आपको NullReferenceException मिलता है। तो मैं कहूंगा कि बराबर विस्तार विधि लिखने का उचित तरीका निम्नानुसार होगा।

class A 
{ 
} 

static class AExtensions 
{ 
    void F(this A a) 
    { 
     if (a == null) 
     { 
      throw new NullReferenceException(); 
     } 
     //do stuff 
    } 
} 

हालांकि, .NET इस पर मुझसे असहमत है। .NET में मानक को ArgumentException फेंकना है - इसलिए इसके बजाय ऐसा करना सबसे अच्छा है।

Answer 4

सरल; स्ट्रिंग के लिए एक और तरीका बनाएं, IsInValid()

सार्वजनिक स्थैतिक बूल IsInValid (यह स्ट्रिंग एस) { वापसी (s == null) || (एस लम्बाई == 0); }

उपयोग जहाँ आप जाँच करना चाहते हैं ...

इसके अलावा, आप इस एक्सटेंशन का उपयोग कहीं भी