switch作为条件判断(分支结构)中的一种方式,以至于我们对于它使用的频率处于较高水平的水平线上,为此我们应该使用Extra method来对这类判断条件进行抽取,另外从我自身而言,我发现我以前常常在当中根据不同的值,做不同的操作,而这类操作我是直接写在case里面的!而这里《重构》的作者并没有明确的讲明这样做的好处到底在哪里?我们现在就拿我们前一篇文章来作接下来的讲解。
现在我们模拟一个场景,作为项目经理的老王,告诉程序员小新说:现在我们的影片类型需要在增加一个叫做VIP,而这个类型租赁价格的计算当然也是不同,那么这个getCharge的分支中就不得不做接下来一步的修改,就是将switch中的条件在多加一条,如下
1 public double getCharge() { 2 double result = 0; 3 switch(getMovie().getPriceCode()) { 4 case Movie.REGULAR: 5 result += 2; 6 if (getDaysRented() > 2) 7 result += (getDaysRented() - 2)*1.5; 8 break; 9 case Movie.NEW_RELEASE: 10 result += getDaysRented() * 3; 11 break; 12 case Movie.CHILDRENS: 13 result += 15; 14 if (getDaysRented() > 3) 15 result += (getDaysRented() - 3) * 1.5; 16 break; 17 case Movie.VIP: 18 result += getDaysRented() * 3 + 10; 19 break; 20 } 21 return result; 22 }
不知道大家有没有发现什么问题呢?是这样子的哈!小新在编写他的VIP类型的价格换算逻辑的时候,他可能无意间动到了儿童类型的价格换算业务,那么当我们如果做的并非是如此简单的项目的时候,我们应当意识到,每个不同类型的逻辑处理代码在一个分支中不应该体现出来,也就是说,分支这里只是为了决定选择谁作为代表来运行某种运算,但是这个代表是如何进行运算的,它内部的实现是如何进行的我们其实是不需要去知道的,所以我们这里应该使用多态的方式,将每个不同的逻辑封装到对应的类中,并由接口开放出来一个开关,以控制在筛选完成之后对于事件的驱动!