Enumerable.Any()与多次枚举疑问：Rider/ReSharper警告是否误报？

这个警告绝对不是误报，可不能轻易忽略它！

先把核心逻辑理清楚：IEnumerable<string>只是一个“可枚举”的抽象契约，它本身并不是像List、Array那样的具体集合。如果传入的productCodes是一个延迟执行的查询——比如LINQ的Where/Select结果，或者你自己写的带副作用的迭代器——那每次调用枚举相关的方法（比如Any()，或者DoStuff里如果遍历了这个集合），都会从头开始执行整个枚举逻辑。

举个真实场景的例子：如果productCodes是从数据库查出来的dbContext.Products.Select(p => p.Code)，那第一次调用Any()会触发一次数据库查询，等DoStuff里再遍历这个集合时，又会触发第二次查询——这不仅白白浪费性能，还可能因为数据库数据变化，导致两次枚举的结果不一致，埋下bug隐患。

再说说Any()函数本身：它确实会触发枚举，但它是短路执行的——只要找到第一个元素就停止遍历，不会像ToList()那样走完全部元素。但不管怎样，它已经完成了一次枚举操作，而DoStuff如果也需要遍历集合的话，就是第二次枚举了。

想要解决这个问题也很简单，先把IEnumerable转换成具体的集合即可，比如：

public void ProcessProductCodes(IEnumerable<string> productCodes) 
{
    var codeList = productCodes.ToList();
    if (codeList.Any()) 
    {
        DoStuff(codeList);
    }
}

这样只会枚举一次，既消除了警告，也避免了潜在的性能问题和意外行为。

内容的提问来源于stack exchange，提问作者Andre Tchernikov