Warum sagt mir ReSharper, dass mein Ausdruck immer wahr ist?

Question

Ich habe diese LINQ-Anweisung, die Null zurückgibt, wenn die Sequenz leer ist. Ich weise das Ergebnis einem Nullwert zu DateTime. Später überprüfe ich, ob die DateTime.HasValue Eigenschaft und ReSharper mir sagt, dass der Ausdruck immer wahr ist.

DateTime? latestUploadDateInBLSO = Documents.Where(d => d.DocumentLinkId == documentLinkId && 
                    d.UploadedInStage.StageNumber == 6 && 
                    d.DocumentOwnerTeam.TeamId == AuthorUser.Team.TeamId) 
                    .Select(d => d.UploadedOnDate) 
                    .DefaultIfEmpty() 
                    .Max(); 

if (latestUploadDateInBLSO.HasValue) { // <-- Says this is always true 
        Documents.Single(d => d.DocumentLinkId == documentLinkId && 
              d.UploadedOnDate == latestUploadDateInBLSO && 
              d.UploadedInStage.StageNumber == 6 && 
              d.DocumentOwnerTeam.TeamId == AuthorUser.Team.TeamId).IsLatestVersion = true; 
       } 

Da latestUploadDateInBLSO kann null sein, wie ist das Ausdruck immer wahr?

Answer 1

Da UploadedOnDate nicht nullfähig ist, wird das Ergebnis immer ein DateTime-Wert und niemals null sein. Wenn die Liste leer ist, erhalten Sie default(DateTime), also DateTime.MinValue.

Wenn Sie möchten, dass null zurückgegeben wird, müssen Sie Ihre UploadedOnDate in DateTime? umwandeln. Sie können DefaultIfEmpty gemäß the docsMax weglassen null für eine leere Sequenz, wenn der Typ nullfähig ist.

DateTime? latestUploadDateInBLSO = Documents 
    .Where(d => d.DocumentLinkId == documentLinkId && d.UploadedInStage.StageNumber == 6 && d.DocumentOwnerTeam.TeamId == AuthorUser.Team.TeamId) 
    .Select(d => (DateTime?)d.UploadedOnDate) 
    .Max(); 

Answer 2

Es ist wegen des Anrufs DefaultIfEmpty mit einer Folge von Nicht-Nullable-Elementen kombiniert (DateTime) - Sie sagen, wenn die Sammlung von den Where und Select zurückgekehrt ist leer, stattdessen zurückgeben eine Sammlung mit einer einzigen notleidenden DateTime innerhalb es, so wird es nie zurückkehren null.

Hier ist eine kleine Probe mit der Ausgabe von LINQPad:

List<DateTime> l = new List<DateTime>(); 
DateTime x = l.DefaultIfEmpty().Max(); 
x.Dump(); 

var y = new DateTime(); 
y.Dump(); 

l.DefaultIfEmpty().Dump(); 

Answer 3

Wenn uploadedOnDate auch von Datetime-Typ ist, dann ist es nicht NULL. Der Standardwert für DateTime entspricht DateTime.MinValue. Aus diesem Grund hat Ihr Nullwert immer einen Wert. Wenn Sie dies ändern möchten, müssen Sie explizit über DefaultIfEmpty sagen und NULL als Standardwert zurückgeben.

Answer 4

DefaultIfEmpty initialisiert wahrscheinlich Ihr DateTime-Objekt auf seinen Standardwert DateTime.MinValue, so dass es niemals null ist und HasValue daher immer true zurückgibt.