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