动机

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

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

做法

范例

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

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)

function notSummer () {
  return date.before(SUMMER_START) || date.after(SUMMER_END)
}

function winterCharge (quantity) {
  return quantity * _summerRate
}

function summerCharge () {
  return quantity * _winterRate + __winterServiceCharge
}

  通过这段代码可以看出整个重构带来的清晰性。像这样的情况下,许多程序员都不会去提炼分支条件。因为这些分支条件往往非常短,看上去似乎没有提炼的必要。但是,在代码意图和代码自身之间往往存在不小差距。哪怕在上面这样一个小小的例子中,notSummer(date)这个语句也能够比原本的代码更好地表达自己的用途。对于原来的代码,必须看着它,想一想才能知道其作用。———notSummer(date)看上去就像一段注释那样清楚明白。

声明

以上摘抄自《重构改善既有代码的设计》一书,代码部分由原来的java改写成javascript。目的仅为加深印象和理解,如有侵权请联系。