Decompose Conditional (分解条件表达式)

Summary:

从if、then、else三个段落中分别提炼出独立函数

Motivation: 

程序之中,复杂的逻辑是最常导致复杂度上升的地点之一。你必须编写代码来检查不同的条件分支、根据不同的分支做不同的事,然后,你很快就会得到一个相当长的函数。大型函数自身就会使代码的可读性下降,而条件逻辑则会使代码更难阅读。在带有复杂条件逻辑的函数中,代码(包括检查条件分支的代码和真正实现功能的代码)会告诉你发生的事,但常常让你弄不清楚为什么会发生这样的事,这就说明代码的可读性的确大大降低了。

和任何大块头代码一样,你可以将它分解为多个独立函数,根据没个小块代码的用途,为分解而得到的新函数命名,并将原函数中对应的代码改为调用新建函数,从而更清楚地表达自己的意图。对于条件逻辑,将每个分支条件分解成新行数还可以给你带来更多好处:可以突出条件逻辑,更清楚地表明每个分支的作用,并且突出每个分支的原因。

Mechanics:

1.将if段落提炼出来,构成一个独立函数。

2.将then段落和else段落都提炼出来,各自构成一个独立函数

如果发现嵌套的条件逻辑,可以先观察是否可以使用Replace Nested Conditional with Guard Clauses。如果不行,才开始分解其中的每个条件

         范例

假设要计算购买牟阳商品的总价(总价=数量x单价),而这个商品在冬季和夏季的单价是不同的

if (date.before(SUMMER_START)||date.after(SUMMER_END))
     charge = quantity * _winterRate + _winterServiceCharge;
else charge = quantity * _summerRate

我们把每个分支的判断条件都提炼到一个独立函数中,如下所示:

if(notSummber(date))
   charge = winterCharge(quantity);
else charge = summerCharge(quantity);

private boolean notSummer(Date date){
   return date.before(SUMMER_START)|| date.after(SUMMER_END);
}

private double summerCharge(int quantity){
   return quantity * _summerRate;
}

private double winterCharge(int quantity){
   return quantity * _winterRate + _winterServiceCharge;
}

通过这段代码可以看出整个重构带来的清晰性。实际工作中,我们应该逐步进行每一次提炼,并在每次提炼之后编译并测试。

像这样的情况下许多程序员都不会去提炼分支条件。因为这些分支条件往往非常短,看上去似乎并没有提炼的必要。但是,尽管这些条件往往很短,在代码意图和代码自身之间往往存在不小的差距。哪怕在上面这样一个小小例子中,notSummer(date)这个语句也能够比原本的代码更好地表达自己的用途。对于原来的代码,我们必须看着它,想一想,才能说出其作用。当然,在这个简单的例子中,这并不困难。不过即使如此,提炼出来的额函数可读性也更高一些--它看上去就像一段注释那样清楚而明白。

你可能感兴趣的:(Decompose Conditional (分解条件表达式))