两个类之间有双向关联,但其中一个类如今不再需要另一个类的特性。
动机:
双向关联很有用,但你必须为它付出代价,那就是维护双向连接。
大量的双向连接也很容易造成“僵尸对象”。
双向关联也迫使两个类之间有了依赖:对其中一个类的任何修改,都可能引发另一个类的变化。
做法:
找出保存“你想去除的指针”的字段,检查它的每一个用户,判断是否可以去除该指针。
=》不但要检查直接访问点,也要检查调用这些直接访问点的函数。
=》考虑有无可能不通过指针取得被引用对象,如果有可能,你就可以对取值函数使用substitute algorithm,从而让客户在没有指针的情况下也可以使用该取值函数。
=》对于使用该字段的所有函数,考虑将被引用对象作为参数传进去。
如果客户使用了取值函数,先运用self encapsulate field将待删除字段自我封装起来,然后使用substitute algorithm对付取值函数,令它不再使用该字段。
如果客户未使用取值函数,那就直接修改待删除字段的所有被引用点:改以其他途径获得该字段所保存的对象。
如果已经没有任何函数使用待删除字段,移除所有对该字段的更新逻辑,然后移除该字段。
旧代码
class Order...
Customer getCustomer(){
return _customer;
}
void setCustomer(){
if(_customer != null)
_customer.friendOrders().remove(this);
_customer = arg;
if(_customer != null)
_customer.friendOrders().add(this);
}
private Customer _customer;
class Customer...
void addOrder(Order arg){
arg.setCustomer(this);
}
private Set _orders = new HashSet();
Set friendOrders(){
return _orders;
}
新代码
class Order...
double getDiscountPrice(Customer customer){
return getGrossPrice() * (1 - customer.getDiscount());
}
class customer...
double getPriceFor(Order order){
Assert.isTrue(_orders.contains(order));
return order.getdiscountedPrice(this);
Customer getCustomer(){
Iterator iter = Customer.getInstances().iterator();
while(iter.hasNext()){
if(each.containsOrder(this)) return each;
}
return null;
}
}