今天在重构代码时对下面的一堆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");
}