跳过然后在可空的对象上

Question

我有一个查询，应该这样订购：

var list = new List<MonthClosureViewModel>();
var orderedList = list
    .OrderByDescending(x => x.Project)
    .ThenByDescending(x => x.ChargeLine)
    .ThenByDescending(x => x.DomesticSite) //<- x.DomesticSite might be null sometimes
    .ThenByDescending(x => x.ChargeSite)   //<- x.ChargeSite   might be null sometimes
    .ThenByDescending(x => x.RateGroup)
    .ThenByDescending(x => x.ApprovedHrs)
    .ThenByDescending(x => x.NotApprovedHrs);

public class MonthClosureViewModel
{
    public Project Project { get; set; }
    public ChargeLine ChargeLine { get; set; }
    public Site DomesticSite { get; set; }
    public Site ChargeSite { get; set; }
    public RateGroup RateGroup { get; set; }
    public decimal Rate { get; set; }
    public decimal ApprovedHrs { get; set; }
    public decimal NotApprovedHrs { get; set; }
}

但是如果任何对象为null（完全按设计），则此查询失败。 如果object为null，我如何在结尾处放置空值或跳过排序？

添加： @ LasseV.Karlsen提到我可能有另一个问题。 我真的得到了ArgumentNullException ，但原因并不是后面的某个对象是null （我在调试器中看到它并错误地认为这是我的问题）。 真正的原因是@RaphaëlAlthaus提到我没有在MonthClosureViewModel任何类中实现IComparable<> ...

完成后，即使object为null ，一切都按预期开始工作

Answer 1

您需要将defaultSite和defaultChargeSite定义为比其他类中的所有其他对象更小或更大

var list = new List<MonthClosureViewModel>();
var orderedList = list
     .Where(x => x != null);
    .OrderByDescending(x => x.Project)
    .ThenByDescending(x => x.ChargeLine)
    .ThenByDescending(x => x.DomesticSite==null?defaultSite:x.DomesticSite) //<- x.DomesticSite might be null sometimes
    .ThenByDescending(x => x.ChargeSite==null?defaultChargeSite:x.ChargeSite)   //<- x.ChargeSite   might be null sometimes
    .ThenByDescending(x => x.RateGroup)
    .ThenByDescending(x => x.ApprovedHrs)
    .ThenByDescending(x => x.NotApprovedHrs);

Answer 2

您尝试按复杂属性排序，这是（如您所提到的）真正的问题。

为了使你成为可能，你也必须这样做

1. 在你的类上实现IComparable<T>

2. 使用OrderBy / OrderByDescending的其他重载，它还将IComparer<TKey>作为参数（OrderBy重载的msdn ）

3. 在order by子句中使用complex属性的简单属性（使用null检查。在这种情况下，必须进行null检查以避免Null引用异常）：

例如 ：

.OrderByDescending(x => x.Project == null ? string.Empty : x.Project.Name)