重构读书笔记-8_8-Change_Bidirectional_

重构第八章

8.Change Bidirectional Association to Unidirectional(将双向关联改为单向)

两个classes之间有双向关联，但其中一个class如今不再需要另一个class的特性。去除不必要的关联。

Example:

class Order...
    Customer getCustomer() {
        return _customer;
    }
    void setCustomer(Customer arg) {
        if(_customer != null) _customer.friendOrders().remove(this);
        _custoemr = 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 getDiscountedPrice(Customer customer) {
        return getGrossPrice() * (1 - customer.getDiscount());
    } 

Ananlyse:
示例中，customer和order拥有着双向的连接，他们彼此保留着对对方class的关联。但是这有一些问题:
1.复杂度提高
2.容易造成对象已经死亡，但是没有清除的情况
3.造成紧耦合的情况
目前，假设，我需要去除Order中->Customer的连接:
去除Order 类中的_Customer成员变量，通过其他的方式提供。修改有关_customer的所有引用点。

End:

class Order...
    Customer getCustomer() {
        Iterator iter = Customer.getInstances().iterator();
        while(iter.hasNext()) {
            Customer each = (Customer) iter.next();
            if(each.containsOrder(this)) return each;
        }
        return null;
    }

    double getDiscountedPrice() {
        return getGrossPrice() * (1 - getCustomer().getDiscount());
    }

Conclusion:

修改了取值函数之后，虽然表面上Order class和Customer Class依然保有着双向连接，但是其实，已经是单向链接了，不过其彼此的依存关系依旧存在。
一旦我们将所有_customer的引用点都替换成getCustomer()函数，再讲_customer的设值函数和值域都剔除掉，本项重构就算完成了。

我们还可以通过传入Customer class 类型的参数，来去除掉两者的连接关系。

class Order...
    double getDiscountedPrice(Customer customer) {
        return getGrossPrice() * (1 - customer.getDiscount());
    }

注意

重构必须在有单元测试的情况下，保证之前的功能修改后不收影响。切记！！！