linq foreach表现

Question

我正在迭代一个包含大约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
                    }

Answer 1

两个即时建议:

• 不要使用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？

Answer 2

Linq使用延迟执行.在有人使用返回的IEnumerable之前,您的linq查询实际上不会执行.您看到的执行时间是查询的结果,而不是foreach.

Answer 3

这是延期执行.写x.ToList();,它也需要约3秒.