程序中,有个函数与其所驻类之外的另一个类进行更多交流:调用后者,或被后者调用。
在该函数最常引用的类中建立一个有着类似行为的新函数。将旧函数变成一个单纯的委托函数,或是将旧函数完全移除。
动机:
如果一个类有太多行为,或如果一个类与另一个类有太多合作而形成高度耦合,我就会搬移函数。
使用另一个对象的次数比使用自己所驻对象的次数还多。观察调用它的那一端,它调用的那一端,以及继承体系中它的任何一个重定义函数。
做法:
检查源类中被源函数所使用的一切特性,考虑它们是否也该被搬移。
=》如果某个特性只被你打算搬移的那个函数用到,就应该将它一并搬移。如果另有其他函数使用了这个特性,你可以考虑将使用该特性的所有函数全都一并搬移。有时候,搬移一组函数比逐一搬移简单些。
检查源类的子类和超类,看看是否有该函数的其他声明。
=》如果出现其他声明,你或许无法进行搬移,除非目标类也同样表现出多态性。
在目标类中声明这个函数
=》你可以为此函数选择一个新名称,对目标类更有意义的名称。
将原函数的代码复制到目标函数中。调整后者,使其能在新家中正常运行。
=》如果目标函数使用了源类中的特性,你的决定如何从目标函数引用源对象。如果目标类中没有相应的引用机制,就把源对象的引用当做参数,传给新建立的目标函数。
=》如果源函数包含异常处理,你得判断逻辑上应该由哪个类来处理这一异常。如果应该由源类来负责,就把异常处理留在原地。
决定如何从源函数正确引用目标对象。
=》可能会有一个现成的字段或函数帮助你取得目标对象。如果没有,就看能否轻松建立一个这样的函数。如果还是不能,就得在源类中新建一个字段来保存目标对象。这可能是一个永久性修改,但你也可以让它是暂时的,因为后续的其他重构项目可能会把这个新建字段去掉。
修改源函数,使之成为一个纯委托函数。
决定是否删除源函数,或将它当做一个委托函数保留下来。
如果要移除源函数,请将源类中对源函数的所有调用,替换为目标函数的调用。
旧代码
class Account...
double overdraftCharge(){
if(_type.isPremium()){
double result = 10;
if(_daysOverDraw > 7) result += (_daysOverdrawn - 7) * 0.85;
}
else return _daysOverdrawn * 1.75;
}
double bankcharge(){
double result = 4.5;
if(_daysOverdrawn > 0) result += overdraftCharge();
else return result;
}
private AccountType _type;
private int _daysOverdrawn;
新代码
如果只有但特性交叉。
class AccountType...
double overdraftCharge(int daysOverdrawn){
if(isPremium()){
double result = 10;
if(daysOverdrawn > 7) result += (daysOverdrawn - 7) * 0.85;
return result;
}
else return daysOverdrawn * 1.75;
}
class Account ..
double overdraftcharge(){
return _type.overdraftCharge(_daysOverdrawn);
}
class Account...
double bankcharge(){
double result = 4.5;
if(_daysOverdrawn > 0)
result += _type.overdraftCharge(_daysOverdrawn);
return result;
}
如果有多特性和函数交叉
class AccountType...
double overdraftCharge(Account account){
if(isPremium()){
double result = 10;
if(account.getDaysOverdrawn() > 7) result += (account.getDaysOverdrawn() - 7) * 0.85;
return result;
}
else return account.getDaysOverdrawn * 1.75;
}
class Account ..
double overdraftcharge(){
return _type.overdraftCharge(_daysOverdrawn);
}
class Account...
double bankcharge(){
double result = 4.5;
if(_daysOverdrawn > 0)
result += _type.overdraftCharge(_daysOverdrawn);
return result;
}