在 foreach 循环中检查 null

Question

是否有更好的方法来执行以下操作：
在继续循环之前，我需要在 file.Headers 上检查 null

if (file.Headers != null)
{
  foreach (var h in file.Headers)
  {
   //set lots of properties & some other stuff
  }
}

简而言之，由于我的代码中发生的缩进级别，在 if 中编写 foreach 看起来有点难看。

会评估为

foreach(var h in (file.Headers != null))
{
  //do stuff
}

可能吗？

Answer 1

作为对 Rune 建议的轻微修饰，您可以创建自己的扩展方法：

public static IEnumerable<T> OrEmptyIfNull<T>(this IEnumerable<T> source)
{
    return source ?? Enumerable.Empty<T>();
}

然后你可以写：

foreach (var header in file.Headers.OrEmptyIfNull())
{
}

根据口味改名:)

Answer 2

假设 file.Headers 中元素的类型是 T 你可以这样做

foreach(var header in file.Headers ?? Enumerable.Empty<T>()){
  //do stuff
}

如果 file.Headers 为空，这将创建一个空的 T 枚举。但是，如果文件类型是您拥有的类型，我会考虑更改 Headers 的 getter。 null 是 unknown 的值，所以如果可能，当 null 实际上（/最初）应该解释为“我不知道是否有任何元素”时，而不是使用 null 作为“我知道没有元素”，而是使用空设置以表明您知道集合中没有元素。这也将是 DRY'er，因为您不必经常进行 null 检查。

EDIT 作为 Jons 建议的后续，您还可以创建一个扩展方法，将上述代码更改为

foreach(var header in file.Headers.OrEmptyIfNull()){
  //do stuff
}

在你不能改变getter的情况下，这将是我自己的首选，因为它通过给操作命名（OrEmptyIfNull）更清楚地表达了意图

上面提到的扩展方法可能会使优化器无法检测到某些优化。具体来说，那些与 IList 相关的使用方法重载 this 可以消除

public static IList<T> OrEmptyIfNull<T>(this IList<T> source)
{
    return source ?? Array.Empty<T>();
}

Answer 3

坦率地说，我建议：接受null 测试。 null 测试只是 brfalse 或brfalse.s；其他一切都将涉及更多工作（测试、分配、额外的方法调用、迭代器上不必要的GetEnumerator()、MoveNext()、Dispose() 等）。

if 测试简单、明显且高效。

Answer 4

使用Null-conditional Operator 和 ForEach() 比标准的 foreach 循环更快。
不过，您必须将集合转换为 List。

   listOfItems?.ForEach(item => // ... );

Answer 5

迭代前的“if”没问题，那些“漂亮”的语义很少会降低您的代码的可读性。

无论如何，如果缩进打扰了你，你可以改变 if 来检查：

if(file.Headers == null)  
   return;

只有当 headers 属性有一个真值时，你才会进入 foreach 循环。

我可以考虑的另一个选项是在 foreach 循环中使用 null-coalescing 运算符并完全避免 null 检查。 示例：

List<int> collection = new List<int>();
collection = null;
foreach (var i in collection ?? Enumerable.Empty<int>())
{
    //your code here
}

（将集合替换为您的真实对象/类型）

Answer 6

我正在为这些场景使用一个不错的小扩展方法：

  public static class Extensions
  {
    public static IList<T> EnsureNotNull<T>(this IList<T> list)
    {
      return list ?? new List<T>();
    }
  }

鉴于 Headers 是列表类型，您可以执行以下操作：

foreach(var h in (file.Headers.EnsureNotNull()))
{
  //do stuff
}

Answer 7

在某些情况下，我更喜欢另一种通用变体，假设默认集合构造函数通常返回空实例。

最好将此方法命名为NewIfDefault。它不仅对集合有用，所以类型约束IEnumerable<T> 可能是多余的。

public static TCollection EmptyIfDefault<TCollection, T>(this TCollection collection)
        where TCollection: class, IEnumerable<T>, new()
    {
        return collection ?? new TCollection();
    }