Resharper：IEnumerable 的可能多重枚举

Question

我正在使用新的 Resharper 6

我明白这意味着什么，并在适当的时候听取了建议，但在某些情况下，我不确定这是否真的很重要。

如下代码：

var properties = Context.ObjectStateManager.GetObjectStateEntry(this).GetModifiedProperties();
if (properties.Contains("Property1") || properties.Contains("Property2") || properties.Contains("Property3")) {
    ...
}

它在第二行每一次提到properties 都加了下划线，警告我正在多次枚举这个 IEnumerable。

如果我将 .ToList() 添加到第 1 行的末尾（将 properties 从 IEnumerable<string> 转换为 List<string>），警告就会消失。

但是可以肯定的是，如果我将其转换为 List，那么它将首先枚举整个 IEnumerable 以构建 List，然后根据需要枚举 List 以查找属性（即 1 个完整枚举，以及3个部分枚举）。而在我的原始代码中，它只进行了 3 个部分枚举。

我错了吗？这里最好的方法是什么？

Answer 1

如果您在 IEnumerable 上调用 Contains()，它将调用扩展方法，该方法将遍历项目以找到它。 IList 对 Contains() 有真正的实现，这可能比通过值的常规迭代更有效（它可能有一个带有哈希的搜索树？），因此它不会警告 IList。

由于扩展方法只会知道它是IEnumerable，因此它可能无法使用Contains() 的任何内置方法，即使理论上可以识别已知类型并按顺序相应地转换它们使用它们。

Answer 2

我不知道你的 properties 到底是什么 - 但如果它本质上代表一个未实现的数据库查询，那么你的 if 语句将执行三个查询。

我怀疑这样做会更好：

string[] propertiesToFind = { "Property1", "Property2", "Property3" };
if (properties.Any(x => propertiesToFind.Contains(x))
{
     ...
}

这将逻辑上只迭代序列一次 - 如果涉及数据库查询，它很可能只使用 SQL“IN”子句在数据库中完成所有操作一个查询。