überspringen ThenBy auf NULL festlegbare Objekte

Question

ich eine Abfrage haben, die so bestellt werden sollte:

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

Aber wenn jedes Objekt null ist (total durch Design) diese Abfrage fehlschlägt. Wie kann ich Null-Werte am Ende setzen oder die Reihenfolge überspringen, wenn das Objekt null ist?

HINZUGEFÜGT: als @ LasseV.Karlsen erwähnt ich könnte ein anderes Problem haben. Ich habe wirklich ArgumentNullException, aber der Grund war nicht hinter einem Objekt waren null (Ich sah es im Debugger und fälschlicherweise dachte, dass es mein Problem war). Der wahre Grund war, als @ RaphaëlAlthaus erwähnt, dass ich nicht IComparable<> auf jedem meiner Klassen in MonthClosureViewModel geführt habe ...

Nachdem ich es alles getan habe, beginnen, wie beabsichtigt arbeiten, auch wenn Objekt null

Answer 1

Sie versuchen, mit komplexen Eigenschaften zu bestellen, das ist das eigentliche Problem (wie Sie erwähnt).

Um es möglich zu machen Sie haben werden, entweder

1. IComparable<T> auf Ihre Klassen implementieren

2. Verwendung andere Überlastungen von OrderBy/OrderByDescending, die nehmen auch ein IComparer<TKey> als Argument (msdn für die OrderBy-Überladung)

3. Verwenden Sie eine einfache Eigenschaft der komplexen Eigenschaft in Ihrer Order by-Klausel (mit Null-Check. In diesem Fall ist die Null l Prüfung notwendig wird eine Nullreferenz Exception) zu vermeiden:

Zum Beispiel:

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

Answer 2

Sie werden defaultSite und defaultChargeSite werden entweder kleiner oder größer als alle anderen Objekte in ihrer Klasse definieren, müssen

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