if（items！= null）在foreach（item中的T项目）之前是否多余？

我经常遇到如下代码：

if ( items != null)
{
   foreach(T item in items)
   {
        //...
   }
}

基本上，该if条件确保foreach仅在items不为null 的情况下才执行块。我想知道是否if确实需要该条件，或者如果满足，foreach将解决该情况items == null。

我的意思是，我可以简单地写吗

foreach(T item in items)
{
    //...
}

不用担心是否items为null？是if条件多余的？还是取决于类型，items或者取决于T？

您仍然需要检查（items！= null），否则将获得NullReferenceException。但是，您可以执行以下操作：

List<string> items = null;  
foreach (var item in items ?? new List<string>())
{
    item.Dump();
}

但您可以检查其性能。所以我仍然更喜欢首先使用if（items！= null）。

基于Eric的Lippert建议，我将代码更改为：

List<string> items = null;  
foreach (var item in items ?? Enumerable.Empty<string>())
{
    item.Dump();
}

使用C＃6，您可以将新的null条件运算符与List<T>.ForEach(Action<T>)（或您自己的IEnumerable<T>.ForEach扩展方法）一起使用。

List<string> items = null;
items?.ForEach(item =>
{
    // ...
});

真正的收获应该是一个序列，首先应该几乎不为null。只需使它在所有程序中不变即可，如果您有序列，则它永远不会为null。它始终被初始化为空序列或其他一些真实序列。

如果一个序列永远不会为空，那么显然您不需要检查它。

实际上，@ Connect上有一个功能请求：http : //connect.microsoft.com/VisualStudio/feedback/details/93497/foreach-should-check-for-null

响应非常合乎逻辑：

我认为大多数foreach循环都是出于迭代非null集合的目的而编写的。如果尝试通过null进行迭代，则应获取异常，以便可以修复代码。

您总是可以使用空列表进行测试...但这就是我在msdn网站上找到的内容

foreach-statement:
    foreach   (   type   identifier   in   expression   )   embedded-statement 

如果expression的值为null，则抛出System.NullReferenceException。

它不是多余的。在运行时，项目将强制转换为IEnumerable，并将调用其GetEnumerator方法。这将导致取消引用将失败的项目

您可以将null检查封装在扩展方法中，并使用lambda：

public static class EnumerableExtensions {
  public static void ForEach<T>(this IEnumerable<T> self, Action<T> action) {
    if (self != null) {
      foreach (var element in self) {
        action(element);
      }
    }
  }
}

代码变为：

items.ForEach(item => { 
  ...
});

如果您只想调用一个接受项并返回的方法，则可以更加简洁void：

items.ForEach(MethodThatTakesAnItem);

您确实需要这个。当foreach访问容器以其他方式设置迭代时，您将获得异常。

在幕后，foreach使用在集合类上实现的接口来执行迭代。通用等效接口在这里。

C＃语言的foreach语句（对于Visual Basic中的每个语句）隐藏了枚举数的复杂性。因此，建议使用foreach而不是直接操作枚举器。

该测试是必需的，因为如果集合为null，则foreach将抛出NullReferenceException。试用起来其实很简单。

List<string> items = null;
foreach(var item in items)
{
   Console.WriteLine(item);
}

第二个将抛出一条NullReferenceException消息Object reference not set to an instance of an object.

如此处所述，您需要检查它是否不为空。

不要使用计算结果为null的表达式。

在C＃6中，您可以这样写：

// some string from file or UI, i.e.:
// a) string s = "Hello, World!";
// b) string s = "";
// ...
var items = s?.Split(new char[] { ',', '!', ' ' }) ?? Enumerable.Empty<string>();  
foreach (var item in items)
{
    //..
}

基本上是Vlad Bezden的解决方案，但使用?? 表达式始终生成不为null的数组，因此可以在foreach中生存，而不是在foreach括号内进行此检查。