代码重构

这是一个持续跟新的文章。

这篇文章会记录下我对代码重构的思考和读书的总结。

1.提炼函数

• 在JavaScript开发中，我们大部分时间都在与函数打交道，所以我们希望这些函数有良好的命名，函数体内包含的逻辑清晰明了。
• 如果一个函数过长，不得不加上若干注释才能让这个函数显得易读一些，那这些函数就很有必要进行重构。
• 如果在函数中有一段代码可以被独立出来，那我们最好把这些代码放进另外一个独立的函数中，好处有如下几点。
  
  • 避免出现超大函数。
  • 独立出来的函数有助于代码复用。
  • 独立出来的函数更容易被覆写。
  • 独立出来的函数如果有一个良好的命名，它本身就起到了注释的作用。

        var getUserInfo = function () {            ajax('http:// xxx.com/userInfo', function (data) {                console.log('userId:' + data.userId);                console.log('userName:' + data.userName);                console.log('nickName:' + data.nickName);            });        };        改成：        var getUserInfo = function () {            ajax('http:// xxx.com/userInfo', function (data) {                printDetails(data);            });        };        var printDetails = function (data) {            console.log('userId:' + data.userId);                console.log('userName:' + data.userName);                console.log('nickName:' + data.nickName);        }

2.合并重复的条件片段

        var paging = function (currPage) {            if(currPage <= 0){                currPage = 0;                jump(currPage); //跳转            }else if(currPage >= totalPage){                currPage = totalPage;                jump(currPage); //跳转            }else{                jump(currPage); //跳转            }        };

    可以看到，负责跳转的代码jump(currPage)在每个条件分支内都出现了，所以完全可以把这句代码独立出来。

        var paging = function (currPage) {            if(currPage <= 0){                currPage = 0;            }else if(currPage >= totalPage){                currPage = totalPage;            }            jump(currPage); //跳转        };

3.把条件分支语句提炼成函数

        var getPrice = function (price) {            var date = new Date();            if(date.getMonth() >= 6 && date.getMonth() <= 9){ //夏天                return price * 0.8;            }            return price;        }        上面的代码表达的意图和代码自身还存在一些距离，阅读代码的人必须要多花一些精力才能明白它传达的意图。        其实可以把这句代码提炼成一个单独的函数，技能更准确地表达代码的意思，函数名本身又能起到注释的作用。        代码如下        var isSummer = function () {            var date = new Date();            return date.getMonth() >= 6 && date.getMonth() <= 9;        }        var getPrice = function (price) {            if(isSummer()){ //夏天                return price * 0.8;            }            return price;        };

4.提前让函数退出代替嵌套条件分支

许多程序员都有这样一个观念：“每个函数只能有一个入口和一个出口。”现代编程语言都会限制函数只有一个入口。但关于“函数只有一个出口”，往往会有一些不同的看法。下面这段伪代码是遵守“函数只有一个出口”的典型代码：

    var del = function (obj) {            var ret;            if(!obg.isReadOnly) { //不为只读才能删除                if(obj.isFolder) { //如果是文件夹                    ret = deleteFolder(obj);                }else if (obj.isFile){                    ret = deleteFile(obj);                }            }            return ret;        }        用《重构》中的话说，嵌套的条件分支往往是由一些深信“每个函数只能有一个出口的”程序员写出的。        但实际上，如果对函数的剩余部分不感兴趣，那就应该立即退出。        引导阅读者去看一些没有用的else片段，只会妨碍他们对程序的理解。        var del = function (obj) {            if(obj.isReadOnly){ //反转if表达式                return;            }            if(obj.isFolder){                return deleteFolder(obj);            }            if(obj.isFile){                return deleteFile(obj);            }        }