एक निजी संग्रह पर पुनरावृत्ति की अनुमति कैसे दें लेकिन संशोधन नहीं?

Question

अगर मैं निम्नलिखित वर्ग के सदस्य हैं:

private List<object> obs; 

और मैं क्लास 'इंटरफेस के हिस्से के रूप में इस सूची की ट्रेवर्सल अनुमति देना चाहते हैं, मैं इसे कैसे करना होगा?

इसे सार्वजनिक बनाना काम नहीं करेगा क्योंकि मैं सूची को सीधे संशोधित करने की अनुमति नहीं देना चाहता हूं।

Answer 1

आप एक IEnumerable<T> के रूप में यह पर्दाफाश होगा, लेकिन बस उसे सीधे वापस नहीं:

public IEnumerable<object> Objects { get { return obs.Select(o => o); } } 

के बाद से है, जिसे आपने केवल सूची के ट्रेवर्सल चाहता था, यह सब आप की जरूरत है।

एक एक IEnumerable<T> के रूप में सीधे List<object> वापसी करने के लिए परीक्षा हो सकती है, लेकिन यह है कि, सही नहीं होगा और क्योंकि एक को आसानी से रन टाइम पर IEnumerable<T> निरीक्षण का निर्धारण यह एक List<T> है और इस तरह के लिए यह डाली और सामग्री उत्परिवर्तित सकता है।

हालांकि, return obs.Select(o => o); का उपयोग करके आप List<object> पर एक पुनरावर्तक लौटते हैं, List<object> के प्रत्यक्ष संदर्भ के लिए नहीं।

कुछ सोच सकते हैं कि यह सी # भाषा विशिष्टता की धारा 7.15.2.5 के अनुसार "अपमानजनक अभिव्यक्ति" के रूप में योग्यता प्राप्त करता है। हालांकि, Eric Lippert goes into detail as to why this projection isn't optimized away।

इसके अलावा, लोग सुझाव दे रहे हैं कि AsEnumerable extension method का उपयोग करें। यह गलत है, क्योंकि मूल सूची की संदर्भ पहचान बनाए रखा जाता है।

AsEnumerable<TSource>(IEnumerable<TSource>) विधि अन्य एक प्रकार है कि IEnumerable<T> ही IEnumerable<T> लागू करता है से स्रोत का संकलन समय प्रकार बदलने के लिए की तुलना में कोई प्रभाव नहीं है: दस्तावेज की टिप्पणियां अनुभाग से।

दूसरे शब्दों में, सब यह होता है IEnumerable<T>, जो referencial अखंडता की रक्षा में मदद नहीं करता करने के लिए स्रोत पैरामीटर डाली है, मूल संदर्भ दिया जाता है और List<T> वापस करने के लिए डाली जा सकती है और इस सूची में उत्परिवर्तित लिए इस्तेमाल किया जा।

Answer 2

क्या आपने System.Collections.ReadOnlyCollectionBase से कक्षा प्राप्त करने पर विचार किया है?

Answer 3

आप ReadOnlyCollection का उपयोग कर सकते हैं या List की एक प्रति बना सकते हैं और इसे कॉपी कर सकते हैं (कॉपी ऑपरेशन के प्रदर्शन दंड पर विचार करते हुए)। आप List<T>.AsReadOnly का भी उपयोग कर सकते हैं।

Answer 4

बेनकाब एक ReadOnlyCollection<T>

Answer 5

आप दो तरह से कर सकते हैं:

1. या तो एक केवल पढ़ने के लिए संग्रह में सूची परिवर्तित द्वारा:

   new System.Collections.ObjectModel.ReadOnlyCollection<object>(this.obs) 
   

2. या के एक IEnumerable वापस लौट कर आइटम:

   this.obs.AsEnumerable() 
   

Answer 6

इस मुद्दे पर दिलचस्प पोस्ट और संवाद: http://davybrion.com/blog/2009/10/stop-exposing-collections-already/। सार्वजनिक IEnumerable TraverseTheList()

इतनी के रूप में implimented:

Answer 7

करने के लिए अपने इंटरफेस के लिए निम्न विधि हस्ताक्षर जोड़ने

public IEnumerable<object> TraverseTheList() 
{ 

    foreach(object item in obj) 
    { 
     yield return item; 
    } 


} 

कि आप निम्न करने की अनुमति देगा:

foreach(object item in Something.TraverseTheList()) 
{ 
// do something to the item 
} 

उपज वापसी आपके लिए एक गणक बनाने के लिए कंपाइलर को बताती है।

Answer 8

यह पहले से ही कहा जा चुका है, लेकिन मुझे कोई जवाब सुपरकलर के रूप में नहीं दिख रहा है।

सबसे आसान तरीका है बस एक ReadOnlyCollection वापस जाने के लिए है

private List<object> objs; 

public ReadOnlyCollection<object> Objs { 
    get { 
     return objs.AsReadOnly(); 
    } 
} 

इस के साथ दोष यह है, यह है कि आप बाद में अपने कार्यान्वयन को बदलना चाहते हैं, तो कुछ कॉल करने को पहले से ही इस तथ्य पर निर्भर है, हो सकता है कि संग्रह यादृच्छिक पहुंच प्रदान करता है। तो एक सुरक्षित परिभाषा सिर्फ बेनकाब करने के लिए एक IEnumerable

public IEnumerable<object> Objs { 
    get { 
     return objs.AsReadOnly(); 
    } 
} 

ध्यान दें कि आप इस कोड को संकलित करने के लिए AsReadOnly() कॉल करने के लिए की जरूरत नहीं है किया जाएगा। लेकिन अगर आप नहीं करते हैं, तो कॉलर ने मेरी वापसी सूची को वापस सूची में डाला और अपनी सूची संशोधित की।

// Bad caller code 
var objs = YourClass.Objs; 
var list = objs as List<object>; 
list.Add(new object); // They have just modified your list. 

ही संभावित समस्या भी इस समाधान

public IEnumerable<object> Objs { 
    get { 
     return objs.AsEnumerable(); 
    } 
} 

के साथ मौजूद है तो मैं निश्चित रूप से सुझाव है कि आप आप सूची में AsReadOnly() कहते हैं, और है कि मूल्य वापस है।