LINQ- Макс в котором условие

Question

У меня есть класс TaskWeekUI с этим определением:

public class TaskWeekUI { 
    public Guid TaskWeekId { get; set; } 
    public Guid TaskId { get; set; } 
    public Guid WeekId { get; set; } 
    public DateTime EndDate { get; set; } 
    public string PersianEndDate { get; set; } 
    public double PlanProgress { get; set; } 
    public double ActualProgress { get; set; } } 

и я написал этот вопрос:

TaskWeekUI ti = tis.First(t => t.PlanProgress > 0 && t.EndDate == tis.Where(p => p.PlanProgress != null && p.PlanProgress > 0).Max(w => w.EndDate)); 

Является ли это запрос верно? Могу ли я написать свой запрос лучше этого?

Answer 1

Я думаю, что вы хотите, тот, чьи PlanProgress > 0 имеет самые последние EndDate.

TaskWeekUI ti = tis.Where(t => t.PlanProgress > 0) 
        .OrderByDescending(t => t.EndDate) 
        .FirstOrDefault(); 

Answer 2

Этот запрос кажется правильным с точки зрения полученного результата.

Но во внутреннем запросе tis.Where(p => p.PlanProgress != null && p.PlanProgress > 0).Max(w => w.EndDate) вычисляется для каждого элемента в коллекции с t.PlanProgress > 0

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

var max = tis.Where(p => p.PlanProgress != null && p.PlanProgress > 0).Max(w => w.EndDate); 
tis.First(t => t.PlanProgress > 0 && t.EndDate == max); 

идти дальше с. PlanProgress! = Null всегда верно, поскольку p.PlanProgress не имеет тип Nullable. Таким образом, наш код становится так:

var max = tis.Where(p => p.PlanProgress > 0).Max(w => w.EndDate); 
    tis.First(t => t.PlanProgress > 0 && t.EndDate == max); 

Или вы можете изменить определение класса и сделать p.PlanProgress обнуляемого типа:

public class TaskWeekUI { 
    public Guid TaskWeekId { get; set; } 
    public Guid TaskId { get; set; } 
    public Guid WeekId { get; set; } 
    public DateTime EndDate { get; set; } 
    public string PersianEndDate { get; set; } 
    public double? PlanProgress { get; set; } 
    public double ActualProgress { get; set; }  
} 

var max = tis.Where(p => p.PlanProgress.HasValue && p.PlanProgress.Value > 0).Max(w => w.EndDate); 
    tis.First(t => t.PlanProgress.HasValue && t.PlanProgress.Value > 0 && t.EndDate == max); 

Answer 3

Вам не нужно сравнивать PlanProgress с нулем, потому что дважды это тип структуры, он не может быть нулевым.

Если вы хотите TaskWeekUI с Max EndDate и положительным PlanProgress Вы можете попробовать этот код:

TaskWeekUI ti = tis.Where(t => t.PlanProgress > 0).Max(w => w.EndDate);