Question

不重复：我不认为这是重复的，因为在我的具体情况下我觉得忽略警告实际上更好。

例如，

IEnumerable<Item> milionItems = GetAMillionItemsFromDatabase();

var item1 = millionItems.FirstOrDefault(x=> x.Condition == "Excellent");

var item2 = millionItems.FirstOrDefault(x=> x.Condition == "Good");

我在'millionItems'下面收到了警告信息，我知道这意味着什么但是我不确定是否总是值得ToList只是为了摆脱它。

GetAMillionItemsFromDatabase().ToList();

这看起来非常糟糕，因为它会同时将一百万条记录带入内存。

但是，如果我不这样做并继续枚举IEnumerable，即使它会两次打到数据库，它也不会带来所有数据，因为它会找到第一个匹配项并返回。在这种情况下，在我看来，实际上忽略该消息更好。

Answer 1

很有可能在这种情况下两次转到DB会比通过当前代码执行的IEnumerable的客户端搜索更好。

如果您无法将搜索推送到数据库（即保持IQueryable<Item>允许链接），您仍然可以通过检查每个项目的两个条件来优化查找：

  foreach(var x in millionItems)
  {
     item1 = item1 == null && x=> x.Condition == "Excellent" ? x : item1;
     item2 = item2 == null && x=> x.Condition == "Good" ? x : item2;

     if (item1 != null && item2 != null)
     {
           break;
     }
  }

这很有可能通过客户端的很多项目，但至少它不会同时保留在内存中。

使用ToList转换为列表不太可能更好，如果这些只是您需要构建的2个查询。

Resharper的'可能的多次枚举IEnumerable警告'

1 个答案: