你也可以说这并不是一个真正的重构,因为方法实际上改变了,但这是一个灰色地带,可以开放讨论。一个拥有大量布尔类型参数的方法将很快变得无法控制,产生难以预期的行为。参数的数量将决定分解的方法的数量。来看看该重构是如何开始的:
public class BankAccount {
public void CreateAccount(Customer customer, boolean withChecking, boolean withSavings, boolean withStocks) {
// do work
}
}
要想使这样的代码运行得更好,我们可以通过命名良好的方法暴露布尔参数,并将原始方法更改为private
以阻止外部调用。显然,你可能需要进行大量的代码转移,也许重构为一个Parameter Object会更有意义。
public class BankAccount {
public void CreateAccountWithChecking(Customer customer) {
CreateAccount(customer, true, false);

}
public void CreateAccountWithCheckingAndSavings(Customer customer) {
CreateAccount(customer, true, true);
}
private void CreateAccount(Customer customer, Boolean withChecking, Boolean withSavings) {
// do work
}
}




posted on 2016-08-19 10:09  jgig11  阅读(207)  评论(0编辑  收藏  举报