Question

if ((DateTime.Now.DayOfWeek != DayOfWeek.Friday && DateTime.Now.DayOfWeek != DayOfWeek.Saturday) &&
((DateTime.Now.Hour >= 10 && DateTime.Now.Hour < 13) || (DateTime.Now.Hour >= 20 && DateTime.Now.Hour < 23)))

我必须缩短这个条件，有什么建议吗？

Answer 1

好吧，你可以建立一个扩展方法：

public static bool BoundsCheck(this DateTime d, int min, int max, int min2, int max2)
{
    return (d.DayOfWeek != DayOfWeek.Friday &&
        d.DayOfWeek != DayOfWeek.Saturday &&
        d.Hour >= min &&
        d.Hour < max) ||
        (d.Hour >= min2 && d.Hour < max2);
}

然后像这样调用它：

if (DateTime.Now.BoundsCheck(10, 13, 20, 23))...

Answer 2

这个更短吗？也许，但在我看来更重要的是它更具可读性和可维护性：

var now = DateTime.Now;
var notAllowedDays = new[] { DayOfWeek.Friday, DayOfWeek.Saturday };
var allowedHours = Enumerable.Range(10, 3).Concat(Enumerable.Range(20, 3));
if(!notAllowedDays.Contains(now.DayOfWeek) && allowedHours.Contains(now.Hour))
{

}

Answer 3

您可以更改使用小时数

(DateTime.Now.Hour % 12) +1 >= 10 && (DateTime.Now.Hour % 12) +1 < 13

甚至可能没有第二次检查。

我认为除了寻找其他方法（如其他答案）之外，你所能提供的远不止这些改善

<强>更新我测试了以上和它的错误，但这更加虐待狂并且有效

var check = (DateTime.Now.Hours - 10 % 12) % 10;
 var checkV = (DateTime.Now.Hours >= 10 && check < 3);

测试代码

for (int i = 0; i < 24; i++)
{
    var check = (i - 10 % 12) % 10;
    bool checkV = (i >= 10 && check < 3);
    Console.WriteLine(i.ToString() + ": " + checkV.ToString());
 }
 Console.ReadKey();

更新2 完整的缩短代码

if(   (int)DateTime.Now.DayOfWeek < 5 && 
           DateTime.Now.Hours >= 10 && 
         ((DateTime.Now.Hours - 10 % 12) % 10) < 3)

Answer 4

if (!this.ItsPartyDay() && (this.ItsLunchTime() || this.ItsDinnerTime()))
{
    ...
}

private bool ItsPartyDay()
{
    return (Int32)DateTime.Now.DayOfWeek >= 5;
}

private bool ItsLunchTime()
{
    return (DateTime.Now.Hour >= 10 && DateTime.Now.Hour < 13);
}

private bool ItsDinnerTime()
{
    return (DateTime.Now.Hour >= 20 && DateTime.Now.Hour < 23);
}

Answer 5

我认为没有任何合理的解决方案，但这里有一对想到的。使用DateTime和DayOfWeek的别名。另一个选择是在条件之前将所有这些值分配给变量。

所以你可以这样做;

 string fri = DayOfWeek.Friday;
 string sat = DayOfWeek.Saturday;

然后在条件中使用那些。或;

 using dt = DateTime;

然后你可以做dt.Now.DayOfWeek

我个人不建议做其中任何一件事。你实际上并没有缩短条件，你只是在重构。如果你在一个课程中有很多这些，那么可能值得权衡，否则它可能不会。

编辑：Michael Perrenoud的扩展方法建议是一个合理的解决方案，实际上效果很好。

缩短DateTime条件检查

5 个答案: