《重构》笔记 二 重新组织你的函数 Inline Methord Inline Temp 。。

二 Inline Method

    一个函数的本体应该于其名称同样清晰易懂。

    动机：如果你遇到的函数，其内部代码和其名字同样清晰易懂，你因该去掉该函数。

    做法：1.检查函数，确定它不具多态

            2.找出这个函数的所有被调用点

            3.将调用点替换为函数本体

            4.编译测试

 

   eg.

private function getRateing():Number {
    return (moreThanFiveLateDeliveries()) ? 2 : 1;
}

private function moreThanFiveLateDeliveries():Boolean {
   return _numerOfLateDeliveries > 5;
}

   ==>

 

private function getRateing():Number {
    return (_numerOfLateDeliveries > 5;) ? 2 : 1;
}

 

 

三 Inline Temp

你有一个临时变量只被简单的赋值一次，而她妨碍了其他重构手法，将所有对改变量的引用替换为表达式本身

 

动机：Inline Temp多半是为了 Replace Temp With Query的一部分来使用，所一真正的动机在后者。唯一单独使用Inline Temp 情况是，某个临时变量被赋予了某个函数的返回值，一般来讲不会有任何危害，你可以放心的留在那，但是如果它妨碍了其他的重构手法，如 Extract Methord ，那你就应该将它 Inline化。

 

做法：

    1.检查是否真的被赋值一次

     2.找到引用点，并替换为表达式

     3.每次修改后测试

 

eg.

 

var basePrice = anOder.basePrice();
return (basePrice > 1000);

   ==>

 

return (anOther.basePrice() > 1000)

 

 

 

 

   四 Replace Temp with Query

 你的程序用一个临时变量来，保存某一表达式的运算结果。

 

动机：临时变量的问题在于他们是暂时的，而且只能在所属函数中用。由于临时变量只能在所属函数中可见，所以他们可能会驱使你写出更长的函数，因为这样你才能访问到想要访问的临时变量。如果把临时变量替换为一个查询式，那么同一个Class中的所有函数都可以共享这份信息，Replace Temp with Query 往往是Extract Methord 之前必不可少的步骤。局部变量的存在会是代码很难被提炼，所一你因该尽量把他们改为查询式。

 

做法：

   1.找出只被赋值一次的临时变量，若果某个临时变量赋值超过一次，用SplitTempoary Variale分割

   2.将改临时变量的赋值表达式，提炼到单独的函数中去

   3.编译测试

   4.在该变量身上实施 Inline Temp

 

  eg.

 

private function getPrice():Number {
    var  basePrice:int = _quantity * _itemPrice;


    var discountFactor;
    if(basePrice > 1000) discountFactor = 0.95;
    else  discountFactor = 0.98;
    return basePrice * discountFactor;
}

 

   ==>

 

private function getPrice():Number {
    var  basePrice:int = basePrice

();
    var discountFactor;
    if(basePrice > 1000) discountFactor = 0.95;
    else  discountFactor = 0.98;
    return basePrice * discountFactor;
}

privaet function basePrice

():int {
    return _quantity * _itemPrice;
}

 

   ==>Inline Temp

 

 

private function getPrice():Number {
    var discountFactor;
    if(basePrice()

> 1000) discountFactor = 0.95;
    else  discountFactor = 0.98;
    return basePrice()



 * discountFactor;
}

privaet function basePrice():int {
    return _quantity * _itemPrice;
}

 

   ==>同样

 

 

 

private function getPrice():Number {
    var discountFactor = discountFactor
();

    return basePrice() * discountFactor ;
}

privaet function basePrice():int {
    return _quantity * _itemPrice;
}

private function discountFactor
():Number {
   if(basePrice() > 1000) return 0.95;
   else return 0.98;
}

 

  ==》Inline temp

 

private function getPrice():Number {
    return basePrice() * discountFactor();
}

privaet function basePrice():int {
    return _quantity * _itemPrice;
}

private function discountFactor():Number {
   if(basePrice() > 1000) return 0.95;
   else return 0.98;
}