两个类之间有双向关联,但其中一个类如今不再需要另一个类的特性。
动机:
双向关联很有用,但你必须为它付出代价,那就是维护双向连接。
大量的双向连接也很容易造成“僵尸对象”。
双向关联也迫使两个类之间有了依赖:对其中一个类的任何修改,都可能引发另一个类的变化。
做法:
找出保存“你想去除的指针”的字段,检查它的每一个用户,判断是否可以去除该指针。
=》不但要检查直接访问点,也要检查调用这些直接访问点的函数。
=》考虑有无可能不通过指针取得被引用对象,如果有可能,你就可以对取值函数使用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; } }