两个类之间有双向关联,但其中一个类如今不再需要另一个类的特性。

动机:

双向关联很有用,但你必须为它付出代价,那就是维护双向连接。

大量的双向连接也很容易造成“僵尸对象”。

双向关联也迫使两个类之间有了依赖:对其中一个类的任何修改,都可能引发另一个类的变化。

做法:

找出保存“你想去除的指针”的字段,检查它的每一个用户,判断是否可以去除该指针。

=》不但要检查直接访问点,也要检查调用这些直接访问点的函数。

=》考虑有无可能不通过指针取得被引用对象,如果有可能,你就可以对取值函数使用substitute algorithm,从而让客户在没有指针的情况下也可以使用该取值函数。

=》对于使用该字段的所有函数,考虑将被引用对象作为参数传进去。

如果客户使用了取值函数,先运用self encapsulate field将待删除字段自我封装起来,然后使用substitute algorithm对付取值函数,令它不再使用该字段。

如果客户未使用取值函数,那就直接修改待删除字段的所有被引用点:改以其他途径获得该字段所保存的对象。

如果已经没有任何函数使用待删除字段,移除所有对该字段的更新逻辑,然后移除该字段。

旧代码

 

[java] view plain copy
 
  1. class Order...  
  2. Customer getCustomer(){  
  3.     return _customer;  
  4. }  
  5. void setCustomer(){  
  6.     if(_customer != null)  
  7.         _customer.friendOrders().remove(this);  
  8.         _customer = arg;  
  9.     if(_customer != null)  
  10.         _customer.friendOrders().add(this);  
  11. }  
  12. private Customer _customer;  
  13.   
  14. class Customer...  
  15. void addOrder(Order arg){  
  16.     arg.setCustomer(this);  
  17. }  
  18. private Set _orders = new HashSet();  
  19. Set friendOrders(){  
  20.     return _orders;  
  21. }  



新代码

 

[java] view plain copy
 
    1. class Order...  
    2. double getDiscountPrice(Customer customer){  
    3.     return getGrossPrice() * (1 - customer.getDiscount());  
    4. }  
    5.   
    6. class customer...  
    7. double getPriceFor(Order order){  
    8.     Assert.isTrue(_orders.contains(order));  
    9.     return order.getdiscountedPrice(this);  
    10.   
    11. Customer getCustomer(){  
    12.     Iterator iter = Customer.getInstances().iterator();  
    13.     while(iter.hasNext()){  
    14.         if(each.containsOrder(this)) return each;  
    15.     }  
    16.     return null;  
    17. }  
    18. }  
posted on 2018-02-08 17:28  Sharpest  阅读(244)  评论(0)    收藏  举报