在 C# 中，执行包含 null 大小写的 Any() 的最简洁方法是什么？

Question

我在 C# 中有以下代码：

 if (this.Trucks.Any() || this.Cars.Any())
 {
      return true;
 }
 return false;

但我刚刚意识到有时这些数组中的一个或其中一个为 null，所以在我的情况下，我希望 null 只返回 false（而不是抛出异常）。

我可以像这样添加预先的空检查：

 if ((this.Trucks != null && this.Trucks.Any()) || 
    (this.Cars != null && this.Cars.Any()))
 {
    return true;
 }
  return false;
 }

但想看看是否有更清洁的方法

Answer 1

我建议您尽可能地使用非空合约（空对象模式）。这意味着您可能会在构造函数空数组中初始化（Enumerable.Empty<T>() 在 IEnumerable 的情况下 - 在数组的情况下添加 .ToArray() ）。空值检查的问题在于，它通常会在代码上传播和成倍增加——代码（和单元测试）的复杂性会不必要地增加。忘记在x 位置检查 null 比忘记在一个位置初始化空对象更容易。您还可以使用Code contracts 库在运行时检查您的“合同”。事实上，“null”的发明者称这是十亿美元的错误："Null References: The Billion Dollar Mistake"

Answer 2

如果您想将空序列视为空序列，我会创建一个扩展方法：

public static class Seq
{
    public static IEnumerable<T> EmptyIfNull<T>(this IEnumerable<T> seq)
    {
        return seq ?? Enumerable.Empty<T>();
    }
}

return this.Trucks.EmptyIfNull().Any() || this.Cars.EmptyIfNull().Any()

如果您不喜欢使用扩展方法处理空值，您可以删除 this 修饰符并使用

return Seq.EmptyIfNull(this.Trucks).Any() || Seq.EmptyIfNull(this.Cars).Any()

Answer 3

制作你自己的实现：

public static Class StaticHelpers
{
    public static bool AnyEx<T>(this IEnumerable<T> enumerable)
    {
        if (enumerable == null) return false;

        return enumerable.Any();
    }
}

像这样使用它：

if (this.Trucks.AnyEx() || this.Cars.AnyEx())
{
    return true;
}
return false;

Answer 4

您只需在主要的if 条件下检查它们怎么样：

if ((this.Trucks != null && this.Trucks.Any()) || 
    (this.Cars != null && this.Cars.Any()))
{
  return true;
}
return false;

这可以重写为一长行：

return ((this.Trucks != null && this.Trucks.Any()) || 
        (this.Cars != null && this.Cars.Any()));