代码重构(二)

1.传递对象参数代替过长的参数列表

有时候一个函数有可能接收多个参数,而参数的数量越多,函数就越难理解和使用。使用该函数的人首先得搞明白全部参数的含义,在使用的时候,还要小心翼翼,以免少传了某个参数或者把两个参数搞反了位置。如果我们想在第3个参数和第4个参数之中增加一个新的参数,就会涉及许多代码的修改,代码如下:

        var setUserInfo = function( id, name, address, sex, mobile, qq ){
            console.log( 'id= ' + id );
            console.log( 'name= ' +name );
            console.log( 'address= ' + address );
            console.log( 'sex= ' + sex );
            console.log( 'mobile= ' + mobile );
            console.log( 'qq= ' + qq );
        };

        setUserInfo( 1314, 'sven', 'shenzhen', 'male', '137********', 377876679 );

这时我们可以把参数都放入一个对象内,然后把该对象传入setUserInfo函数,setUserInfo函数需要的数据可以自行从该对象里获取。现在不用再关心参数的数量和顺序,只要保证参数对应的key值不变就可以了:

        var setUserInfo = function( obj ){
            console.log( 'id= ' + obj.id );
            console.log( 'name= ' + obj.name );
            console.log( 'address= ' + obj.address );
            console.log( 'sex= ' + obj.sex );
            console.log( 'mobile= ' + obj.mobile );
            console.log( 'qq= ' + obj.qq );
        };

        setUserInfo({
            id: 1314,
            name: 'sven',
            address: 'shenzhen',
            sex: 'male',
            mobile: '137********',
            qq: 377876679
        });

2.尽量减少参数数量

如果调用一个函数时需要传入多个参数,那这个函数是让人望而生畏的,我们必须搞清楚这些参数代表的含义,必须小心翼翼地把它们按照顺序传入该函数。而如果一个函数不需要传入任何参数就可以使用,这种函数是深受人们喜爱的。在实际开发中,向函数传递参数不可避免,但我们应该尽量减少函数接收的参数数量。下面举个非常简单的示例。有一个画图函数draw,它现在只能绘制正方形,接收了3个参数,分别是图形的width、heigth以及square

        var draw = function( width, height, square ){};

但实际上正方形的面积是可以通过width和height计算出来的,于是我们可以把参数square从draw函数中去掉:

        var draw = function( width, height ){
            var square = width * height;
        };

假设以后这个draw函数开始支持绘制圆形,我们需要把参数width和height换成半径radius,但图形的面积square始终不应该由客户传入,而是应该在draw函数内部,由传入的参数加上一定的规则计算得来。此时,我们可以使用策略模式,让draw函数成为一个支持绘制多种图形的函数

3.少用三目运算符

有一些程序员喜欢大规模地使用三目运算符,来代替传统的if、else。理由是三目运算符性能高,代码量少。不过,这两个理由其实都很难站得住脚。

即使我们假设三目运算符的效率真的比if、else高,这点差距也是完全可以忽略不计的。在实际的开发中,即使把一段代码循环一百万次,使用三目运算符和使用if、else的时间开销处在同一个级别里。

同样,相比损失的代码可读性和可维护性,三目运算符节省的代码量也可以忽略不计。让JS文件加载更快的办法有很多种,如压缩、缓存、使用CDN和分域名等。把注意力只放在使用三目运算符节省的字符数量上,无异于一个300斤重的人把超重的原因归罪于头皮屑。

如果条件分支逻辑简单且清晰,这无碍我们使用三目运算符:

        var global = typeof window ! == "undefined" ? window : this;

但如果条件分支逻辑非常复杂,如下段代码所示,那我们最好的选择还是按部就班地编写if、else。if、else语句的好处很多,一是阅读相对容易,二是修改的时候比修改三目运算符周围的代码更加方便:

        if ( ! aup || ! bup ) {
            return a === doc ? -1 :
              b === doc ? 1 :
              aup ? -1 :
              bup ? 1 :
              sortInput ?
              ( indexOf.call( sortInput, a ) - indexOf.call( sortInput, b ) ) :
              0;
        }

4.合理使用链式调用

经常使用jQuery的程序员相当习惯链式调用方法,在JavaScript中,可以很容易地实现方法的链式调用,即让方法调用结束后返回对象自身,如下代码所示:

        var User = function(){
            this.id = null;
            this.name = null;
        };

        User.prototype.setId = function( id ){
            this.id = id;
            return this;
        };

        User.prototype.setName = function( name ){
            this.name = name;
            return this;
        };

        console.log( new User().setId( 1314 ).setName( 'sven' ) );

或者:

        var User = {
            id: null,
            name: null,
            setId: function( id ){
              this.id = id;
              return this;
            },
            setName: function( name ){
              this.name = name;
              return this;
            }
        };

        console.log( User.setId( 1314 ).setName( 'sven' ) );

使用链式调用的方式并不会造成太多阅读上的困难,也确实能省下一些字符和中间变量,但节省下来的字符数量同样是微不足道的。链式调用带来的坏处就是在调试的时候非常不方便,如果我们知道一条链中有错误出现,必须得先把这条链拆开才能加上一些调试log或者增加断点,这样才能定位错误出现的地方。

如果该链条的结构相对稳定,后期不易发生修改,那么使用链式调用无可厚非。但如果该链条很容易发生变化,导致调试和维护困难,那么还是建议使用普通调用的形式:

        var user = new User();

        user.setId( 1314 );
        user.setName( 'sven' );

5.分解大型类

HTML5版“街头霸王”的第一版代码中,负责创建游戏人物的Spirit类非常庞大,不仅要负责创建人物精灵,还包括了人物的攻击、防御等动作方法,代码如下:

        var Spirit = function( name ){
            this.name = name;
        };

        Spirit.prototype.attack = function( type ){    // 攻击
            if ( type === 'waveBoxing' ){
              console.log( this.name + ':使用波动拳’ );
            }else if( type === 'whirlKick' ){
                console.log( this.name + ':使用旋风腿’ );
            }
        };

        var spirit = new Spirit( 'RYU' );

        spirit.attack( 'waveBoxing' );      // 输出:RYU:使用波动拳
        spirit.attack( 'whirlKick' );    // 输出:RYU:使用旋风腿

后来发现,Spirit.prototype.attack这个方法实现是太庞大了,实际上它完全有必要作为一个单独的类存在。面向对象设计鼓励将行为分布在合理数量的更小对象之中:

        var Attack = function( spirit ){
            this.spirit = spirit;
        };

        Attack.prototype.start = function( type ){
            return this.list[ type ].call( this );
        };

        Attack.prototype.list = {
            waveBoxing: function(){
              console.log( this.spirit.name + ':使用波动拳’ );
            },
            whirlKick: function(){
              console.log( this.spirit.name + ':使用旋风腿’ );
            }
        };

现在的Spirit类变得精简了很多,不再包括各种各样的攻击方法,而是把攻击动作委托给Attack类的对象来执行,这段代码也是策略模式的运用之一:

        var Spirit = function( name ){
            this.name = name;
            this.attackObj = new Attack( this );
        };

        Spirit.prototype.attack = function( type ){    // 攻击
            this.attackObj.start( type );
        };

        var spirit = new Spirit( 'RYU' );

        spirit.attack( 'waveBoxing' );    // 输出:RYU:使用波动拳
        spirit.attack( 'whirlKick' );    // 输出:RYU:使用旋风腿

6.用return退出多重循环

假设在函数体内有一个两重循环语句,我们需要在内层循环中判断,当达到某个临界条件时退出外层的循环。我们大多数时候会引入一个控制标记变量:

        var func = function(){
            var flag = false;
            for ( var i = 0; i < 10; i++ ){
              for ( var j = 0; j < 10; j++ ){
                  if ( i * j >30 ){
                      flag = true;
                      break;
                  }
              }
              if ( flag === true ){
                  break;
              }
            }
        };

第二种做法是设置循环标记:

        var func = function(){
            outerloop:
            for ( var i = 0; i < 10; i++ ){
              innerloop:
              for ( var j = 0; j < 10; j++ ){
                  if ( i * j >30 ){
                      break outerloop;
                  }
              }
            }
        };

这两种做法无疑都让人头晕目眩,更简单的做法是在需要中止循环的时候直接退出整个方法:

        var func = function(){
            for ( var i = 0; i < 10; i++ ){
              for ( var j = 0; j < 10; j++ ){
                  if ( i * j >30 ){
                      return;
                  }
              }
            }
        };

当然用return直接退出方法会带来一个问题,如果在循环之后还有一些将被执行的代码呢?如果我们提前退出了整个方法,这些代码就得不到被执行的机会:

        var func = function(){
            for ( var i = 0; i < 10; i++ ){
              for ( var j = 0; j < 10; j++ ){
                  if ( i * j >30 ){
                      return;
                  }
              }
            }
            console.log( i );    // 这句代码没有机会被执行
        };

为了解决这个问题,我们可以把循环后面的代码放到return后面,如果代码比较多,就应该把它们提炼成一个单独的函数:

        var print = function( i ){
            console.log( i );
        };

        var func = function(){
            for ( var i = 0; i < 10; i++ ){
              for ( var j = 0; j < 10; j++ ){
                  if ( i * j >30 ){
                      return print( i );
                  }
              }
            }
        };
        func();

你可能感兴趣的:(代码重构(二))