简化条件表达式

作者: 司鑫 | 来源:发表于2017-08-06 10:37 被阅读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 就结束,不会再向下执行)。

    相关文章

      网友评论

        本文标题:简化条件表达式

        本文链接:https://www.haomeiwen.com/subject/sxrglxtx.html