Составьте это в одно утверждение?

Question

У меня есть база данных, которая включает в себя один ко многим внешним ключом, например

Table {shop 
[Id] 
...} 

Table {user 
[Id] 
[Shop_id] 
[Archived] 
} 

У меня также есть метод, как показано ниже

public IEnumerable<shop> GetShopDetails(int shopId) 
{ 
    var foo = (from s in context.Shops 
    where s.Id = shopId 
    select s).ToList(); 

    return foo; 
} 

Как следствие этого, он будет возвращать все пользователи для это магазин. В большинстве случаев я только хочу, чтобы пользователи не архивировались.

Есть ли способ записать это в одно утверждение, поэтому я могу пройти, скажем, второй параметр includeArchived и использовать это, чтобы определить, верну ли я всех пользователей или только те, которые являются активными.

В настоящее время я могу заставить его работать либо путем добавления метода к моему объекту магазина, который возвращает подмножество пользователей, либо я могу загрузить магазин, получить его идентификатор, а затем создать отдельную коллекцию пользователей, которые содержат соответствующий fk, но любой метод кажется мне немного неуклюжим.

Answer 1

Просто условно добавить еще Where условие:

public IQueryable<user> GetShopUsers(int shopId, bool includeArchived = false) 
{ 
    var foo = from u in context.Users 
       where u.Shop_id = shopId 
       select u; 

    if(!includeArchived) 
     foo = foo.Where(u => !u.Archived); 

    return foo; 
} 

Answer 2

Я думаю, это то, что вы хотите? Вы можете интегрировать другой логический параметр в свой запрос так же, как вы уже делали с shopId.

public IQueryable<user> GetUsers(int shopId, bool includeArchived) 
{ 
    return from user in context.Users 
      where user.Shop_id = shopId 
      where includeArchived || !user.Archived 
      select user; 
} 

UPDATE: Не уверен, что вы можете отфильтровать собственную коллекцию пользователей Shop Entity. Можно попробовать запрос построение анонимного объекта:

var foo = from s in context.Shops 
      where s.Id = shopId 
      select new { 
       Shop = s, 
       Users = s.Users.Where(u => includeArchived || !u.Archived) 
      };