メンバーコレクションを公開するためのReadOnlyCollectionまたはIEnumerable？

呼び出しコードがコレクションを反復するだけの場合、内部コレクションをIEnumerableではなくReadOnlyCollectionとして公開する理由はありますか？

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インターフェースで十分な場合は、それを使用します。それはそれについて推論する適切な方法ですか、それとも何か不足していますか？

ありがとう/ Erik

より近代的なソリューション

内部コレクションを変更可能にする必要がない限り、System.Collections.Immutableパッケージを使用してフィールドタイプを不変コレクションに変更し、それを直接公開することができます- Fooもちろん、それ自体は不変であると仮定します。

質問に直接対応するために回答を更新

呼び出しコードがコレクションを反復するだけの場合、内部コレクションをIEnumerableではなくReadOnlyCollectionとして公開する理由はありますか？

それは、呼び出し元のコードをどれだけ信頼するかによって異なります。このメンバーを呼び出すすべてのものを完全に制御していて、コードが使用しないことを保証する場合：

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を使用するよりもかなり遅くなります。この例を見てください：

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に追加する方法については、私のブログエントリを参照してください。