为什么 resharper 在迭代列表时不建议使用 linq？

Question

我在 Visual Studio 2017 中使用 Resharper 2018.1.3（我更新到 2018.2.0 并且遇到了同样的问题）。

当我输入此代码时：

var test = new List<string>();
var strings = new[] { "a", "b", "c" };
foreach (var a in strings)
{
    test.Add(a);
}

Resharper 建议将其转换为 LINQ 表达式。

但如果我使用List:

var test = new List<string>();
var strings = new List<string> { "a", "b", "c" };
foreach (var a in strings)
{
    test.Add(a);
}

Resharper 没有给我任何提示。为什么？

Answer 1

虽然在这种特殊情况下转换为 LINQ 看起来很安全，但通常它可能会产生意想不到的副作用。 System.Collections.Generic.List 有一个 GetEnumerator 方法的特殊实现，它返回一个结构。 LINQ 方法适用于IEnumerable，其中GetEnumerator 方法返回接口IEnumerator 的实例。因此，将foreach 转换为 LINQ 可能会导致意外装箱。

在某些情况下，foreach 使用的GetEnumerator 方法和IEnumerable.GetEnumerator 的实现可能会返回完全不同的Enumerators，就像我们在IDictionary 中所做的那样，其中GetEnumerator 枚举DictionaryEntrys 而实现@ 987654334@ 在 System.Collections.Generic.Dictionary 的情况下迭代实际上是 KeyValuePair 实例的对象。