代码沉重不堪重负,难以修改的一大问题就在于,条件逻辑过于复杂,我们需要判断很多逻辑,同时在不同的地方又有相似之处,在没有优化之前,可能通过简单的拷贝,来迅速完成功能,然而,同时带来了风险,拷贝来的代码,很可能带有不必要的条件逻辑,因此,增加了重构的工作量;
1.分解条件表达式
比较沉重的业务功能,可能拥有较为复杂的条件逻辑,在大型功能中,冗长的函数让人头大,且毫无修改心思,谁知道会不会触发罗七八糟的逻辑,而且,为了一个小修改,就要通读冗杂的逻辑,而你其实,只需要其中一个小改动
这是提取函数的一个具体分之,将一段的冗长的代码分解为多个小函数,配合语义化函数,这样,修改的时候,你只需要去查看那几个逻辑即可
2.合并条件表达式
这是和1对立的方法,但是,不可否认,这是一个好的方法,有些人在条件逻辑中,条件不同,但输出完全一致,而事实上,我们可以把他们合并起来,让代码更优雅
示例:
//源逻辑
let test = {
"name": "lock",
age: null
}
if (test.name) {
if (test.age) {
console.log("test", test);
return true;
}
}
return false;
//优化后,合并条件表达式
let test = {
"name": "lock",
age: null
}
if (test.name && test.age) {
console.log("test", test);
return true;
}
return false;
3.以卫语句取代嵌套条件表达式
这个方法的精髓就在于,在if/else语句中,给某一分之以特别重视,即,当出现此条件是,立刻返回/退出当前函数
4.以多态取代条件表达式
在复杂逻辑处理时,我们经常会在一个条件内,处理很长的函数,同时,此方法,拥有多个不同的条件,且处理方案均不相同,此时,多态便尤为重要!
这是一个简单的例子
//源逻辑
function caseInfo(name){
let discountCaseId;
if(name === "one"){
discountCaseId = 1;
}else if(name === "two") {
discountCaseId = 7;
}
return discountCaseId;
}
//优化后代码逻辑
function caseMap(name){
const caseMap = {one: 1, two: 7};
const discountCaseId = caseMap[name];
return discountCaseId;
}
当然,用于处理条件逻辑是这样的
//源逻辑
async revenueDaily(data) {
let {store_id, tenancy_id, report_date, upload_type, list} = data;
let token = await Order.getToken();
if(upload_type === "2"){
//...
return NCC.post(ROUTE.SALE_ORDER, saleData, token);
}else if(upload_type === "3"){
//...
return NCC.post(ROUTE.RECEIVABLE, receivableData, token);
}else if(upload_type === "4"){
//...
return Order.receivableForAllStores(data, token);
}
}
//优化后的逻辑
async revenueDaily(data) {
let {store_id, tenancy_id, report_date, upload_type, list} = data;
let token = await Order.getToken();
return uploadFunction[upload_type](token, {report_date, store_id, tenancy_id, list});
}
let uploadFunction = {
"2": async (token, data) => {
//...
return NCC.post(ROUTE.SALE_ORDER, saleData, token);
},
"3": async (token, data) => {
//...
return NCC.post(ROUTE.RECEIVABLE, receivableData, token);
},
"4": async (token, data) => {
//...
return Order.receivableForAllStores(data, token);
}
};
5.引入特值
这种方法适用于,代码中某个特殊的值或情况经常出现,切处理逻辑相同,即可采用引入特值方法,本质上,他和萃取函数相同,不过可以理解为对极限情况或固定情况的处理;
常见的特值处理有,判断字符是否为不存在、判断邮箱格式是否正确
6. 断言
断言,每个程序员多多少少都用到了一些,相信大部分人将其用在了入参判断,处理异常数据抛出,
在《重构2》中,作者提出了自己的理念,作者不希望断言是一个发现程序中bug的操作,而是来检测“必须为真”的条件。在程序行为中,有没有断言都应该是一样的,如果断言失败了,说明程序处理问题,比如,你的正值数字多加了一个负号;
特别铭记
T-1 单元测试
单元测试,永远是代码重构的好帮手,你不可能用人脑记得所有的极限情况,相信我!
T-2 杜绝拷贝,从你我做起
拷贝代码,是程序员的常态,但我希望,你可以完全理解代码的每一个调用,首先,你会明白你找到的代码是什么功能,其次如果时间来得及,尽可能自己敲进去,而不是简单的拷贝,在你敲进去的时候,你会理解其思路,甚至优化它!