c# - Should I redundantly test arguments (e.g collection emptiness)? -

अनावश्यक संग्रह की जांच के साथ कोई समस्या है?:

  SomeMethod () { आकार = GetShapes (); / / शायद आकृति (आकार। किसी भी ())? अगर (आकृतियों। किसी भी ()) {टॉगल दृश्यता (आकार); }} ToggleVisibility (IEnumerable & lt; आकार & gt; आकृतियों) {// हो सकता है आचरण (आकार। Any ())? अगर (आकृतियों। किसी भी ()) {// do stuff}}  

मैं डॉन नहीं लगता है कि यहां एक बड़ी समस्या है क्योंकि किसी भी कॉल को महँगाई नहीं है।

इसमें एक छोटी सी समस्या है कि टॉगल विज़िबिलिटी की जिम्मेदारी और व्यवहार घोषित नहीं किया गया है। टॉगल विज़िबिलिटी को कॉल करने वालों को पता होना चाहिए कि आकृतियों के खाली या शून्य होने पर यह कैसे व्यवहार करेगा। ऐसा करने का सबसे अच्छा तरीका XML टिप्पणियों के माध्यम से है ताकि यह इंटेलिसेंस में दिखाया जा सके। इससे टॉगल विज़िबिलिटी कॉल करने वाले को यह तय होगा कि क्या उन्हें चेक करना आवश्यक है कि क्या संग्रह रिक्त या शून्य है।