简化条件表达式

我们在编写代码的过程中,有时会因为复杂的业务,导致我们编写的代码圈复杂度过大,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 就结束,不会再向下执行)。

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 203,456评论 5 477
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 85,370评论 2 381
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 150,337评论 0 337
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,583评论 1 273
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,596评论 5 365
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,572评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 37,936评论 3 395
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,595评论 0 258
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 40,850评论 1 297
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,601评论 2 321
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,685评论 1 329
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,371评论 4 318
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 38,951评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,934评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,167评论 1 259
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 43,636评论 2 349
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,411评论 2 342

推荐阅读更多精彩内容

  • 1 Decompose Conditional(分解条件表达式) 从复杂表达式if-then-else三个段落中分...
    hklbird阅读 1,427评论 0 0
  • 简化条件表达式 9.1 Decompose Conditional (分解条件表达式) 你有一个复杂的条件表达式语...
    rxdxxxx阅读 479评论 0 0
  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 171,394评论 25 707
  • 今天早上,简单的breakfast。昨晚做的全麦面包,想给孩子做个漂亮点的造型,貌似模具不是很给力。后来又...
    菩提释阅读 274评论 0 3
  • 国庆八天,几乎都宅在婆婆家,这次回去,婆婆的态度似乎又好些,估摸着她也觉得我们也就呆几天,所以放宽了标准,也...
    爱自满溢阅读 216评论 0 0