今天在重构代码时对下面的一堆if语句实在看着不顺眼。
if(activation == null)
{
_logger.LogError("x1");
return BooleanResult.Fail();
}
if(activation.DateAdded < DateTime.Now.AddHours(-1))
{
return BooleanResult.Fail("x2");
}
if(activation.Token != token)
{
_logger.LogError("x3");
return BooleanResult.Fail();
}
if(activation.Activated)
{
return BooleanResult.Fail("x4");
}
突然想到 C# 7.0 的新特性 —— Pattern Matching in C# 7.0 Case Blocks,眼前一亮。于是试试看能否消灭这一堆if,结果如愿以偿,又一次实际体会到 C# 7.0 的魅力。
switch(activation)
{
case null:
_logger.LogError("x1");
return BooleanResult.Fail();
case var a when a.DateAdded < DateTime.Now.AddHours(-1):
return BooleanResult.Fail("x2");
case var a when a.MailToken != token:
_logger.LogError("x3");
return BooleanResult.Fail();
case var a when a.Activated:
return BooleanResult.Fail("x4");
}