एक IEnumerable <T> या ICollection <T> रूप में एक आंतरिक सूची <T> लौटने के लिए ठीक?

Question

संभव डुप्लिकेट:
Collection<T> versus List<T> what should you use on your interfaces?

इस विधि-लौटे चर myVar पर विचार करें एक List<T> है, लेकिन विधि MyMethod() की वापसी प्रकार एक IEnumerable<T> है:

public IEnumerable<T> MyMethod(string stuff) 
{ 
    var myVar = new List<T>(); 
    //do stuff 

    return myVar; 
} 

अनिवार्य रूप से, मैं क्या जानना चाहता हूं अगर रिटर्न है एनजी myVar एक अलग प्रकार के रूप में ठीक है।

विशेष रूप से, मेरी स्थिति के संबंध में, 'डू स्टफ' DataRow के माध्यम से जाता है और वस्तुओं की सूची में DataRow में आइटम असाइन करता है। मेरे पास ICollection और IList अन्य स्थानों पर रिटर्न प्रकारों के साथ समान स्थितियां भी हैं।

कारण मैं वापस जाने के लिए या तो IEnumerable या ICollection है ताकि मैं जरूरत से ज्यादा वापस नहीं कर रहा हूँ चाहता हूँ। लेकिन साथ ही, यह कॉलर को लौटाए गए मान को List पर बदलने की अनुमति देता है यदि उसे ऐसा करने की आवश्यकता है।

हालांकि, यह अजीब लगता है कि मेरा रिटर्न स्टेटमेंट List लौटा रहा है, विधि के रिटर्न प्रकार के बजाय। क्या यह सामान्य अभ्यास है? क्या ऐसा करने में कुछ गड़बड़ है?

स्पष्टीकरण (शिकार टिप्पणियों के जवाब में):

बस स्पष्ट करने के लिए, क्या मैं उत्सुक हूँ है के बारे में है, तो यह ठीक है कि शरीर में मेरी वापसी कथन एक List<T> लौटा रहा है है, लेकिन विधि हैडर IEnumerable, या संभवतः ICollection, Collection, आदि की वापसी प्रकार ... कुछ अलग पिण्डों से से लौटने के बयान है।

Answer 1

न केवल इसके साथ कुछ भी गलत है, लेकिन यह वास्तव में अच्छा अभ्यास है: केवल सख्ती से जरूरी है। इस तरह, कॉलर इस तथ्य पर भरोसा नहीं कर सकता कि विधि List<T> लौटाएगी, इसलिए अगर किसी कारण से आपको कुछ और लौटने के लिए कार्यान्वयन को बदलने की जरूरत है, तो आप अपना अनुबंध तोड़ नहीं देंगे। हालांकि कॉलिंग कोड तोड़ सकता है अगर यह (गलती से) विधि वास्तव में क्या विधि देता है, इस बारे में धारणाएं बनाते हैं।

Answer 2

इसमें कुछ भी गलत नहीं है।

असल में लौटने ठोस वर्ग को पूरा करने के लिए एक ही रास्ता है आवश्यकता "इंटरफेस रिटर्न"।

कमियां हो यदि आप विशेष रूप से कक्षाओं का ज्ञान कोड (List<T> r = (List<T>)MyMethod("ff")) कॉल करने में चुपके से दे सकता है। लेकिन यदि आप परिणाम को इंटरफ़ेस के रूप में देखते हैं (जैसा कि आपको माना जाता है) यह ठीक है।

Answer 3

कारण वापस जाने के लिए IEnumerable<T> प्रकार है इतना है कि वे फोन करने वाले सूची (एक सूची के लिए इसे वापस कास्टिंग के बिना) को संशोधित नहीं कर सकते हैं। यदि आप सभी (अधिकांश?) देखते हैं तो एक्सटेंशन विधियां IEnumerable<T> तर्क लेती हैं, ताकि आप जान सकें कि एक्सटेंशन विधि आपकी सूची को संशोधित नहीं करती है।

दूसरे, IEnumerable<T> से List<T> इनहेरिट करती है।

संपादित करें: थॉमस के रूप में एक टिप्पणी है कि IEnumerable<T> एक List<T> वापस करने के लिए डाली जा सकता है और संशोधित कॉल करने वाले को में विस्तार से बताया। यदि आपका मुख्य लक्ष्य इसे केवल पढ़ने के लिए है, तो आप सूची को myVar.AsReadOnly() के रूप में वापस कर सकते हैं।लेकिन फिर ReadOnlyCollection<T> है।

Answer 4

मुझे लगता है कि यह इष्टतम नहीं है, क्योंकि यह कॉलर को List<T> पर डालने की अनुमति देता है, जो कार्यान्वयन विस्तार पर निर्भर करता है। मैं Select(x=>x) जैसे किसी प्रकार का मास्किंग जोड़ूंगा।

गतिशील रूप से टाइप किए गए कॉलर को यह भी ध्यान नहीं दिया जा सकता है कि आपने अपनी सूची IEnumerable<T> पर डाली है। उसके लिए यह बस List<T> है।

IEnumerable<T> के रूप में एक आंतरिक स्थायी List<T> लौटाना बहुत गलत होगा, क्योंकि यह कॉलर को आपकी कक्षा की आंतरिक स्थिति को बदलने की अनुमति देता है। लेकिन चूंकि आप प्रत्येक कॉल पर List<T> का एक नया उदाहरण लौटाते हैं, जिसका उपयोग कहीं और नहीं किया जाता है, यह तर्क आपके उदाहरण में लागू नहीं होता है।