Linq bizarrerie

Question

je recevais des lignes de répéter cette requête LINQ:

   public static Func<DataContext, string, IQueryable<Building>> 
      GearFilteredBuildings = 
       CompiledQuery.Compile((DataContext db, string filter) => 
        from b in db.Building 
         join r in db.Router on b equals r.Building 
        orderby !b.Active 
        where filter.Length == 5 && r.Name.Substring(1, 5).ToLower() == filter 
         || filter.Substring(0, 3) == r.Name.Substring(3, 3).ToLower() 
        select b); 

Après quelques tripoter, je suis les bâtiments distincts avec ce:

 public static Func<DataContext, string, IQueryable<Building>> 
      GearFilteredBuildings = 
       CompiledQuery.Compile((DataContext db, string filter) => 
        (from b in db.Building 
        join r in db.Router on b equals r.Building 
        orderby !b.Active 
        where filter.Length == 5 && r.Name.Substring(1, 5).ToLower() == filter 
         || filter.Substring(0, 3) == r.Name.Substring(3, 3).ToLower() 
        group b by b.Id into g 
         select g) as IQueryable<Building>); 

Est-ce une solution acceptable? Sinon, comment cela pourrait-il être fait?

Answer 1

Je ne sais pas (ne pouvait pas tester IDE) mais select...join...lalala peut remplacer par la syntaxe LINQ-chaîne:

db.Routers 
    .Where(r => filter.Length == 5 && r.Name.Substring(1, 5).ToLower() == filter || filter.Substring(0, 3) == r.Name.Substring(3, 3).ToLower()) 
    .GroupBy(r => r.Building) 
    .Select(g => g.Key) 
    .OrderBy(b => !b.Active) 

aussi: que je ne vois pas join s sont vraiment requred dans votre requête, que vous avez propriétés de navigation (r.Building) dans votre modèle.

Ou une autre approche pourrait être utilisée, sélectionner tous les bâtiments nécessaires, et utiliser .Distinct() après:

db.Routers 
    .Where (r => filter.Length == 5 && r.Name.Substring(1, 5).ToLower() == filter || filter.Substring(0, 3) == r.Name.Substring(3, 3).ToLower()) 
    .Select(r => r.Building) 
    .Distinct() 
    .OrderBy(b => !b.Active)