范例（Examples）
本例从Change Unidirectional association to Bidirectional(197)留下的代码开始进行,其中Customer和Order之间有双向关联:
class Order...
Customer getCustomer() {
return _customer;
}
void setCustomer(Custoemr arg) ...
if(_customer != null) _customer.friendOrders().remove(this);
_customer = arg;
if(_customer != null) _customer.friendOrders().add(this);
}
private Customer _customer; //译注:这是Order-to-Customer link也是本例的移除对象.
class Customer ...
void addOrder(Order arg) {
arg.setCustomer(this);
}
private Set _orders = new HashSet();
//译注:以上是Customer-to-Order link
Set friendOrders() {
return _orders;
}
后来我发现,除非先有Customer对象,否则不会存在Order对象.因此我想将[从Order到Customer的连接]移除掉.
对于本项重构来说,最困难的就是检查可行性.如果我知道本项重构是安全的,那么重构手法自身十分简单.问题在于是否有任何代码倚赖_customer值域的存在.如果确实有,那么在删除这个值域之后,我必须提供替代品.
首先,我需要研究所有读取这个值域的函数,以及所有使用这些函数的函数.我能找到另一条途径来供应Customer对象吗----这通常意味将Customer对象作为引数(argument)传递给其用户(某函数).下面是一个简化例子:
class Order...
double getDiscountedPrice() {
return getGrossPrice() * (1 - _customer.getDiscount());
}
改变为:
class Order...
double getDiscountedPrice(Customer customer) {
return getGrossPrice() * (1 - customer.getDiscount());
}
如果待改函数是被Customer对象调用的,那么这样的修改方案特别容易实施,因为Customer对象将自己作为引数(argument)传给函数很是容易.所以下列代码:
class Customer...
double getPriceFor(Order order) {
Assert.isTrue(_orders.contains(order)); //see Introduce Assertion(267)
return order.getDiscountedPrice();
变成了:
class Customer...
double getPriceFor(Order order) {
Assert.isTrue(_orders.contains(order));
return order.getDiscountedPrice(this);
另一个作法就是修改取值函数(getter),使其在不使用_customer值域的前提下返回一个Customer对象.如果这行得通,我就可以使用Substitute Algorithm(139)修改Order.getCustomer()函数算法.我有可能这样修改代码:
Customer getCustomer() {
Iterator iter = Customer.getInstance().iterator();
while(iter.hasNext()) {
Customer each = (Customer)iter.next();
if(each.containsOrder(this) return each;
}
return null;
}
这段代码比较慢,不过确实可行.而且,在数据库环境下,如果我需要使用数据库查询语句,这段代码对系统性能的影响可能并不显著.如果,Order class中有些函数使用_customer值域,我可以实施Self Encapsulate Field(171)令它们转而改用上述的getCustomer()函数.
如果我要保留上述的取值函数(getter),那么Order和Customer的关联从接口上看虽然仍然是双向,但实现上已经是单向关系了.虽然我移除了反向指针,但两个classes彼此之间的依存关系(inter-dependencies)仍然存在.
如果我要替换取值函数(getter),那么我就专注地替换它,其他部分留待以后处理.我会逐一修改取值函数的调用者.让它们通过其他来源取得Customer对象.每次修改后都编译并测试.实际工作中这一过程往往相当快.如果这个过程让我觉得很棘手很复杂,我会放弃本项重构.
一旦我消除了_customer值域的所有读取点,我就可以着手处理[对此值域进行赋值动作]的函数了.很简单,只要把这些赋值动作全部移除,再把值域一并删除,就行了.由于已经没有任何代码需要这个值域,所以删掉它并不会带来任何影响.