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

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

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

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

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

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


当前回答

foreach循环调用GetEnumerator方法。 如果集合为空,此方法调用将导致NullReferenceException。

返回一个空集合是不好的做法;你的方法应该返回一个空的集合。

其他回答

foreach循环调用GetEnumerator方法。 如果集合为空,此方法调用将导致NullReferenceException。

返回一个空集合是不好的做法;你的方法应该返回一个空的集合。

空集合和对集合的空引用之间有很大区别。

当你在内部使用foreach时,这会调用IEnumerable的GetEnumerator()方法。当引用为空时,将引发此异常。

但是,使用空IEnumerable或IEnumerable< t>是完全有效的。在这种情况下,foreach将不会“迭代”任何东西(因为集合是空的),但它也不会抛出,因为这是一个完全有效的场景。


编辑:

就个人而言,如果你需要解决这个问题,我会推荐一个扩展方法:

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

然后你可以调用:

foreach (int i in returnArray.AsNotNull())
{
    // do some more stuff
}
SPListItem item;
DataRow dr = datatable.NewRow();

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

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

因为在幕后,foreach获取了一个枚举数,等价于:

using (IEnumerator<int> enumerator = returnArray.getEnumerator()) {
    while (enumerator.MoveNext()) {
        int i = enumerator.Current;
        // do some more stuff
    }
}