Decompose Conditional (分解条件表达式)

Summary: java

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

Motivation:  函数

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

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

Mechanics: it

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

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

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

         范例

假设要计算购买牟阳商品的总价(总价=数量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)这个语句也可以比本来的代码更好地表达本身的用途。对于原来的代码,咱们必须看着它,想想,才能说出其做用。固然,在这个简单的例子中,这并不困难。不过即便如此,提炼出来的额函数可读性也更高一些--它看上去就像一段注释那样清楚而明白。

相关文章
相关标签/搜索