标签：function for 程序 可操作性 erro 需要 htm func 比较
之前有整理过一些关于代码重构的心得的文字：http://www.cnblogs.com/x3d/p/5797606.html ， 其中主要想表达的意思是要做好一件事，需要这样一而再、再而三的过程才行。
重构有整体和局部两种情况，前次是讲整体，这次主要讲局部。
局部的重构，主要针对小范围的代码，可能是一个类、一个方法/函数，甚至只是函数中的一段代码。局部的重构，往往具备更高的可操作性，大家在编程实践过程中，也更有可能会意识到这点。
这里先摘取一段业务代码：
if (code >= 16 && code <= 30) {
termCode = '30d'
}
........
if (code >= 331 && code <= 365) {
termCode = '365d'
}大致可以看出来这是关于根据时间范围构造最终条件的判断逻辑，分了12个区间。
这样的代码，在一个经过比较专业训练的程序员看来，肯定是会觉得不行的，语句明显不够精炼，而且逻辑上有漏洞。
首先他确实完成了这个逻辑，是能够交差了的。但过不了谁那一关？
我们知道一个目标，不要重复你自己，代码应该越写越少。越写越少，有两种解读：一是经过不断的代码库积累，可重用代码越来越多，需要从头写的代码是越来越少的；二是实现同一个功能所需的代码应该越来越简练、行数越来越少。
先按第二种解读，这段代码本身看看有没有可以缩短的可能，答案是肯定有！
将逻辑提炼为数据结构，减少字面量，就接近真相了，比如像下面这样改写（代码仅供演示，未执行测试）：
const dayRanges = [15, 30, 60, 90, 120, 150, 180, 210, 240, 270, 300, 330, 365];
if (code < dayRanges[0] || code > dayRanges[dayRanges.length - 1]) {
throw new Error('请选择正确的时间范围' );
}
for (let i = 0; i < dayRanges.length; i++) {
if ((code >= (dayRanges[i] + 1)) && (code <= dayRanges[i + 1])) {
termCode = dayRanges[i + 1] + 'd';
break;
}
}再按第一种解读，就是要注重代码的积累和重用，对这段代码进行封装，让它尽量可以被重复使用，这样总体代码量肯定是变少的，而且可以预期的维护工作量变小。
exports.termCode = function(dayRanges, code) {
if (code < dayRanges[0] || code > dayRanges[dayRanges.length - 1]) {
throw new Error('请选择正确的时间范围' );
}
for (let i = 0; i < dayRanges.length; i++) {
if ((code >= (dayRanges[i] + 1)) && (code <= dayRanges[i + 1])) {
termCode = dayRanges[i + 1] + 'd';
break;
}
}
};
module.exports = exports;const lib = require('./lib');
const dayRanges = [15, 30, 60, 90, 120, 150, 180, 210, 240, 270, 300, 330, 365];
const code = 28;
const termCode = lib. termCode(dayRanges, code);通过反复的主动的练习、实践，把最单调的日常开发工作变得让自己更有使命感、成就感。
标签：function for 程序 可操作性 erro 需要 htm func 比较
原文地址：https://www.cnblogs.com/x3d/p/8996548.html
