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, вы можете избежать копирования и простого приведения, используя простой вызов Skip:

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

(Существует множество других вариантов тривиального обертывания - приятная особенность Skipнад Select / Where заключается в том, что нет делегата, который нужно выполнять бессмысленно для каждой итерации.)

Если вы не используете .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)

тогда достаточно возврата IEnumerable .

Если вам нужен произвольный доступ к элементам:

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 с помощью адаптера.