重构手法之重新组织函数

重构手法之重新组织函数

在重构的手法中,很大的一部分是对函数进行整理,使函数能够恰当地包装代码(让代码自己说话而不是写更多的注释)。重新组织函数的驱动力,往往都是由于函数过长。因为函数过长就以为着包含了更多属性和逻辑,这样复杂的逻辑和诸多属性(如函数内部的局部变量或者静态变量等)会让代码变得难以维护,需要对其进行重新组织。

提炼函数

在冗长的函数中提炼出精小的函数,让每个短小函数负责的功能简洁,并且让函数名解释函数的用途。这是因为:
+ 如果每个函数的粒度都很小,那么函数被复用的机会就越大,组合就越发灵活
+ 函数粒度越小,覆写的时候也会更加容易
总而言之:易于重用,易于维护

以查询取代临时变量

提炼函数中临时变量是一个让人头疼的问题,我们需要消除某些临时变量。对于那些用于保存求值结果的临时变量,我们可以利用一个方法来取代之。下面是一个例子:

double rate = _whatUsed / _total;
if (rate > 0)
    return true;
else
    return false;

利用一个方法来消除double rate可以写成:

/*
    double rate(){
        return  _whatUsed / _total;
    }
*/
if (this->rate())
    return true;
else
    return flase;

当我们利用一个查询来替换临时变量,那么在这个类中个的所有方法都可以获取该信息,而不是仅仅只有临时变量原本所在的函数才能获取该信息。这样在我们提取一个新的方法时即可不通过参数传递的方式就可以直接获取该信息。这是一个准则:如果有某种信息,需要在类中不止一个方法中使用,那么该信息应当属于类而不是属于某个函数,在后文中以函数对象替换函数的手法也是基于此原则的。

引入解释性变量

其实写代码的时候写注释只是一种补充手段,我们为了增加代码的可读性应该让代码自己说话而不是尽量多地去注释,过多的注释会在代码中带来背景干扰。引入解释性变量就是基于让代码自己说话的想法引入的。下面是一个例子:

if (plamtform.toUppercase().indexof("MAC") >  0 &&
                browser.toUppercase().indexof("IE") > 0){
      //...do something
}

引入解释变量之后如下:

auto isMAC = plamtform.toUppercase().indexof("MAC") > -1;
auto isIE = browser.toUppercase().indexof("IE") > -1;

if (isMAC && isIE){
    //do something....
}

代码可读性大大提高。

分解临时变量

在代码中对一个临时变量进行多次赋值使用,则意味着一个临时变量拥有多重语意,这会在后期维护中带来疑惑,需要分解该临时变量。下面是一个例子:

auto temp =  (_hight + _width) * 2;
std::cout << temp << std::endl;
temp = _hight * _width;
std::cout << temp << std::endl;

分解变量之后:

auto perimeter =  (_hight + _width) * 2;
std::cout << perimeter << std::endl;
auto area = _hight * _width;
std::cout << area << std::endl;

以函数对象替换函数

在进行函数提炼的时候,如果被提炼的函数有许多临时变量,并且提炼出来的子函数会用到这些临时变量。这样可以将这些临时变量作为参数传递到子函数当中,但是如果用到的临时变量很多,那么函数的参数列表就会变得很长。于是我们可以使用函数对象来替换原来的函数。这样,原函数中的临时变量现在就可以变成类的一个属性,可以被所有的方法共享,于是提炼出来的子函数,就可以不依赖临时变量,提取子函数就会变得十分方便,因为不用考虑临时变量的问题。

猜你喜欢

转载自blog.csdn.net/u013298300/article/details/52128401