有一个复杂的(if-then-else)语句。

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

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

 动机

程序之中,大型函数会使代码可读性下降,逻辑复杂的代码更加难以阅读。
应该将它分解成多个独立的函数,根据每小块的功能命名函数,将原来对应的代码换成函数调用,从而清晰的表达自己的意图。
将每一个分支条件分解形成新函数,还可以突出条件逻辑,更清楚的表明分支的作用。

作法

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

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

3. 如果发现嵌套的条件逻辑,应该先看看是否可以使用Replace Nested Conditional with Guard Clauses[以卫语句取代嵌套条件式]如果不行才开始分解其中的每个条件。

 

假设我要计算购买某样商品东西的总价(总价=数量*单价),而这个商品在夏季和冬季的价格是不同的:

if(date.before(SUMMER_START) || date.after(SUMMER_END))
    charge=quantity*_winterRate+_winterServiceCharge;
else
    charge=quantity*_summerRate;
我把每个分支的判断条件都提炼到一个独立函数中,如下所示

if (notSummer(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)这个语句也能够比原本的代码更好地表达自己的用途,对于原来的代码,我必须看着它,想一想,才能说出其作用。当然,在我们这个简单例子中,这并不困难。不过,即使如此,提炼出来的函数可读性也更高一些,它看上去就像一段注释那样清楚而明白。

posted on 2016-04-28 10:56  Sharpest  阅读(398)  评论(0编辑  收藏  举报