LINQ- Max en donde la condición

Question

Tengo una TaskWeekUI clase con esta definición:

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

y escribí esta consulta:

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

¿Es esta consulta es cierto? ¿Puedo escribir mi consulta mejor que esto?

Answer 1

Creo que desea que el uno cuya PlanProgress > 0 tiene un EndDate más reciente.

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

Answer 2

Esta consulta parece ser correcta desde el punto de vista del resultado obtenido.

Pero en su consulta interna tis.Where(p => p.PlanProgress != null && p.PlanProgress > 0).Max(w => w.EndDate) se calcula para cada elemento de colección con t.PlanProgress > 0

Así que es una mejor manera de obtener el valor máximo fuera de una consulta de la siguiente manera:

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

Yendo más lejos p. PlanProgress! = Null es siempre cierto, ya que p.PlanProgress no es de tipo Nullable. Así que nuestro código se convierte de esta manera:

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

o puede cambiar una definición de la clase y hacer p.PlanProgress de tipo anulable:

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

No es necesario comparar con PlanProgress nulo por doble es tipo de estructura, no puede ser nulo.

Si desea TaskWeekUI con Max y EndDate PlanProgress positivo Puede probar este código:

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