java 代码重构-第一章(分解并重组statement())

?

2.1 把switch代码在方法中抽出

第一个明显引起我注意的就是长得离谱的statement() 。每当看到这样长长的函数，我就想把它大卸八块。要知道，代码区块愈小，代码的功能就愈容易管理，代码的处理和搬移也都愈轻松。

重构过程的第一阶段中，我将说明如何把长长的函数切开，并把较小块的代码移至更合适的class 内。我希望降低代码重复量，从而使新的（打印HTML 报表用的）函数更容易撰写。

第一个步骤是找出代码的逻辑泥团（logical clump）并运用 Extract Method。本例一个明显的逻辑泥团就是switch 语句，把它提炼（extract）到独立函数中似乎比较好。

和任何重构准则一样，当我提炼一个函数时，我必须知道可能出什么错。如果我提炼得不好，就可能给程序引入臭虫。所以重构之前我需要先想出安全作法。

首先我得在这段代码里头找出函数内的局部变量（local variables）和参数（parameters）。我找到了两个：each 和thisAmount，前者并未被修改，后者会被修改。任何不会被修改的变量都可以被我当成参数传入新的函数，至于会被修改的变量就需格外小心。如果只有一个变量会被修改，我可以把它当作返回值。thisAmount 是个临时变量，其值在每次循环起始处被设为0，并且在switch 语句之前不会改变，所以我可以直接把新函数的返回值赋予它。

下面展示重构前后的代码。重构前的代码在下面类对比.凡是从函数提炼出来的代码，以及新代码所做的任何修改，只要我觉得不是明显到可以一眼看出，就以粗体字标示出来特别提醒你。

?

上一篇文章，最开始的代码

Customer?

?

/** * 通计清单 * @return */public String statement(){    double totalAmount = 0;//合计    int frequentRentePoints = 0;    Enumeration<Rental> enu_rentals = rentals.elements();    String result = "Rental Record for "+ this.getName() + " ";    while(enu_rentals.hasMoreElements()){        double thisAmount = 0;        Rental each = enu_rentals.nextElement();        switch (each.getMovie().getPriceCode()) {        case Movie.REGULAR:            thisAmount += 2;            if(each.getDeysRented() > 2){                thisAmount += (each.getDeysRented() - 2) * 1.5;            }            break;        case Movie.NEW_RELEASE:            thisAmount += each.getDeysRented() * 3;            break;        case Movie.CHILDRENS:            thisAmount += 1.5;            if(each.getDeysRented() > 3){                thisAmount += (each.getDeysRented() -3 ) * 1.5;            }            break;        }        frequentRentePoints ++;        if((each.getMovie().getPriceCode() == Movie.NEW_RELEASE) && each.getDeysRented() > 1){            frequentRentePoints ++;        }        result += ""+each.getMovie().getTitle() + "" +String.valueOf(thisAmount) + "";        totalAmount += thisAmount;    }    result += "Amount owed is " + String.valueOf(totalAmount) + "";    result += "You earned " + String.valueOf(frequentRentePoints) + " frequent renter points";    return result;}

?

?

跟下面重构的代码对比(Customer 完整代码)

?

package com.mkfree.refactoring.shap1;import java.util.Enumeration;import java.util.Vector;/** * 顾客 * @author hk * * 2012-12-25 下午10:59:03 */public class Customer {    private String name;    private Vector<Rental> rentals = new Vector<>();    public Customer(String name) {        this.name = name;    }    /**     * 添加     * @param rental     */    public void addRentals(Rental rental){        rentals.add(rental);    }    public String getName() {        return name;    }    /**     * 通计清单     * @return     */    public String statement(){        double totalAmount = 0;//合计        int frequentRentePoints = 0;        Enumeration<Rental> enu_rentals = rentals.elements();        String result = "Rental Record for "+ this.getName() + " ";        while(enu_rentals.hasMoreElements()){            double thisAmount = 0;            Rental each = enu_rentals.nextElement();            amountFor(each);//计算一笔租片费            frequentRentePoints ++;            if((each.getMovie().getPriceCode() == Movie.NEW_RELEASE) && each.getDeysRented() > 1){                frequentRentePoints ++;            }            result += ""+each.getMovie().getTitle() + "" +String.valueOf(thisAmount) + "";            totalAmount += thisAmount;        }        result += "Amount owed is " + String.valueOf(totalAmount) + "";        result += "You earned " + String.valueOf(frequentRentePoints) + " frequent renter points";        return result;    }    /**     * 把计算一笔租片     * @param each     * @return     */    private int amountFor(Rental each) { // 计算一笔租片费。        int thisAmount = 0;        switch (each.getMovie().getPriceCode()) {        case Movie.REGULAR: // 普通片            thisAmount += 2;            if (each.getDeysRented() > 2)                thisAmount += (each.getDeysRented() - 2) * 1.5;            break;        case Movie.NEW_RELEASE: // 新片            thisAmount += each.getDeysRented() * 3;            break;        case Movie.CHILDRENS: // 儿童。            thisAmount += 1.5;            if (each.getDeysRented() > 3)                thisAmount += (each.getDeysRented() - 3) * 1.5;            break;        }        return thisAmount;    }}

?

?

?

每次做完这样的修改之后，我都要编译并测试。这一次起头不算太好——测试失败了，有两笔测试数据告诉我发生错误。一阵迷惑之后我明白了自己犯的错误。我愚蠢地将amountFor() 的返回值型别声明为int，而不是double 。

以上代码就已经消除了switch里的代码...是不是,整个类好看些呢呢?

?

?

本文章来自：http://blog.mkfree.com/posts/18

?