重构第十一章
2.Pull Up Method(函数上移)
有些函数,在各个subclass中产生完全相同的结果,将该函数移至superclass。
Example:
class RegularCustomer extends Customer{
void createBill(date Date) {
double chargeAmount = chargeFor(lastBillDate, date);
addBill(date, charge);
}
}
class PerferredCustomer extends Customer{
void createBill(date Date) {
double chargeAmount = chargeFor(lastBillDate, date);
addBill(date, charge);
}
}
End:
class Customer {
void createBill(date Date) {
double chargeAmount = chargeFor(lastBillDate, date);
addBill(date, charge);
}
abstract double chargeFor(date start, date end);
}
class PerferredCustomer extends Customer{
double chargeFor(date start, date end) {
...
}
}
class RegularCustomer extends Customer{
double chargeFor(date start, date end) {
...
}
}
Conclusion:
重复的代码,意味着你会面临着[修改一个却未能修改另一个]的风险,所以避免重复是很重要的。
在各个subclass中,可能会有重复的动作,这是最显而易见的Pull Up Method(函数上移)适用场景;有一种特殊情况也需要使用Pull Up Method(函数上移)方法:subclass的函数覆写了superclass的函数,但是仍然做相同的工作。
注意
重构必须在有单元测试的情况下,保证之前的功能修改后不收影响。切记!!!
网友评论