小酌重构系列[23]——封装条件

 

当条件判断语句较为复杂时（有多个不同的检查项），就像下面这幅图所表示的，会使得代码的可读性会大打折扣，也难以清晰地传达判断意图。

再者，当判断逻辑变更时，我们不得不去修改if语句里面的判断代码。
如果判断写得有问题，则会影响方法的正确性，也会给该方法的单元测试带来一些障碍。

我们可以根据检查项是否需要参数来封装条件，如果检查项不需要参数，则可以将其提取为属性；如果需要参数，则将其提取为方法。
本文要讲的重构策略“封装条件”是基于“提取方法”这个重构策略的。

示例

重构前

这个示例中，PerformCoolFunction()方法的if条件检查项有三个。

public class RemoteControl
{
    private string[] Functions { get; set; } private string Name { get; set; } private int CreatedYear { get; set; } public string PerformCoolFunction(string buttonPressed) { // Determine if we are controlling some extra function // that requires special conditions if (Functions.Length > 1 && Name == "RCA" && CreatedYear > DateTime.Now.Year - 2) return "doSomething"; } } 

我们可以不必去理解这个复杂判断的含义，从代码层面看，这个判断有三个问题：

• 可读性较差，同时也意味着较差的维护性
• 增加了编写PerformCoolFunction方法单元测试的复杂度，我们不得不为if判断单独追加一个测试case
• if判断所用的参数和PerformCoolFunction方法无关

重构后

这三个检查项构成的条件表达式可读较差，所以将这个条件表达式提取为一个方法。这样在做条件判断时就变成了if (HasExtraFunctions)，代码的可读性由此增强。另外，我们通过提取出来的条件判断方法名称，也可以准确地洞悉判断的意图。

public class RemoteControl
{
    private string[] Functions { get; set; } private string Name { get; set; } private int CreatedYear { get; set; } private bool HasExtraFunctions { get { return Functions.Length > 1 && Name == "RCA" && CreatedYear > DateTime.Now.Year - 2; } } public string PerformCoolFunction(string buttonPressed) { // Determine if we are controlling some extra function // that requires special conditions if (HasExtraFunctions) return "doSomething"; } } 

用bool变量代替判断条件

这个示例的判断条件只有3个，而且每个条件的判断语句比较短。
如果遇到一些判断语句较长，且条件个数很多的判断，我们该怎么办？
我的建议是用bool变量代替判断条件。

偷个懒，我们仍然用上面的例子来示范这种方式：

public class RemoteControl
{
    private string[] Functions { get; set; } private string Name { get; set; } private int CreatedYear { get; set; } private bool HasExtraFunctions { get { // 前缀c表示condition bool cFuncLen = (Functions.Length > 1); bool cName = (Name == "RCA"); bool cCreatedYear = (CreatedYear > DateTime.Now.Year - 2); return cFuncLen && cName && cCreatedYear; } } public string PerformCoolFunction(string buttonPressed) { // Determine if we are controlling some extra function // that requires special conditions if (HasExtraFunctions) return "doSomething"; } }

转载于:https://www.cnblogs.com/lv448395223/p/5618157.html