《重构——改善既有代码》
Motivation
出现依恋情节或者职责不清,应将行为移动到正确的类中。
Mechanics
1 检查源函数中所使用一切特性(包括field和函数)
2 检查子类和父类是否也定义了该函数
3 先委托,然后根据实际情况,决定是否将对源函数的调用替换为对目标函数的调用。
Eclipse refactor菜单下直接有Move 选项,可以直接使用该选项完成Move Method重构。但实际应用中,还是有一些细致的不同,下面详细说明:
默认情形:[Move] method, 必须要求源类中必须以目标类对象为成员变量,否则找不到目标对象。另外,移动后,默认总是以源类对象作为函数参数。
特殊情形1:目标类是源类的内部类,实际使用copy代码的方法。
特殊情形2:目标类中成员中已经有源类的对象,实际使用[Move] + [Rename] + [Change Method Signature]的方法
特殊情形3:有时候,只需要将源类的一个成员变量作为函数参数传入,而没有必要传入整个源类对象。实际使用[ExtractLocal Variable] + [Introduce Parameter] / [Extract Method] + [Move],
也可以使用[ExtractLocal Variable] + [Extract Method] + [Inline] + [Move]
double bankCharge(Account account) { double result = 4.5; if (account._daysOverdrawn > 0) result += account.overdraftCharge(); return result; }
// 1. 对需要move的函数中的成员变量,应用[Extract Local Variable]替换为局部变量,同时必然会增加“localVariable = _field”这样一句赋值语句。
publicclass Account {
double overdraftCharge() {
finalintdaysOverdrawn =_daysOverdrawn;
if (_type.isPremium()) {
double result = 10;
if (daysOverdrawn> 7)
result += (daysOverdrawn - 7) * 0.85;
return result;
} else
returndaysOverdrawn* 1.75;
}
// 2. 对上面新增的赋值语句"final int daysOverdrawn = _daysOverdrawn;",选中field " _daysOverdrawn",应用Introduce Parmeter,将该field作为函数参数传入。
public class Account {
double overdraftCharge(int dayOverdrawn) {
final
int
daysOverdrawn =
_daysOverdrawn;
final int daysOverdrawn = dayOverdrawn;
if ( _type.isPremium()){
// 3. 删除该赋值语句”localVariable = _field”。
public class Account {
double overdraftCharge(int dayOverdrawn) {
final
int
daysOverdrawn =
dayOverdrawn;
if ( _type.isPremium()){
=========================================================================================
还有一种更通用的调整方法,其核心思想是将旧方法接口委托给新方法,然后应用内联将所有旧方法替换为新方法,最后move。
第一步同上,第二步改为使用[Extract Method],将 赋值语句"final int daysOverdrawn = _daysOverdrawn;"以下的部分抽取为新的函数,函数名与目标函数相同,都是overdraftCharge.
public classAccount {
double overdraftCharge() {
final int daysOverdrawn = _daysOverdrawn;
return overdraftCharge(daysOverdrawn);
}
private double overdraftCharge(final int daysOverdrawn) {
if (_type.isPremium()){
double result = 10;
if (daysOverdrawn > 7)
result +=(daysOverdrawn - 7) * 0.85;
return result;
} else
return daysOverdrawn * 1.75;
}
//3. 使用[Inline]替换掉目标函数
先对daysOverdrawn应用[Inline]
double overdraftCharge() {
final
int
daysOverdrawn =
_daysOverdrawn;
return overdraftCharge(_daysOverdrawn);
}
再对overdraftCharge() 应用[Inline]
double
overdraftCharge() {
return
overdraftCharge(
_daysOverdrawn
);
}
原来调用overdraftCharge()的地方自动调整为 overdraftCharge(_daysOverdrawn)
result += overdraftCharge(); --> result += overdraftCharge(_daysOverdrawn);
over
以上完整源码可以在here下载