我们在编写代码的过程中,有时会因为复杂的业务,导致我们编写的代码圈复杂度过大,if...then...else
过多嵌套,写完的代码可能是这样...
这种代码给人的第一感觉就是很美腻(对称美),除此之外,我想不到了...... 但它的坏处就很明显了。在写这个代码的时候也许只有两个人可以懂(上帝和你自己),写完之后我想可能就只有一个人懂了(不是你自己。而是上帝)。
当然了这样的代码如果没有 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 就结束,不会再向下执行)。
网友评论