2010-07-22 3 views
2

Я не самый большой с LINQ, но я пытаюсь получить все ModuleAvailabilities, где academicYear - текущий год.Улучшите этот запрос LINQ для удобства чтения и производительности?

Есть ли какие-либо улучшения, которые должны быть сделаны здесь?

pathway.PathwayFoundationModule.Attach(
    pathway.PathwayFoundationModule.CreateSourceQuery() 
     .Include("Module") 
     .Include("Module.ModuleAvailabilities.Location") 
     .Where(o => o.Module.ModuleAvailabilities 
        .Where(x => x.AcademicYear == academicYear.Current) 
        .Count() >= 0) 
); 
+6

Вы говорите, когда '.Count()> = 0 ', это не всегда> = 0? –

ответ

5

Я думаю, что вы имеете в виду

pathway.PathwayFoundationModule.Attach(
      pathway.PathwayFoundationModule.CreateSourceQuery() 
       .Include("Module") 
       .Include("Module.ModuleAvailabilities.Location") 
       .Where(o => o.Module.ModuleAvailabilities 
        .Any(x => x.AcademicYear == academicYear.Current)); 
+0

Отлично. спасибо –

Смежные вопросы