2010-07-22 8 views
2

Je ne suis pas le meilleur avec LINQ, mais j'essaie de récupérer tous les ModuleAvailabilitiesacademicYear est l'année en cours.Améliorer cette requête LINQ pour la lisibilité et la performance?

Y a-t-il des améliorations à apporter ici?

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

Vous dites quand le '.Count()> = 0', ne l'est pas toujours> = 0? –

Répondre

5

Je pense que vous voulez dire

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

Parfait. Je vous remercie –

Questions connexes