ReadOnlyCollection hoặc IEnumerable để trưng bày các bộ sưu tập thành viên?

Có bất kỳ lý do nào để hiển thị một bộ sưu tập nội bộ dưới dạng ReadOnlyCollection chứ không phải là IEnumerable nếu mã cuộc gọi chỉ lặp lại trên bộ sưu tập?

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

Như tôi thấy, IEnumerable là một tập hợp con của giao diện ReadOnlyCollection và nó không cho phép người dùng sửa đổi bộ sưu tập. Vì vậy, nếu giao diện IEnumberable là đủ thì đó là giao diện để sử dụng. Đó có phải là một cách suy luận đúng đắn về nó hay tôi đang thiếu một cái gì đó?

Cảm ơn / Erik

Giải pháp hiện đại hơn

Trừ khi bạn cần bộ sưu tập nội bộ có thể thay đổi, bạn có thể sử dụng System.Collections.Immutablegói, thay đổi loại trường của bạn thành một bộ sưu tập bất biến, và sau đó phơi bày điều đó trực tiếp - tất nhiên là giả sử Foonó là bất biến.

Cập nhật câu trả lời để giải quyết câu hỏi trực tiếp hơn

Có bất kỳ lý do nào để hiển thị một bộ sưu tập nội bộ dưới dạng ReadOnlyCollection chứ không phải là IEnumerable nếu mã cuộc gọi chỉ lặp lại trên bộ sưu tập?

Nó phụ thuộc vào mức độ bạn tin tưởng mã gọi. Nếu bạn hoàn toàn kiểm soát mọi thứ sẽ gọi thành viên này và bạn đảm bảo rằng sẽ không có mã nào sử dụng:

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

sau đó chắc chắn, sẽ không có hại gì nếu bạn trả lại bộ sưu tập trực tiếp. Tôi thường cố gắng để có một chút hoang tưởng hơn thế.

Tương tự như vậy, như bạn nói: nếu bạn chỉ cần IEnumerable<T> , thì tại sao lại buộc mình vào bất cứ điều gì mạnh mẽ hơn?

Câu trả lời gốc

Nếu bạn đang sử dụng .NET 3.5, bạn có thể tránh tạo một bản sao và tránh việc truyền đơn giản bằng cách sử dụng một cuộc gọi đơn giản để Bỏ qua:

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

(Có rất nhiều tùy chọn khác để gói một cách tầm thường - điều tuyệt vời về SkipChọn / Trường hợp không có đại biểu nào thực hiện vô nghĩa cho mỗi lần lặp.)

Nếu bạn không sử dụng .NET 3.5, bạn có thể viết một trình bao bọc rất đơn giản để làm điều tương tự:

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

Nếu bạn chỉ cần lặp qua bộ sưu tập:

foreach (Foo f in bar.Foos)

sau đó trả lại IEnumerable là đủ.

Nếu bạn cần truy cập ngẫu nhiên vào các mục:

Foo f = bar.Foos[17];

sau đó bọc nó trong ReadOnlyCollection .

Nếu bạn làm điều này thì không có gì ngăn người gọi của bạn chuyển IEnumerable trở lại ICollection và sau đó sửa đổi nó. ReadOnlyCollection loại bỏ khả năng này, mặc dù vẫn có thể truy cập vào bộ sưu tập có thể ghi bên dưới thông qua sự phản chiếu. Nếu bộ sưu tập nhỏ thì cách an toàn và dễ dàng để khắc phục vấn đề này là trả lại một bản sao thay thế.

Tôi tránh sử dụng ReadOnlyCollection càng nhiều càng tốt, nó thực sự chậm hơn đáng kể so với việc chỉ sử dụng Danh sách bình thường. Xem ví dụ này:

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

Đôi khi bạn có thể muốn sử dụng một giao diện, có lẽ vì bạn muốn chế nhạo bộ sưu tập trong quá trình kiểm tra đơn vị. Vui lòng xem mục blog của tôi để thêm giao diện của riêng bạn vào ReadonlyCollection bằng cách sử dụng bộ chuyển đổi.