思特沃克学院小小读书会ThoughtWorks欧亚创新工作室

简化条件表达式

2017-08-06  本文已影响51人  司鑫

我们在编写代码的过程中,有时会因为复杂的业务,导致我们编写的代码圈复杂度过大,if...then...else 过多嵌套,写完的代码可能是这样...

from internet

这种代码给人的第一感觉就是很美腻(对称美),除此之外,我想不到了...... 但它的坏处就很明显了。在写这个代码的时候也许只有两个人可以懂(上帝和你自己),写完之后我想可能就只有一个人懂了(不是你自己。而是上帝)。

当然了这样的代码如果没有 bug 还好,万一遇到了 bug 而原作者又不在,那么这段神奇的代码将可能会被另一个(yu)神(mei)奇的作者换成另一段神奇的代码,并以此 loop 下去......

So...... 如果这是自己写的代码,那么趁着还没人发现,赶紧重构下吧...

1 Decompose Conditional(分解条件表达式)

怎么说

将 `if...then...else` 中的代码块抽离出来

为什么

提高代码可读性

怎么做

先来看段代码
eg:假如一家商场对商品进行打折,价格在 100 - 200 之间的打八折,其它的按原价出售((hen)不(bie)合(jiao)适的栗子)。

function calculatePrice(price) {
  let result;

  if (price >= 100 && price < 200) {
    result = price * 0.8;
  } else {
    result = price;
  }

  return result;
}

代码很简单,好像没什么好重构的,但是代码想要表达的意图和业务的联系并不是很大,也就是说我们把前面对业务的介绍去掉,那么代码就不具备任何可读性了(代码不能很好的表达自己的用途),现在我们对代码进行重构。

function calculatePrice(price) {
  let result;

  if (isDiscount(price)) {
    result = discount(price);
  } else {
    result = price;
  }

  return result;
}

function isDiscount(price) {
  return price >= 100 && price < 200;
}

function discount(price) {
  return price * 0.5;
}

现在我们再来读这段代码,会发现可读性提高了,我们在读到第一个if时,我们不需要再去思考里这个判断语句是什么意思,我们可以直接根据方法名 isDiscount() 就可以知道,这是在判断是否需要打折,需要的话就打折,不需要的话原价返回,这样我们就可以把注意力放在代码的整体逻辑上,而不需要在把过多的注意力放在细节方面。

2 Consolidate Conditional Expression(合并条件表达式)

怎么说

把结果相同的条件表达式合并成一个独立的函数。

为什么

提高代码可读性

怎么做

再来个蹩脚的栗子
eg:某商场再次打折商品,价格大于200的商品打五折,前提是要买够五件该商品。

function calculatePrice(price, num) {
  if (price < 200) return price * num;
  if (num < 5) return price * num;
  return price * num * 0.5;
}

先蹩说话,先进行第一步重构


function calculatePrice(price, num) {
  if (price > 200 || num < 5) return price * num;
  return price * num * 0.5;
}

第二步重构请参考 Decompose Conditional

3 Consolidate Duplicate Conditional Fragments (合并重复的条件片段)

怎么说

将条件表达式内的重复语句块搬移到条件表达式外

为什么

减少重复代码

怎么做
比较简单,就直接上代码

function calculatePrice(price) {
  let result;

  if (price <  200) {
    result = discount(price);
    result -= 10; //重复代码
  } else {
    result = price;
    result -= 10;//重复代码
  }

  return result;
}

重构下


function calculatePrice(price) {
  let result;

  if (price < 200) {
    result = discount(price);
  } else {
    result = price;
  }
  result -= 10;
  
  return result;
}
4 Replace Nested Conditional with Guard Clauses(以卫语句替代嵌套条件表达式)

怎么说

使用卫语句表示所有的特殊情况(一旦找到结果就可以立即返回)

卫语句: 如果某个条件比较特殊,就应该单独检查该条件,并当该条件为真时立即返回。

为什么

提高代码可读性

怎么做
摘抄一个《重构》中的栗子:
一个薪册系统,以特殊来处理死亡、驻外、退休员工的薪资。

function getPayAmount() {
  let result;

  if (isDead) {
    result = deadAmount();
  }else{
    if(isSeparated) {
      result = separatedAmount();
    }else {
      if(isRetired) {
        result = retiredAmount()
      }else {
        result = normalAmount();
      }
    }
  }

  return result;
}

额额额,我觉得很少人会写出这样的代码吧,很罗嗦,来看看重构后的代码

function getPayAmount() {
  if (isDead)    return deadAmount();
  if (isSeparated)return separatedAmount();
  if (isRetired)return retiredAmount();
  
  return normalAmount();
}

虽然我们写代码不会像第一种那样繁琐,但使用卫语句我们可以让代码的可读性大大提高,并提高代码的效率(遇到 return 就结束,不会再向下执行)。

上一篇下一篇

猜你喜欢

热点阅读