帮助优化此linq语句

时间:2011-09-16 04:08:03

标签: c# linq automapper

请帮我优化这个linq语句的性能。大约有4000个驱动程序要通过,页面超时。 我对Linq很新。

tblDriver driver = Mapper.Map<Domain.Driver, tblDriver>(driverDto);

var entries = (from e in driver.tblDriverLogEntries
                where e.OccuredAt <= utcEnd &&
                e.OccuredAt >= utcViolationsStart &&
                e.tblDriverStatusChanges.Where(x => !x.RemovedAt.HasValue).Count() > 0
                select e).Union(
                    //previous amended status
                    (from e in driver.tblDriverLogEntries
                        where e.tblDriverStatusChanges.Where(s => !s.RemovedAt.HasValue).Count() > 0
                        && e.OccuredAt < utcViolationsStart
                        orderby e.OccuredAt descending
                        select e).Take(1).Union(
                            //next amended status
                            (from e in driver.tblDriverLogEntries
                                where e.tblDriverStatusChanges.Where(s => !s.RemovedAt.HasValue).Count() > 0
                                && e.OccuredAt > utcEnd
                                orderby e.OccuredAt ascending
                                select e)
                            )
                    );

1 个答案:

答案 0 :(得分:4)

where e.tblDriverStatusChanges.Where(x => !x.RemovedAt.HasValue).Count() > 0

当您使用Count() > 0时,您正在使用Any():这将为您提供至少一些加速(在您的查询中出现3次):

 where e.tblDriverStatusChanges.Any(x => !x.RemovedAt.HasValue)

您可能还想预先定义并获取此查询的结果,因此您不必重新查询3次,即:

var subset = (from e in driver.tblDriverLogEntries 
             where e.tblDriverStatusChanges.Any(x => !x.RemovedAt.HasValue)
             select e).ToList();

 var entries = (from e in subset 
                where e.OccuredAt <= utcEnd &&
                e.OccuredAt >= utcViolationsStart 
                select e).Union(
                //previous amended status
                (from e in subset
                 where e.OccuredAt < utcViolationsStart
                 orderby e.OccuredAt descending
                 select e).Take(1).Union(
                 //next amended status
                 (from e in subset
                  where e.OccuredAt > utcEnd
                  orderby e.OccuredAt ascending
                  select e)));