LINQ- Max w którym warunek

Question

Mam klasy TaskWeekUI z tej definicji:

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; } } 

i napisałem to zapytanie:

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

Czy ta kwerenda jest prawdą? Czy mogę napisać moje zapytanie lepiej niż to?

Answer 1

Myślę, że chcesz tego, którego PlanProgress > 0 ma najnowsza EndDate.

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

Answer 2

To zapytanie wydaje się poprawne z punktu widzenia uzyskanego wyniku.

Ale w swoim wewnętrznym zapytania tis.Where(p => p.PlanProgress != null && p.PlanProgress > 0).Max(w => w.EndDate) jest obliczana dla każdego elementu w kolekcji z t.PlanProgress > 0

więc jego lepszym sposobem na uzyskanie maksymalnej wartości poza kwerendy następująco:

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

Idąc dalej str. PlanProgress! = Null jest zawsze prawdziwe, ponieważ p.PlanProgress nie ma typu Nullable. Więc nasz kod staje się tak:

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

Albo można zmienić definicję klasy i uczynić p.PlanProgress typu pustych:

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

Nie trzeba porównać PlanProgress NULL ponieważ podwójna jest typu struct, nie może mieć wartości NULL.

Jeśli chcesz TaskWeekUI z max EndDate i pozytywne PlanProgress Można spróbować tego kodu:

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