用C# 7.0的switch...case模式匹配取代一堆if语句
今天在重构代码时对下面的一堆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"); }