用C# 7.0的switch...case模式匹配取代一堆if語句 - dudu－IT工程師數位筆記本

文章出處

今天在重構代碼時對下面的一堆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");
}