LINQ (метод) возвращение несколько записей

Question

Есть две модели, Case и Client:

public class Case 
{ 
    public int Id { get; set; } 
    public string CaseNumber { get; set; } 
    public string Caption { get; set; } 
    public int ClientId { get; set; } 
    public Client Client { get; set; } 
    public string Status { get; set; } 
} 

public class Client 
{ 
    public int Id { get; set; } 
    public string LastName { get; set; } 
    public string FirstName { get; set; } 
    public string PrimaryPhone { get; set; } 
    public string Email { get; set; } 
} 

настоящее время я использую следующий LINQ вызова метода для запроса эти структуры, так что я могу получить список Идентификатор клиента и имя-х на основе статуса любого случая, связанного с ними

public ActionResult Index(string status) 
    { 
     var query = _context.Cases 
      .Include(c => c.Client); 
     if (status != "all") 
     { 
      query = query.Where(c => c.Status == status); 
     } 

     var cases = query.ToList(); 

     return View(cases); 
    } 

Моя проблема заключается в том, что если у меня есть несколько случаев для того же клиента, он получает, что идентификатор и имя несколько раз.

Я подхожу к этому с неправильного угла или что предлагается исправить эту проблему?

Answer 1

Что делать, если вы используете Distinct() как

if (status != "all") 
    { 
     query = query.Where(c => c.Status == status).Distinct(); 
    } 

Answer 2

Вы можете использовать GroupBy несколько свойств, используя тип Анонимус как в this question, где вы можете использовать Id и Name в типе Анонимус. В конце запроса Вы можете добавить: .GroupBy(x => new { x.Id, x.Name })

var cases = query.ToList().GroupBy(x => new { x.Id, x.Name }).Select(y => y.First()).ToList(); 

Answer 3

я решил мою проблему, добавив запись в var cases = query.ToList();, а затем вызвать метод Distinct().

Так что мой окончательный код выглядит следующим образом:

var cases = query.Select(c => c.Client).ToList().Distinct().ToList(); 

@Rahul - Если бы правильное понятие, как раз в том месте, но он указал мне вниз правильный путь. Благодаря!

@meJustAndrew - Мне понравился подход, но он все еще повесился в заявлении @model. Если вы можете добавить какие-либо комментарии о том, где я ошибся, это было бы здорово. Благодаря!

Answer 4

Если Вы добавляете свойство Cases к Client вы можете сделать запрос более экономичен:

public ActionResult Index(string status) 
{ 
    IQueryable<Client> query = _context.Clients; 
    if (status != "all") 
    { 
     query = query.Where(c => c.Cases.Any(cs => cs.Status == status)); 
    } 

    var clients = query.ToList(); 
    // or query.Select(c => new { c.Id, c.Name }) ? 

    return View(clients); 
} 

Это позволяет избежать относительно дорогой Include и обработки, чтобы получить уникальный клиент.

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