我正在迭代一个包含大约1000个元素的匿名类型。
这里的问题是我的循环可能需要几乎3秒才能完成,而循环内部发生的时间不到1毫秒。有一千个元素,我认为循环必须在第二个内完成,而不是3个。
有没有办法让它迭代更快?
// takes 1ms to complete
var x = tt.Where(p => p.Methods.Count() > 0 && p.PerWeek != this.Project.WorkDaysCount && !p.IsManual);
// takes almost 3 seconds to complete
foreach (var item in x)
{
// do stuff that takes < 1 ms
}
答案 0 :(得分:15)
两个即时建议:
p.Methods.Count() > 0
- 需要执行完整计数,即使您只需要知道是否有任何元素。请改用p.Methods.Any()
。this.Project.WorkDaysCount
。我们不知道那里发生了什么的细节,但它可能很昂贵。预先计算它,并使用常量。以下是改进的查询代码:
int workDaysCount = Project.WorkDaysCount;
var x = tt.Where(p => p.Methods.Any() &&
p.PerWeek != workDaysCount &&
!p.IsManual);
正如其他人所说,查询构建本身没有花费任何重要时间的原因是它没有做任何实际工作。但是,当然知道这样做不会更快:)
除此之外,我们还需要了解更多有关背景的信息。这是LINQ to Objects,LINQ to SQL还是其他什么?什么是tt
的类型?
答案 1 :(得分:14)
Linq使用延迟执行。在有人使用返回的IEnumerable之前,您的linq查询实际上不会执行。您看到的执行时间是查询的结果,而不是foreach。
答案 2 :(得分:7)
这是延迟执行。
写x.ToList();
,也需要3秒钟。
答案 3 :(得分:1)
在实现它之前,不会评估您的IEnumerable(x)