所以我经常遇到这种情况……Do.Something(…)返回一个空集合,如下所示:

int[] returnArray = Do.Something(...);

然后,我尝试像这样使用这个集合:

foreach (int i in returnArray)
{
    // do some more stuff
}

我只是好奇,为什么foreach循环不能操作一个空集合?在我看来,零次迭代将被一个空集合执行是合乎逻辑的……相反,它抛出一个NullReferenceException。有人知道为什么吗?

这很烦人,因为我使用的api不清楚它们返回什么,所以我最终到处都是if (someCollection != null)。


当前回答

我认为这里提供的答案对为什么抛出异常的解释非常清楚。我只是想补充一下我通常处理这些系列的方式。因为,有些时候,我不止一次地使用集合,并且每次都必须测试是否为空。为了避免这种情况,我采取了以下措施:

    var returnArray = DoSomething() ?? Enumerable.Empty<int>();

    foreach (int i in returnArray)
    {
        // do some more stuff
    }

通过这种方式,我们可以尽可能多地使用集合,而不用担心异常,也不会用过多的条件语句污染代码。

使用空检查操作符?。也是一个很好的方法。但是,在数组的情况下(就像问题中的例子),它应该在下面转换为List:

    int[] returnArray = DoSomething();

    returnArray?.ToList().ForEach((i) =>
    {
        // do some more stuff
    });

其他回答

这是Do.Something()的错。这里的最佳实践是返回一个大小为0的数组(这是可能的),而不是null。

另一个扩展方法来解决这个问题:

public static void ForEach<T>(this IEnumerable<T> items, Action<T> action)
{
    if(items == null) return;
    foreach (var item in items) action(item);
}

以几种方式消费:

(1)采用接受T的方法:

returnArray.ForEach(Console.WriteLine);

(2)用表达式表示:

returnArray.ForEach(i => UpdateStatus(string.Format("{0}% complete", i)));

(3)采用多行匿名方法

int toCompare = 10;
returnArray.ForEach(i =>
{
    var thisInt = i;
    var next = i++;
    if(next > 10) Console.WriteLine("Match: {0}", i);
});
SPListItem item;
DataRow dr = datatable.NewRow();

dr["ID"] = (!Object.Equals(item["ID"], null)) ? item["ID"].ToString() : string.Empty;

简单的回答是“因为编译器设计者就是这么设计的。”但实际上,您的集合对象是空的,因此编译器无法让枚举器循环遍历集合。

如果你真的需要这样做,试试空合并运算符:

int[] array = null;

foreach (int i in array ?? Enumerable.Empty<int>())
{
   System.Console.WriteLine(string.Format("{0}", i));
}

我认为这里提供的答案对为什么抛出异常的解释非常清楚。我只是想补充一下我通常处理这些系列的方式。因为,有些时候,我不止一次地使用集合,并且每次都必须测试是否为空。为了避免这种情况,我采取了以下措施:

    var returnArray = DoSomething() ?? Enumerable.Empty<int>();

    foreach (int i in returnArray)
    {
        // do some more stuff
    }

通过这种方式,我们可以尽可能多地使用集合,而不用担心异常,也不会用过多的条件语句污染代码。

使用空检查操作符?。也是一个很好的方法。但是,在数组的情况下(就像问题中的例子),它应该在下面转换为List:

    int[] returnArray = DoSomething();

    returnArray?.ToList().ForEach((i) =>
    {
        // do some more stuff
    });