ReadOnlyCollection або IEnumerable для викриття колекцій членів?

Чи є якісь підстави виставляти внутрішню колекцію як ReadOnlyCollection, а не IEnumerable, якщо код, що викликає, лише ітерацію над колекцією?

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);

Як я бачу, IEnumerable є підмножиною інтерфейсу ReadOnlyCollection, і він не дозволяє користувачеві змінювати колекцію. Тож якщо інтерфейсу IEnumberable достатньо, то це саме той, який слід використовувати. Це правильний спосіб міркування про це чи я щось пропускаю?

Дякую / Ерік

Більш сучасне рішення

Якщо вам не потрібна внутрішня колекція, щоб бути незмінною, ви можете використовувати System.Collections.Immutableпакет, змінити тип поля, щоб бути незмінною колекцією, а потім викрити це безпосередньо - якщо вважати Fooсебе непорушним, звичайно.

Оновлена ​​відповідь, щоб вирішити це питання безпосередньо

Чи є якісь підстави виставляти внутрішню колекцію як ReadOnlyCollection, а не IEnumerable, якщо код, що викликає, лише ітерацію над колекцією?

Це залежить від того, наскільки ви довіряєте коду виклику. Якщо ви повністю контролюєте все, що коли-небудь дзвонить цьому учаснику, і ви гарантуєте, що жоден код ніколи не використовуватиме:

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

то впевнений, ніякої шкоди не буде, якщо ви просто повернете колекцію безпосередньо. Я, як правило, намагаюся бути трохи більше параноїком, ніж це.

Так само, як ви кажете: якщо вам тільки потрібно IEnumerable<T> , то навіщо прив’язувати себе до чогось сильнішого?

Оригінальна відповідь

Якщо ви використовуєте .NET 3.5, ви можете уникнути створення копії та уникнути простого перенесення, використовуючи простий виклик для пропуску:

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

(Існує безліч інших варіантів для завершення тривіального завершення - приємна річ щодо SkipOver / Select, де немає делегата, який би безглуздо виконувався для кожної ітерації.)

Якщо ви не використовуєте .NET 3.5, ви можете написати просту обгортку, щоб зробити те саме:

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

Якщо вам потрібно лише повторити колекцію:

foreach (Foo f in bar.Foos)

потім повертається IE Чимало досить.

Якщо вам потрібен випадковий доступ до предметів:

Foo f = bar.Foos[17];

потім загорніть його в ReadOnlyCollection .

Якщо ви це зробите, то ніщо не зупиняє ваші абоненти перекидати IEnumerable назад до ICollection та змінювати його. Програма ReadOnlyCollection видаляє цю можливість, хоча все ще можна отримати доступ до базової колекції, що записується, за допомогою відображення. Якщо колекція невелика, тоді безпечним і простим способом подолати цю проблему є повернення копії.

Я уникаю використання ReadOnlyCollection якомога більше, це насправді значно повільніше, ніж просто використання звичайного списку. Дивіться цей приклад:

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());

Іноді ви можете використовувати інтерфейс, можливо, тому що ви хочете знущатися над колекцією під час тестування одиниць. Будь ласка, дивіться мій запис у щоденнику щодо додавання власного інтерфейсу до ReadonlyCollection за допомогою адаптера.