IEnumerable vs Ilist - IsNullOrEmpty 扩展方法

Question

我有一个自定义集合IList<user>作为用户。当我尝试检查users是否为空或为空时，我没有得到任何智能帮助（如IsNullOrEmpty），所以我编写了以下扩展方法

public static bool IsNullOrEmpty<T>(this IEnumerable<T> source)
    {
        if (source.IsNullOrEmpty())
        {
            return true;
        }
        return false;
    }

令我惊讶的是，我发现 IEnumberable 有 IsNullOrEmpty()。

据我所知，IList 扩展了ICollection，它再次扩展了IEnumerable，如果是这样，那么IList 应该支持IsNullOrEmpty。

请纠正我的错误。

Answer 1

IEnumerable<T> 没有IsNullOrEmpty 方法，它是你上面写的扩展。如果你调用它，你会得到一个StackOverflowException。

你可以这样实现它：

public static bool IsNullOrEmpty<T>(this IEnumerable<T> source)
{
    return source == null || !source.Any();
}

但重要的是要注意，这种方法不是很有帮助，甚至可能会变得更糟。因为Enumerable.Any 将“使用”查询。因此，如果它不是内存中的集合，它必须调用GetEnumerator 并开始枚举它以检查是否至少有一个项目。有时，如果对象被枚举（例如在 .NET File.ReadLines 中），该对象将被丢弃，如果您稍后再次尝试使用它，这将导致 ObjectDisposedException。

此方法具有不需要的副作用的另一个示例是当序列是过滤的查询时（...Where(x => Compute(x, random.Next()))）。此查询可能每次都会产生不同的结果。引用 Marc Gravell 的评论：“IEnumerable 不应被假定为可重复的”。

在 Linq-To-Sql 或 Linq-To-Entities 中，您将在每次 IsNullOrEmpty 调用时调用数据库。

Answer 2

正如@Marc Gravel 在 cmets 中提到的“不应假定IEnumerable<T> 是可重复的”。假设您的源 IEnumerable 是从数据读取器创建的。你只有一次机会迭代它。如果您只是为了查看它是否为空而消耗了唯一的机会，您将无法再次访问记录。也许你想为IList而不是IEnumerable写一个扩展方法（如果需要的话）。

public static bool IsNullOrEmpty<T>(this IList<T> source)
{
    return source == null || source.Count == 0;
}