这是resharper中的错误吗？

Question

我这里有一些 Resharper 曲线。

他们告诉我，我有一个可能的多个 IEnumerable 枚举。但是，您可以看到这是不正确的。 final 明确声明为列表 (List<Point2D>)，pointTangents 之前声明为 List<PointVector2D>

知道为什么 Resharper 会告诉我这个吗？

编辑实验以查看是否可以使用更简单的代码进行复制

正如您在下面看到的那样，即使 Bar 被声明为采用 IEnumerable 作为参数，也没有曲线和警告。

Answer 1

看起来很像RSRP-429474 False-positive warning for possible multiple enumeration：

我有这个代码：

List<string> duplicateLabelsList = allResourcesLookup.SelectMany(x => x).Select(x => x.LoaderOptions.Label).Duplicates<string, string>().ToList(); ; 
if (duplicateLabelsList.Any()) 
throw new DuplicateResourceLoaderLabelsException(duplicateLabelsList);

对于duplicateLabelsList 的两种用法，我都收到了警告 可能的多个枚举，尽管我已经调用了 ToList 和 因此不应该有多重枚举。

（当前）有一个 9.2 的修复版本，（当前）尚未发布。

Answer 2

为IEnumberable<TSource>类型定义了扩展方法public static TSource Last<TSource>(this IEnumerable<TSource> source);。

如果看一下Last<TSource>的实现：

public static TSource Last<TSource>(this IEnumerable<TSource> source)
{
    if (source == null) throw Error.ArgumentNull("source");
    IList<TSource> list = source as IList<TSource>;
    if (list != null)
    {
         int count = list.Count;
         if (count > 0) return list[count - 1];
    }
    else
    {
         using (IEnumerator<TSource> e = source.GetEnumerator()) 
         {
              if (e.MoveNext())
              {
                    TSource result;

                    do
                    {
                       result = e.Current;
                    } while (e.MoveNext());

                    return result;
              }
         }
    }
    throw Error.NoElements();
}

很明显，如果source 实现了IList，则不会枚举source，因此您认为这是Resharper 中的“错误”是正确的。

我认为它更像是误报，可能是因为 Resharper 没有一般的方法知道Last() 的实现避免了不必要的枚举。它可能决定根据 Last<TSource> 是为类型化 IEnumerable<T> 对象定义的事实来标记潜在的多重枚举。