重构--Replace Temp with Query(以查询取代临时变量)(六)

• 重构方式
  • 你的程序以一个临时变量保存某一表达式的运算结果；将这个表达式提炼到一个独立函数中，将这个临时变量的所有引用点替换为对新函数的调用。此后，新函数就可被其他函数使用；
    • 
• 动机
  • 临时变量的问题：
    • 只能存在于所属函数；
    • 会驱使函数变长，只有这样才能访问到需要的临时变量；
  • Replace Temp with Query往往是在运用Extract Method之前必不可少的一个步骤；局部变量会使代码难以被提炼，所以应该尽可能把它们替换为查询式；
  • 情况：临时变量只被赋值一次，或者赋值给临时变量的表达式不受其他条件影响；
• 做法
  • 找出只被赋值一次的临时变量；
    • 如果某个临时变量被赋值超过一次，考虑使用"分解临时变量"将它分割成多个变量；
  • 将该临时变量声明为final；
  • 编译
    • 这可确保该临时变量的确只被赋值一次；
  • 将"对该临时变量赋值"之语句的等号右侧部分提炼到一个独立函数中；
    • 首先将函数声明为private。日后你可能会发现有更多类需要使用它，那时放松对它的保护也很容易；
    • 确保提炼出来的函数无任何副作用，也就是说该函数并不修改任何对象内容。
  • 编译，测试。
  • 对该临时变量使用"内联临时变量"；
  • 特别说明：
    • 我们常使用临时变量保存循环中的累加信息。在这种情况下，整个循环都可以被提炼为一个独立函数，这也使原本的函数可以少掉几行循环逻辑。
    • 有时，你可能会在一个循环中累加好几个值。这种情况下应该针对每个累加值重复一遍循环，这样就可以将所有临时变量都替换为查询；
• 范例
  • 示例代码：将这两个临时变量都替换掉，每次一个；

double getPrice(){

intbasePrice = _quantity * _itemPrice;

doublediscountFactor;

if(basePrice > 1000){

discountFactor = 0.95;

}else {

discountFactor = 0.98;

}

returnbasePrice * discountFactor;

}

• 保险起见，先把临时变量声明为final，检查它们是否的确只被赋值一次，如果临时变量并没有被赋值一次，就不应该进行这次重构：

double getPrice(){

finalint basePrice = _quantity * _itemPrice;

finaldouble discountFactor;

if(basePrice > 1000){

discountFactor = 0.95;

}else {

discountFactor = 0.98;

}

returnbasePrice * discountFactor;

}

• 开始替换临时变量，先把赋值动作的右侧表达式提炼出来：

double getPrice(){

final intbasePrice = basePrice();

final doublediscountFactor;

if(basePrice > 1000){

discountFactor = 0.95;

}else {

discountFactor = 0.98;

}

returnbasePrice * discountFactor;

}

  

private int basePrice(){

return _quantity * _itemPrice;

}

• 把临时变量basePrice的引用点替换掉：

double getPrice(){

final doublediscountFactor;

if(basePrice() > 1000){

discountFactor = 0.95;

}else {

discountFactor = 0.98;

}

return basePrice() *discountFactor;

}

  

private int basePrice(){

return_quantity * _itemPrice;

}

• 提炼discountFactor():

double getPrice(){

returnbasePrice() * discountFactor();

}

  

private double discountFactor() {

if(basePrice() > 1000){

return 0.95;

}else {

return 0.98;

}

}

  

private int basePrice(){

return_quantity * _itemPrice;

}