1

У меня 3 таблицы, посты и кто эти посты выложил(человек или группа):

public class Post
{
    [Key]
    public long Id { get; set; }
    //some code
    public long? UserId { get; set; }
    public User User { get; set; }
    public long? GroupId { get; set; }
    public Group Group { get; set; }
}

public class User
{
    [Key]
    public long Id { get; set; }
    // some code
    public List<Post> Posts {get;set;}
    public User()
    {
        Posts = new List<Post>();
    }
}

public class Group
{
    [Key]
    public long Id { get; set; }
    public string NameGroup { get; set; }
    // some code
    public List<Post> Posts {get;set;}
    public User()
    {
        Posts = new List<Post>();
    }
}

Условно есть такие списки:

List<long> UserIds = { 645, 2456, 9673456, 635 };
List<long> GroupIds = { 3275, 7904, 234 };

Как мне на основе таких списков сделать выборку в единый список постов с помощью методов расширения (LINQ). А то в голову приходит только такая реализация:

List<Post> Posts = await db.Posts
.Include(d=>d.Group)
.Include(e=>e.User)
.Where(r=>UserIds.Contains(r.UserId)&& GroupIds.Contains(r.GroupId))
.AsNoTracking()
.OrderBy(d => d.DatePost)
.ToListAsync();

но она это не работает(

4
  • .Where(r=>UserIds.Contains(r.UserId)&& GroupIds.Contains(r.GroupId)) замените И на ИЛИ Commented 27 авг. 2019 в 11:47
  • У вас EF/EFCore? Commented 27 авг. 2019 в 11:48
  • @АндрейNOP EF Core, спасибо заработало
    – Yarik08
    Commented 27 авг. 2019 в 11:52
  • @АндрейNOP, я так понимаю у меня проблема была в том, что постов от юзеров могло не быть и поэтому выборка возвращала пустой список?
    – Yarik08
    Commented 27 авг. 2019 в 11:54

1 ответ 1

3

У вас логическая ошибка в запросе, вы подразумеваете ИЛИ, а пишете И здесь:

Where(r => UserIds.Contains(r.UserId) && GroupIds.Contains(r.GroupId))

Логическое И возвращает True только в том случае, когда оба операнда истинны, в вашем конкретном случае это в принципе невозможно, т. к. если пост имеет UserId, то у него GroupId == null, поэтому второй операнд заведомо ложен. И наоборот.

1
  • всё я понял, спасибо большое)
    – Yarik08
    Commented 27 авг. 2019 в 12:07

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.