`
feikiss
  • 浏览: 99983 次
  • 性别: Icon_minigender_1
  • 来自: 西安
社区版块
存档分类
最新评论

clean-code: 如何写出短小的switch语句。

 
阅读更多
读clean code有感,加上自己的一些心得,作此笔记。

写出只做一件事的switch语句很难,Switch天生要做N件事,不行我们总无法避免使用switch语句,不过还是能够确保每个switch都埋藏在较低的抽象层级,而且永远都不会重复。当然,我们利用多态来实现这一点。
请看下面的代码,它呈现了可能依赖于雇员类型的仅仅一种操作。
代码, Payroll.java:
 public Money calculatePay(Employee e) thorws InvalidEmployeeType {
     switch(e.type)  {
        case COMMISSIONED:
             return calculateCommissionedPay(e);
        case HOURLY:
             return calculateHourlyPay(e);
        case SALARIED:
              return calculateSalariedPay(e);
        default:
              throw new InvalidEmployeeType(e.type);
     }
}

该函数有好几个问题,首先,太长,当出现新的雇员时,它还会更长,其次,明显做了不止一件事。第三,违反了单一权责原则,因为有好几个修改它的理由。第四,违反了开-关原则,因为每当添加新的类型时,就必须修改它。不过,该函数最麻烦的可能是到处皆有类似结构的函数。例如,可能会有:
isPayday(Employ e, Date date)
deliverPay(Employee e, Money pay)

该问题的解决方案是将switch语句埋到抽象工厂底下,不让任何人看到。该工厂使用switch语句为Employee的派生类创建适当的实体,而不同的函数,如calculatePay、isPayday和deliverPay等,则有Employee接口动态的接受派遣。
代码, Employee和工厂
public interface Employee{
	public boolean isPayday();
	public Money calculatePay();
	public void deliverPay(Money pay);
	
}
public interface EmployeeFactory{
	public Employee makeEmployee(EmployeeRecord r) throws InvalidEmployeeType;
}
public class EmployeeFactoryImpl implements EmployeeFactory{
	public Employee makeEmployee(EmployeeRecord r) throws InvalidEmployeeType{
		switch(r.type){
		case COMMISSIONED:
			return new CommissionedEmployee(r);
		case HOURLY:
			return new HourlyEmployee(r);
		case SALARIED:
			return new SalariedEmployee(r);
		default:
			throw new InvalidEmployeeType(r.type);
		}
	}
}

这样,每当新增加一种职员的时候,就不用像清单一种去修改客CalculatePay()方法,同时添加新的"重复性的"实例方法,只需要新生成一个Employee的子类,并在工厂EmployeeFactoryImpl中添加一条新的返回类型就行,大大减少了耦合性。当然,这些都是视情况决定的。
1
1
分享到:
评论

相关推荐

Global site tag (gtag.js) - Google Analytics