LINQ- Max ở nơi mà điều kiện

Question

Tôi có một lớp TaskWeekUI với định nghĩa này:

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

và tôi đã viết truy vấn này:

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

là truy vấn này là đúng? Tôi có thể viết truy vấn của mình tốt hơn điều này không?

Answer 1

Tôi nghĩ rằng bạn muốn một mà PlanProgress > 0 có gần đây nhất EndDate.

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

Answer 2

Truy vấn này có vẻ đúng từ quan điểm của kết quả thu được.

Nhưng trong truy vấn bên trong của bạn tis.Where(p => p.PlanProgress != null && p.PlanProgress > 0).Max(w => w.EndDate) được tính cho mỗi phần tử trong bộ sưu tập với t.PlanProgress > 0

Vì vậy, một cách tốt hơn của nó để có được giá trị Max bên ngoài của một truy vấn như sau:

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

Đi p hơn nữa. PlanProgress! = Null là allways true vì p.PlanProgress không thuộc kiểu Nullable. Vì vậy, mã của chúng tôi trở nên như thế này:

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

Hoặc bạn có thể thay đổi định nghĩa về lớp học của bạn và làm cho p.PlanProgress loại Nullable:

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

Bạn không cần phải so sánh với PlanProgress null vì đôi là kiểu struct, nó không thể là null.

Nếu bạn muốn TaskWeekUI với Max EndDate và PlanProgress tích cực Bạn có thể thử mã này:

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