年月比较最好

Question

我编写了这个逻辑来比较源年/月和响应年份/月。但我想用更好的方式写。如果源年/月大于或等于响应年份/月，则传递。有什么建议吗？

if (source?.Year == response?.Year)
{
    return source?.Month >= response?.Month
        ? Outcome.Pass()
        : Outcome.Fail();
}
else if (source?.Year < response?.Year)
{
    return Outcome.Fail();
}
else if (source?.Year > response?.Year)
{
    return Outcome.Pass();
}
else
{
    return Outcome.Fail();
}

Answer 1

您只需将source与response月初进行比较。

var responseMonthBegin = response.HasValue 
    ? new DateTime(response.Value.Year, response.Value.Month, 1)
    : (DateTime?)null;

return source >= responseMonthBegin ? Outcome.Pass() : OutCome.Fail();

Answer 2

只需将每一个截断到本月初：

private static DateTime TruncateToMonth(DateTime date) =>
    new DateTime(date.Year, date.Month, 1, 0, 0, 0);

然后你可以使用：

// TODO: Null handling
return TruncateToMonth(source) >= TruncateToMonth(response)
    ? Outcome.Pass()
    : Outcome.Fail();

注意，TODO在那里-我根本没有尝试处理空值。我永远不会清楚地记住这些规则来预测当前的代码会做什么。如果我不集中精力就找不出答案，那么其他试图阅读代码的人也有可能会遇到同样的情况:)所以我会明确说明你希望如何处理空值。

return source == null || response == null ? Outcome.Fail()
    : TruncateToMonth(source.Value) >= TruncateToMonth(response.Value) ? Outcome.Pass()
    : Outcome.Fail();

正如注释中所指出的，您不需要严格地截断源值--如果需要的话，我只是觉得读得更清楚。