重构 demo篇

 本文背景为学习重构一书中的一些信息的记录。

该篇以一个影片租赁为背景。通过一层层的重构,最终实现相对来说比较完美的代码。

文中经典语句摘要:

如果它没有坏,就不要动它 。” 用来形容我们工作中一些比较古董级的代码,或许写的很烂,但是基本功能是OK的,对于一个团队leader来说,不会主张你去修改他,对于一个保守级的开发来说,也懒的去修改他,虽然你个人任务修改它出错的概率近乎为0,但是仍旧是近乎。只愿我能在工作中永远保持一个敬畏之心,而非畏惧之心。

1、第一版,较多的信息冗余在一起,相对书中的第一版已经有了一些提升,加上了个人在初步做的一些重构,怎么说哥也是写过两年代码的人。简单重构还是了解一些的。

 用户类,定义用户的名称以及租赁了哪些影片


package
com.woniu.refactoring; import java.util.Enumeration; import java.util.Vector; public class Customer { private String _name; private Vector _rentals = new Vector(); public String getName() { return _name; } public void setName(String _name) { this._name = _name; } public Vector getRentals() { return _rentals; } public void setRentals(Vector _rentals) { this._rentals = _rentals; } public String statement() { double totalAmout = 0; int frequentRenterPoints = 0; Enumeration rentals = _rentals.elements(); String result = "Rental Record for " + getName() + "\n"; while(rentals.hasMoreElements()) { double thisAmout = 0; Rental each = (Rental) rentals.nextElement(); //这里是个人阅读到这里时,认为需要重构的位置。用专业点的话叫提炼函数 TODO // switch (each.getMovie().getPriceCode()) { // case Movie.REGULAR: // thisAmout += 2; // if(each.getDaysRented() > 2) { // thisAmout += (each.getDaysRented() - 2) * 1.5; // } // break; // case Movie.NEW_RELEASE: // thisAmout += each.getDaysRented() * 3; // break; // // case Movie.CHILDENS: // thisAmout += 1.5; // if(each.getDaysRented() > 3) { // thisAmout += (each.getDaysRented() - 3) * 1.5; // } // break; // } //上述方法,经过提炼后得到这个相对来说可以重用的一个函数amounFor thisAmout = amountFor(each); //这里是个人阅读到这里时,认为需要重构的位置。用专业点的话叫提炼函数 TODO // frequentRenterPoints ++; // if((each.getMovie().getPriceCode() == Movie.NEW_RELEASE) && each.getDaysRented() > 1) { // frequentRenterPoints ++; // } //进过提炼后的积分计算方式 frequentRenterPoints = getFrequentRenterPoints(each); result += "\t" + each.getMovie().getTitle() + "\t" + String.valueOf(thisAmout) + "\n"; totalAmout += thisAmout; } result += "Amount owed is " + String.valueOf(totalAmout) + "\n"; result += "You earned " + String.valueOf(frequentRenterPoints) + " frequent renter points"; return result; } /** * 用于计算此次租赁产生的积分 * @param rental 租赁 * @return */ private int getFrequentRenterPoints(Rental rental) { if((rental.getMovie().getPriceCode() == Movie.NEW_RELEASE) && rental.getDaysRented() > 1) { return 2;//这里其实在最初我是没有想到可以直接用2来代替的 } return 1; } /** * 用于计算此次租赁消费金额 * @param rental * @return */ private double amountFor(Rental rental) { double thisAmout = 0; switch (rental.getMovie().getPriceCode()) { case Movie.REGULAR: thisAmout += 2; if(rental.getDaysRented() > 2) { thisAmout += (rental.getDaysRented() - 2) * 1.5; } break; case Movie.NEW_RELEASE: thisAmout += rental.getDaysRented() * 3; break; case Movie.CHILDENS: thisAmout += 1.5; if(rental.getDaysRented() > 3) { thisAmout += (rental.getDaysRented() - 3) * 1.5; } break; } return thisAmout; } }

 影片类,定义了影片的类型以及价格

package com.woniu.refactoring;

public class Movie {
    public static final int CHILDENS = 2;
    public static final int REGULAR = 0;
    public static final int NEW_RELEASE = 1;
    
    private String _title;
    private int _priceCode;
    
    public Movie(String title, int priceCode) {
        _title = title;
        priceCode = priceCode;
    }
    
    public int getPriceCode() {
        return _priceCode;
    }
    public void setPriceCode(int _priceCode) {
        this._priceCode = _priceCode;
    }
    public String getTitle() {
        return _title;
    }
    
    
}

 租赁类,记录影片被租赁的时间以及影片信息

package com.woniu.refactoring;

public class Rental {
    private Movie _movie;
    private int _daysRented;
    
    public Rental(Movie movie, int daysRented) {
        _movie = movie;
        _daysRented = daysRented;
    }
    
    public Movie getMovie() {
        return _movie;
    }
    public int getDaysRented() {
        return _daysRented;
    }
    
    
}

 第二版

由于在amountFor()方法中,并没有任何用户的信息,却在用户的类中实现了关于影片计算逻辑,金额的产生发生的位置应该是在租赁中产生的,因此,应该将其移至Rental中,使用(Move Mehtod)来进行处理。

用户类,封装了用户总消费额以及总积分的计算。同时,将关于租赁的单价计算与单积分计算进行了迁移。

package com.woniu.refactoring;

import java.util.Enumeration;
import java.util.Vector;

public class Customer {
    private String _name;
    private Vector _rentals = new Vector();
    
    public String getName() {
        return _name;
    }
    public void setName(String _name) {
        this._name = _name;
    }
    public Vector getRentals() {
        return _rentals;
    }
    public void setRentals(Vector _rentals) {
        this._rentals = _rentals;
    }
    
    
    public String statement() {
//        double totalAmout = 0;
//        int frequentRenterPoints = 0;
        Enumeration rentals = _rentals.elements();
        String result = "Rental Record for " + getName() + "\n";
        while(rentals.hasMoreElements()) {
            double thisAmout = 0;
            Rental each = (Rental) rentals.nextElement();
            //这里是个人阅读到这里时,认为需要重构的位置。用专业点的话叫提炼函数 TODO
//            switch (each.getMovie().getPriceCode()) {
//            case Movie.REGULAR:
//                thisAmout += 2;
//                if(each.getDaysRented() > 2) {
//                    thisAmout += (each.getDaysRented() - 2) * 1.5;
//                }
//                break;
//            case Movie.NEW_RELEASE:
//                thisAmout += each.getDaysRented() * 3;
//                break;
//
//            case Movie.CHILDENS:
//                thisAmout += 1.5;
//                if(each.getDaysRented() > 3) {
//                    thisAmout += (each.getDaysRented() - 3) * 1.5;
//                }
//                break;
//            }
            //上述方法,经过提炼后得到这个相对来说可以重用的一个函数amounFor
//            thisAmout = amountFor(each);
            thisAmout = each.getCharge();//这是第二版的重构,将金额计算与积分计算,移至了Rental中,因为这是在租赁过程中产生的
            //这里是个人阅读到这里时,认为需要重构的位置。用专业点的话叫提炼函数 TODO
//            frequentRenterPoints ++;
//            if((each.getMovie().getPriceCode() == Movie.NEW_RELEASE) && each.getDaysRented() > 1) {
//                frequentRenterPoints ++;
//            }
            //进过提炼后的积分计算方式
//            frequentRenterPoints = getFrequentRenterPoints(each);
//            frequentRenterPoints = each.getFrequentRenterPoints();//这是第二版的重构,将金额计算与积分计算,移至了Rental中,因为这是在租赁过程中产生的
            result += "\t" + each.getMovie().getTitle() + "\t" + String.valueOf(thisAmout) + "\n";
//            totalAmout += thisAmout;//2书中提到该处可以直接使用getCharge()获取金额,进而避免了局部变量的使用,但是,我没有找到关于多次计算的优化点,因此不敢王嘉苟同。
            
        }
        
        result += "Amount owed is " + String.valueOf(getTotalCharge()) + "\n";//对于总金额计算单独提取方法,以备后用
        result += "You earned " + String.valueOf(getTotalFrequentRenterPoints()) + " frequent renter points";
        return result;
    }
    
    /**
     * 获取用户消费的总金额
     * @return
     */
    private double getTotalCharge() {
        double totalAmout = 0;
        Enumeration rentals = _rentals.elements();
        while(rentals.hasMoreElements()) {
            Rental rental = (Rental)rentals.nextElement();
            totalAmout += rental.getCharge();
        }
        return totalAmout;
    }
    
    /**
     * 获取用户消费的总积分
     * @return
     */
    private int getTotalFrequentRenterPoints() {
        int result = 0;
        Enumeration rentals = _rentals.elements();
        while(rentals.hasMoreElements()) {
            Rental rental = (Rental)rentals.nextElement();
            result += rental.getFrequentRenterPoints();
        }
        return result;
    }
    
    /**
     * 用于计算此次租赁产生的积分
     * @param rental 租赁
     * @return
     */
    @Deprecated
    private int getFrequentRenterPoints(Rental rental) {
        if((rental.getMovie().getPriceCode() == Movie.NEW_RELEASE) && rental.getDaysRented() > 1) {
            return 2;//这里其实在最初我是没有想到可以直接用2来代替的
        }
        return 1;
    }
    
    /**
     * 用于计算此次租赁消费金额
     * @param rental
     * @return
     */
    @Deprecated
    private double amountFor(Rental rental) {
        return rental.getCharge();//这里其实没必要再次进行封装函数了。可以直接在循环中使用rental的getCharge方法进行计算。
//        double thisAmout = 0;
//        switch (rental.getMovie().getPriceCode()) {
//        case Movie.REGULAR:
//            thisAmout += 2;
//            if(rental.getDaysRented() > 2) {
//                thisAmout += (rental.getDaysRented() - 2) * 1.5;
//            }
//            break;
//        case Movie.NEW_RELEASE:
//            thisAmout += rental.getDaysRented() * 3;
//            break;
//
//        case Movie.CHILDENS:
//            thisAmout += 1.5;
//            if(rental.getDaysRented() > 3) {
//                thisAmout += (rental.getDaysRented() - 3) * 1.5;
//            }
//            break;
//        }
//        return thisAmout;
    }
    
}

 租赁类,增加了关于租赁单价的计算以及积分计算。

package com.woniu.refactoring;

public class Rental {
    private Movie _movie;
    private int _daysRented;
    
    public Rental(Movie movie, int daysRented) {
        _movie = movie;
        _daysRented = daysRented;
    }
    
    public Movie getMovie() {
        return _movie;
    }
    public int getDaysRented() {
        return _daysRented;
    }
    
    /**
     * 用户此次租赁积分的计算
     * @param rental
     * @return
     */
    public int getFrequentRenterPoints() {
        if((getMovie().getPriceCode() == Movie.NEW_RELEASE) && getDaysRented() > 1) {
            return 2;//这里其实在最初我是没有想到可以直接用2来代替的
        }
        return 1;
    }
    
    
    /**
     * 用于计算此次租赁消费金额
     * 2关于这个方法,其实应该是在租赁关系发生时,已经需要计算好金额了。这样就可以避免了Customer中提到的优化点,
     * 但是,这样同样会造成如果交易取消了,我浪费资源进行了计算。
     * @param rental
     * @return
     */
    public double getCharge() {
        double thisAmout = 0;
        switch (getMovie().getPriceCode()) {
        case Movie.REGULAR:
            thisAmout += 2;
            if(getDaysRented() > 2) {
                thisAmout += (getDaysRented() - 2) * 1.5;
            }
            break;
        case Movie.NEW_RELEASE:
            thisAmout += getDaysRented() * 3;
            break;

        case Movie.CHILDENS:
            thisAmout += 1.5;
            if(getDaysRented() > 3) {
                thisAmout += (getDaysRented() - 3) * 1.5;
            }
            break;
        }
        return thisAmout;
    }
    
}

 

 原本到这里,我理解,此次重构已经结束了。但发现,后续还有新的重构方向。。。。。

 

posted @ 2018-07-06 00:30  woniu4  阅读(201)  评论(0编辑  收藏  举报