检查序列是否为空的推荐方法

Question

一个方法返回一个序列IEnumerable<T>，你现在想检查它是否为空。你如何建议这样做？我正在寻找良好的可读性和良好的性能。

第一个也是最明显的方法是检查计数是否大于零：

if(sequence.Count() == 0)

具有不错的可读性，但性能很差，因为它实际上必须遍历整个序列。

我有时使用的一种方法如下：

if(!sequence.Any())

这并不（据我所知）必须遍历整个序列，但可读性有点倒退和尴尬。 （如果我们检查序列是否为 not 空的话会更好读）。

另一种选择是在 try-catch 中使用 First，如下所示：

try
{
    sequence.First();
}
catch(InvalidOperationException)
{
    // Do something
}

不是一个非常漂亮的解决方案，而且可能也更慢，因为它使用了异常和东西。当然可以通过使用FirstOrDefault 来防止这种情况发生，除非如果序列中的第一项实际上是默认值，你会遇到一个大问题；）

那么，还有其他方法可以检查序列是否为空吗？你通常使用哪一种？您推荐使用哪一种？

注意：为了获得最佳可读性，我可能会将上述 sn-ps 之一放在 IsEmpty 扩展方法中，但我仍然很好奇，因为我会在该方法中也做一些事情：p

Answer 1

我个人会使用!sequence.Any()。

如果您真的需要，您可以随时编写自己的扩展方法：

public static bool IsEmpty<T>(this IEnumerable<T> source)
{
    return !source.Any();
}

然后你可以写：

if (sequence.IsEmpty())

Answer 2

您可以使用此实现创建扩展方法。

public static bool IsEmpty<T>(this IEnumerable<T> items) {
        using (var enumerator = items.GetEnumerator())
        {
            return !enumerator.MoveNext();
        }
 }

Answer 3

您调用的所有这些方法都是 LINQ 扩展方法，因此这取决于 LINQ 提供程序是如何实现的。如果你想知道一个序列是否为空，Count() == 0 或Any() == false 是合适的。我更喜欢Any()自己。

但是，根据您的 sequence 的实际类型，您可能不需要使用 LINQ 扩展方法。 IE。如果它是一个数组，你可以调用sequence.Length。如果是合集，可以使用sequence.Count。

Answer 4

你说：

if(sequence.Count() == 0) 具有不错的可读性，但性能很差，因为它实际上必须遍历整个序列。

这是真的吗？您正在谈论处理 Interface，IEnumerable<T>，但您正在对其实现做出假设，这些假设可能正确，也可能不正确。事实上，我多年来编写的许多自定义集合都保留了一个私有变量，用于在内部存储当前计数，这意味着返回 .Count 是一件小事，不需要迭代整个集合。

话虽如此，除非您知道特定实现对.Count 的优化很差，否则我会使用.Count。尽可能避免过早优化，并坚持可读性。

Answer 5

我使用这个扩展方法来检测序列是否为空或没有任何项目，或者检测序列是否至少有一个项目，就像string.IsNullOrEmpty() 方法一样。

 public static bool IsNullOrEmpty<TSource>(this IEnumerable<TSource> source) {
     if (source == null) {
        return true;
     }

     return !source.Any();
 }

 public static bool IsNotNullOrEmpty<TSource>(this IEnumerable<TSource> source) {
     return !source.IsNullOrEmpty();
 }
 .
 .
 .
 if (!sequence.IsNullOrEmpty()) {
     //Do Something with the sequence...
 }

Answer 6

我有时使用的方法如下：

if(!sequence.Any())

^❶ 这并不（据我所知）必须经历整个序列， ^❷ 但可读性有点落后和尴尬。 （读了很多 如果我们检查序列是否为空，则更好）。

1. 根据微软的说法，Any 确实不必须经历整个序列。引用备注部分：

source 的枚举一有结果就停止 确定。

1. 在测试if-else 语句中是否存在元素时尤其如此。可以说，如果测试if 语句中是否存在元素，以及else 中是否存在元素，则可读性最好，从而避免使用! 运算符：

   if (sequence.Any())
   {
   }
   else
   {
   }
   

   大多数人会认为它比以下更具可读性：

   if (!sequence.Any())
   {
   }
   else
   {
   }