最近在读《重构》这本书,试着把书中对我启发较大的一些点记录下来,以便回顾学习。
一、replace temp with query(以查询取代临时变量)
在写程序的过程中,难免会在函数中定义一些临时变量,有些时候是无法避免的。但因为临时变量可能使用起来简单、快捷,因而有时会出现不恰当的使用。因为有些临时变量在定义之后只进行了一次赋值,那么这种临时变量是否有存在的价值就值得考虑。把临时相关的赋值操作抽象为一个函数,以一个恰当的函数名命名这个函数,那么便可将相应的计算封装起来,这样计的算规则封装在一个函数中,可以将变化隔离独立出来,便于修改。
以书中的一段代码为例:
double getPrice(){
int basePrice = _quantity * _itemPrice;
double discountFactor;
if(basePrice > 1000) {
discountFactor = 0.95;
} else {
discountFactor = 0.98;
}
return baseprice * discountFactor;
}
重构后的代码为:
double getPrice(){
return baseprice() * discountFactor();
}
int basePrice() {
return _quantity * _itemPrice;
}
double discountFactor() {
if(basePrice() > 1000) {
return 0.95;
} else {
return 0.98;
}
}
个人认为这样做的好处是basePrice和discountFactor的计算都隔离出来,如果发生变化容易修改。此外,discountFactor()函数的1000,0.95等数字的可以通过常量的方式定义,更加合理一点。
二、replace method with method object(以函数对象取代函数)
有时,可能会出现某个函数特别复杂,里面有大量的临时变量以及复杂的运算,那么我们可以采用“以函数对象取代函数”的方法来对这个复杂的函数内容进行重构。
做法为将复杂的函数中使用到的临时变量作为函数对象(MethodObject)的成员变量,原先函数中复杂的计算封装到函数对象的某一个方法中(如compute())。
这样即可以在原先复杂的函数中以new MethodObject(...).compute();取代原先的代码。这样的好处在于这段复杂的代码方便复用、重构以及变更。
三、Hide Delegate(隐藏委托关系)
- 首先是隐藏“委托关系”
有时客户端需要先通过服务对象A的方法得到另一个对象B,然后再调用B的函数b,这会产生一个“级联调用”。
有时我们需要隐藏这种“关系”,我们可以在在服务对象A上放置一个简单的“委托函数”a,在函数a的内部调用B的函数b。
这样的话客户端只需要调用A对象的a函数即可。
《重构》书中给出的例子如下:
class Person {
private Department department;
public Department getDepartment() {
return this.department;
}
public void setDepartment(Department department) {
this.department = department;
}
}
class Department {
private String code;
private Person manager;
public Department(Person manager) {
this.manager = manager;
}
public Person getPerson() {
return this.manager;
}
}
- 调用场景及问题如下如下:
要获取某部门的经理:manager = john.getDepartment().getManager();
这样的调用会暴露内部实现,并且调用端会和Department类产生耦合。
- 解决办法:
在Person类中添加一个委托方法:
public Person getManager() {
return department.getManager();
}
manager = john.getManager();
这样调用端就不需要知道Department类的信息,Department类发生变化时只需要通知Person类即可,易于维护。
- 额外说明:
当服务端的委托函数过多时,可能之前的重构方案出现问题,需要采用新的重构方案。
如让客户端依赖受委托的对象。
当然,进行重构时还应该具体情况具体分析。
持续更新
如有不正确的地方,欢迎指出!谢谢大家!