《重构》学习笔记

最近在读《重构》这本书，试着把书中对我启发较大的一些点记录下来，以便回顾学习。

一、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类即可，易于维护。

• 额外说明：

当服务端的委托函数过多时，可能之前的重构方案出现问题，需要采用新的重构方案。
如让客户端依赖受委托的对象。

---

当然，进行重构时还应该具体情况具体分析。

持续更新

如有不正确的地方，欢迎指出！谢谢大家！