taocsdn的专栏

Wangtao's Blog

Chain Constructors

你拥有多个构造子,其中包含了重复的代码。

将构造子串在一起,以使重复代码减到最少。

public class Loan {

...

public Loan(float notional, float outstanding, int rating, Date expiry) {

this.strategy = new TermROC();

this.notional = notional;

this.outstanding = outstanding;

this.rating =rating;

this.expiry = expiry;

}

public Loan(float notional, float outstanding, int rating, Date expiry, Date maturity) {

this.strategy = new RevolvingTermROC();

this.notional = notional;

this.outstanding = outstanding;

this.rating = rating;

this.expiry = expiry;

this.maturity = maturity;

        }

public Loan(CapitalStrategy strategy, float notional, float outstanding,

int rating, Date expiry, Date maturity) {

this.strategy = strategy;

this.notional = notional;

this.outstanding = outstanding;

this.rating = rating;

this.expiry = expiry;

this.maturity = maturity;

}

}



public class Loan {

...

public Loan(float notional, float outstanding, int rating, Date expiry) {

this(new TermROC(), notional, outstanding, rating, expiry, null);

}

public Loan(float notional, float outstanding, int rating, Date expiry, Date maturity) {

this(new RevolvingTermROC(), notional, outstanding, rating, expiry, maturity);

}

public Loan(CapitalStrategy strategy, float notional, float outstanding,

int rating, Date expiry, Date maturity) {

this.strategy = strategy;

this.notional = notional;

this.outstanding = outstanding;

this.rating = rating;

this.expiry = expiry;

this.maturity = maturity;

}

}

动机
在同一个类的两个或更多的构造子中编写重复代码,这就是在为自己埋下麻烦的种子。别人会在你的类中添加新的变量,然后更新一个构造子来对这个变量进行初始化,但是却忘了更新别的构造子。于是,“砰”的一声,向新的bug问好吧。一个类中的构造子越多,代码的重复就会伤害你越重。如果有可能,就应该尽量减少或去除代码重复,这样做的额外好处就是可以帮助你的代码系统减肥。

为了达到这个目标,我们经常会使用Constructor Chaining模式进行重构:特化的(specific)构造子调用普化的(general-purposed)构造子,重复这个过程,直到最普化的构造子也被调用到。如果你的每条调用链的末端都是同一个构造子,我就把它叫做“catch-all”构造子,因为它处理了所有构造子的调用。这个catch-all构造子通常会比其他构造子接受更多的参数,并且可能是(也可能不是)私有的或保护的。

如果你发现多个构造子降低了类的可用性,请考虑使用“用Factory Method模式替换多个构造子”的重构方法。

通信
重复
简化

如果一个类中的多个构造子在实现重复的工作,那么在特化与普化的通信上,你的代码就是失败的。要实现这种通信,就应该让特化的构造子调用普化的,并让每个构造子都做自己独一无二的工作。
构造子中的重复代码让你的类更容易出错、更难维护。寻找通用的功能,将它放到普化的构造子中,将调用转发给这些普化的构造子,并在其他的构造子中实现用途不广泛的功能。
如果超过一个的构造子包含同样的代码,要看出构造子之间的区别就很困难了。让特化的构造子调用普化的,并形成一条调用链,从而对构造子进行简化。

过程
1.       寻找包含重复代码的两个构造子(我把它们分别叫做A和B)。确定A是否可以调用B或者B是否可以调用A,这样重复代码才可以被安全的(和比较容易的)从这某一个构造子中删掉。

2.       编译、测试。

3.       对类中的每个构造子,重复步骤1和2,以获得尽量少的重复代码。

4.       如果某个构造子不必要成为公开的,就改变它的可见性。

5.       编译、测试。

范例
1.       我们将从一段简单代码——一个Loan类——开始。这个类有三个构造子,用来表现三种不同类型的贷款业务。大量丑陋的重复代码。

public Loan(float notional, float outstanding, int rating, Date expiry) {

this.strategy = new TermROC();

this.notional = notional;

this.outstanding = outstanding;

this.rating = rating;

this.expiry = expiry;

}

public Loan(float notional, float outstanding, int rating, Date expiry, Date maturity) {

this.strategy = new RevolvingTermROC();

this.notional = notional;

this.outstanding = outstanding;

this.rating = rating;

this.expiry = expiry;

this.maturity = maturity;

}

public Loan(CapitalStrategy strategy, float notional, float outstanding, int rating,

Date expiry, Date maturity) {

this.strategy = strategy;

this.notional = notional;

this.outstanding = outstanding;

this.rating = rating;

this.expiry = expiry;

this.maturity = maturity;

}

我研究了前面的两个构造子。它们包含重复的代码,而第三个构造子也同样。我考虑第一个构造子应该调用哪一个,并发现它应该调用第三个构造子。因此我把第一个构造子修改成:

public Loan(float notional, float outstanding, int rating, Date expiry) {

this(new TermROC(), notional, outstanding, rating, expiry, null);

}

2.       编译程序,测试这次修改是否正常工作。

3.       重复步骤1和2,尽量去除代码重复。这引出了第二个构造子,它也调用第三个构造子,如下所示:

public Loan(float notional, float outstanding, int rating, Date expiry, Date maturity) {

this(new RevolvingTermROC(), notional, outstanding, rating, expiry, maturity);

}

      现在我知道第三个构造子就是我的catch-all构造子,因为它处理了所有的构造细节。

4.       检查这三个构造子所有的调用者,以确定是否可以改变某一个的可见性。在这个案例中,我不能这样做(假设是这样——在这里你无法知道其他代码的情况)。

5.       编译并测试,完成此次重构。

阅读更多
想对作者说点什么? 我来说一句

没有更多推荐了,返回首页

加入CSDN,享受更精准的内容推荐,与500万程序员共同成长!
关闭
关闭