Üye koleksiyonlarını sergilemek için ReadOnlyCollection veya IEnumerable?

Çağıran kod yalnızca koleksiyon üzerinde yineleniyorsa, dahili bir koleksiyonu bir IEnumerable yerine bir ReadOnlyCollection olarak göstermek için herhangi bir neden var mı?

class Bar
{
    private ICollection<Foo> foos;

    // Which one is to be preferred?
    public IEnumerable<Foo> Foos { ... }
    public ReadOnlyCollection<Foo> Foos { ... }
}


// Calling code:

foreach (var f in bar.Foos)
    DoSomething(f);

Gördüğüm gibi IEnumerable, ReadOnlyCollection arabiriminin bir alt kümesidir ve kullanıcının koleksiyonu değiştirmesine izin vermez. Yani IEnumberable arayüzü yeterliyse, o zaman kullanılacak olan budur. Bu, bunun hakkında akıl yürütmenin uygun bir yolu mu yoksa bir şeyi mi kaçırıyorum?

Teşekkürler / Erik

Daha modern çözüm

Dahili koleksiyonun değiştirilebilir olması gerekmedikçe, System.Collections.Immutablepaketi kullanabilir, alan türünüzü değişmez bir koleksiyon olacak şekilde değiştirebilir ve sonra bunu doğrudan ifşa edebilirsiniz - Footabii ki kendi kendine değişmez olduğunu varsayarak .

Soruyu daha doğrudan ele almak için yanıt güncellendi

Çağıran kod yalnızca koleksiyon üzerinde yineleniyorsa, dahili bir koleksiyonu bir IEnumerable yerine bir ReadOnlyCollection olarak göstermek için herhangi bir neden var mı?

Arama koduna ne kadar güvendiğinize bağlıdır. Bu üyeyi arayacak her şey üzerinde tam kontrole sahipseniz ve hiçbir kodun kullanılmayacağını garanti ediyorsanız :

ICollection<Foo> evil = (ICollection<Foo>) bar.Foos;
evil.Add(...);

o zaman elbette, koleksiyonu doğrudan iade etmenizin bir zararı olmayacaktır. Genelde bundan biraz daha paranoyak olmaya çalışırım.

Aynı şekilde, dediğiniz gibi: eğer sadece ihtiyacınız varsa IEnumerable<T>, o zaman neden kendinizi daha güçlü bir şeye bağlayasınız?

Orijinal cevap

.NET 3.5 kullanıyorsanız, basit bir Atla çağrısı yaparak kopya oluşturmaktan ve basit atamadan kaçınabilirsiniz:

public IEnumerable<Foo> Foos {
    get { return foos.Skip(0); }
}

(Önemsiz bir şekilde kaydırmak için pek çok başka seçenek var - Skipfazla Seç / Nerede ile ilgili güzel olan şey , her yineleme için anlamsızca yürütecek bir temsilcinin olmamasıdır.)

.NET 3.5 kullanmıyorsanız, aynı şeyi yapmak için çok basit bir sarmalayıcı yazabilirsiniz:

public static IEnumerable<T> Wrapper<T>(IEnumerable<T> source)
{
    foreach (T element in source)
    {
        yield return element;
    }
}

Yalnızca koleksiyonu yinelemeniz gerekiyorsa:

foreach (Foo f in bar.Foos)

sonra IEnumerable'ı döndürmek yeterlidir.

Öğelere rastgele erişime ihtiyacınız varsa:

Foo f = bar.Foos[17];

ardından ReadOnlyCollection'a sarın .

Bunu yaparsanız, arayanların IEnumerable'ı ICollection'a geri göndermesini ve ardından değiştirmesini engelleyen hiçbir şey yoktur. ReadOnlyCollection bu olasılığı ortadan kaldırır, ancak altta yatan yazılabilir koleksiyona yansıma yoluyla erişmek hala mümkündür. Koleksiyon küçükse, bu sorunu aşmanın güvenli ve kolay bir yolu bunun yerine bir kopyasını iade etmektir.

ReadOnlyCollection'ı olabildiğince kullanmaktan kaçınırım, aslında normal Listeyi kullanmaktan çok daha yavaştır. Bu örneğe bakın:

List<int> intList = new List<int>();
        //Use a ReadOnlyCollection around the List
        System.Collections.ObjectModel.ReadOnlyCollection<int> mValue = new System.Collections.ObjectModel.ReadOnlyCollection<int>(intList);

        for (int i = 0; i < 100000000; i++)
        {
            intList.Add(i);
        }
        long result = 0;

        //Use normal foreach on the ReadOnlyCollection
        TimeSpan lStart = new TimeSpan(System.DateTime.Now.Ticks);
        foreach (int i in mValue)
            result += i;
        TimeSpan lEnd = new TimeSpan(System.DateTime.Now.Ticks);
        MessageBox.Show("Speed(ms): " + (lEnd.TotalMilliseconds - lStart.TotalMilliseconds).ToString());
        MessageBox.Show("Result: " + result.ToString());

        //use <list>.ForEach
        lStart = new TimeSpan(System.DateTime.Now.Ticks);
        result = 0;
        intList.ForEach(delegate(int i) { result += i; });
        lEnd = new TimeSpan(System.DateTime.Now.Ticks);
        MessageBox.Show("Speed(ms): " + (lEnd.TotalMilliseconds - lStart.TotalMilliseconds).ToString());
        MessageBox.Show("Result: " + result.ToString());

Bazen, belki de birim testi sırasında koleksiyonla dalga geçmek istediğiniz için bir arayüz kullanmak isteyebilirsiniz. Bir adaptör kullanarak ReadonlyCollection'a kendi arayüzünüzü eklemek için lütfen blog girişime bakın.