代码重构--switch的惊恐现身

switch作为条件判断(分支结构)中的一种方式，以至于我们对于它使用的频率处于较高水平的水平线上，为此我们应该使用Extra method来对这类判断条件进行抽取，另外从我自身而言，我发现我以前常常在当中根据不同的值，做不同的操作，而这类操作我是直接写在case里面的！而这里《重构》的作者并没有明确的讲明这样做的好处到底在哪里？我们现在就拿我们前一篇文章来作接下来的讲解。

没有看过前一篇文章的，点击这里带你看哦

现在我们模拟一个场景，作为项目经理的老王，告诉程序员小新说：现在我们的影片类型需要在增加一个叫做VIP，而这个类型租赁价格的计算当然也是不同，那么这个getCharge的分支中就不得不做接下来一步的修改，就是将switch中的条件在多加一条，如下

 public double getCharge() {
         double result = 0;
         switch(getMovie().getPriceCode()) {
         case Movie.REGULAR:
             result += 2;
             if (getDaysRented() > 2)
                 result += (getDaysRented() - 2)*1.5;
             break;
         case Movie.NEW_RELEASE:
             result += getDaysRented() * 3;
             break;
         case Movie.CHILDRENS:
             result += 15;
             if (getDaysRented() > 3)
                 result += (getDaysRented() - 3) * 1.5;
             break;
         case Movie.VIP:
             result += getDaysRented() * 3 + 10;
             break;
         }
         return result;
     }

不知道大家有没有发现什么问题呢？是这样子的哈！小新在编写他的VIP类型的价格换算逻辑的时候，他可能无意间动到了儿童类型的价格换算业务，那么当我们如果做的并非是如此简单的项目的时候，我们应当意识到，每个不同类型的逻辑处理代码在一个分支中不应该体现出来，也就是说，分支这里只是为了决定选择谁作为代表来运行某种运算，但是这个代表是如何进行运算的，它内部的实现是如何进行的我们其实是不需要去知道的，所以我们这里应该使用多态的方式，将每个不同的逻辑封装到对应的类中，并由接口开放出来一个开关，以控制在筛选完成之后对于事件的驱动！