List vs IEnumerable in private,lazy-loaded property

Question

我有一个公共属性 (AllCustomers)，它由一个私有属性支持，用于延迟加载。

我了解公共属性应该是 IEnumerable ("program to interfaces, not implementations")。

但是，我可以看到两种构建私有财产的方法。

第一个选项，带有私有列表-

private List<Customer> _AllCustomers;
public IEnumerable<Customer> AllCustomers
{
    get
    {
        if (_AllCustomers == null)
        {
            _AllCustomers = DAL.GetAllCustomers().ToList(); 
        }
        return _AllCustomers;
    }
}

第二个选项，带有私有 IEnumerable-

private IEnumerable<Customer> _AllCustomers;
public IEnumerable<Customer> AllCustomers
{
    get
    {
        if (_AllCustomers == null)
        {
            _AllCustomers = DAL.GetAllCustomers(); 
        }
        return _AllCustomers;
    }
}

我认为第一个选项看起来更正确，因为它会访问数据库一次并存储结果，而第二个会导致多次数据库访问。

我的问题是-

• 我的分析是否正确？
• 不同方法的含义是什么？
• 是否有任何时候首选第二个选项？
• 有没有更好、更惯用的方式来表达 第二个选项？

Answer 1

这里涉及到几个级别的“懒惰”。一个是 IEnumerable 实现固有的惰性，另一个是您自己添加到属性中的惰性实现。

您的第一个实现将访问数据库一次，第一次访问 AllCustomers。它将在GetAllCustomers 中构造查询，并在您调用ToList 时执行它，并将结果存储在本地。

您的第二个实现也只会访问数据库一次（假设您的 LINQ 实现还不错）。但是，这将比第一种情况晚 - 即使调用您的AllCustomer 属性也只会返回一个IQueryable，这只会在实际访问或枚举AllCustomers 时执行。这可能是紧随其后，也可能不是——这是一个更懒惰的实现。同样，假设您的 LINQ 提供程序不是太愚蠢，遍历整个集合仍然只会访问数据库一次。

为什么要选择第一个选项？因为（同样，取决于实现），迭代AllCustomers 两次 可能会命中数据库两次。 Resharper 非常方便地警告我们，当我们可能有一个 IEnumerable 的多个枚举时。将其存储在本地 List 将确保我们保留缓存的本地副本。为确保在代码中明确表达这一点，请考虑公开 IReadOnlyList 而不是 IEnumerable。

Answer 2

当第一次调用 getter 时，两者都只会访问 DB 一次，因为 _AllCustomers 支持字段仅是 NULL。

根据个人喜好，我通常更喜欢在我的接口和方法声明中使用IEnumerable，但是支持字段是具体的类，例如List<stuff>。让修改您的收藏之类的事情变得更容易。

Answer 3

我了解公共属性应该是 IEnumerable（“程序到接口，而不是实现”）。

这种理解是有缺陷的。接口编程的原则并没有说应该编程到哪个接口。在你的例子中。我会公开IList<T>，并将其设为只读，假设您不希望消费者修改列表：

private List<Customer> _AllCustomers;
public IList<Customer> AllCustomers
{
    get
    {
        if (_AllCustomers == null)
        {
            _AllCustomers = DAL.GetAllCustomers().ToList().AsReadOnly(); 
        }
        return _AllCustomers;
    }
}