Помогите оптимизировать этот оператор linq - PullRequest
1 голос
/ 16 сентября 2011

Пожалуйста, помогите мне оптимизировать этот оператор 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 Ответ

4 голосов
/ 16 сентября 2011
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)));
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...