面向对象的设计 - 一流的集合/封装的集合是否应该实现集合/IEnumerable 接口？

Question

我是 Jeff Bay 的 Object Calisthenics 练习的粉丝：

http://www.xpteam.com/jeff/writings/objectcalisthenics.rtf

通过创建对象来封装集合的实现以及对象集合的迭代，我在实现一流的集合方面取得了很好的成果。

这意味着我有一个 EmployeeList 对象，而不是一个 List。

通常，当我创建这些对象时，我不会实现任何集合接口，因此外部用户无法枚举内部集合，因为我认为这就是重点。然而，一位同事提出了相反的建议 - 实现 ICollect 接口以使其更灵活。

这个重构站点似乎在同一个方向，虽然它并没有完全删除对内部集合的访问：http://sourcemaking.com/refactoring/encapsulate-collection

    // Foreach method
    foreach (var employee in employees)
    {
        if (employee.IsManager)
        {
            managers.Add(employee);
        }
    }

    // Linq method
    managers = employees.Where(e => e.IsManager);


    // Encapsulated iteration method
    managers = employees.GetAllManagers();

    public IEnumerable<Employee> GetAllManagers() // Inside Employees class
    {
        return employees.Where(e => e.IsManager);
    }

我是否说最后一种方法是有利的？例如，您何时希望通过实现 IEnumerable 来公开您的内部集合？

Answer 1

IEnumerable 可能是一个不错的选择。您可以拥有一个 Employee 对象的私有列表，并将其公开为 IEnumerable，以便人们在循环、LINQ 等中使用它。

由于IEnumerable 不允许您修改被迭代的集合，您不必担心EmployeeList 的使用者会更改内部结构。

提供一个迭代方法并不理想，因为它破坏了POLA，并且需要使用 API 的程序员付出一些努力才能找到他想要调用的方法。仔细想想，在 .NET 中你永远不会有像下面这样的代码。

var list = new List<T>;
foreach (var item in list.GetAllItems())
{
    // Whut? 
}