重构-改善既有代码的设计 读书笔记一

影片出租店的程序:计算每位顾客的消费金额并打印详单。
操作者告诉程序：顾客租了哪些影片，租期多长。程序根据租赁时间和影片类型计算费用。
影片分为3类：普通片，儿童片，新片。除了计算费用还要为常客计算积分，积分根据组片种类是否为新片而不同。

Movie(movieKind, name)
Rent(影片名称，租赁时间)
RentTotal(Rent列表)

阶段1：从statement()到htmlStatement()
使用的重构方法：
1.extract method(抽取逻辑，可能在多处使用的相同逻辑，只需要保留一份逻辑代码，而在使用时对该逻辑进行调用。
而不是写重复的逻辑代码。)

确保逻辑代码唯一，而不是逻辑代码重复。
即确保逻辑实体唯一，而不是逻辑实体重复。
确保只有一个该逻辑的实体，而不是多个该逻辑的实体。因为如果某个逻辑产生多个实体，必须维护它们的一致性。
唯一的逻辑实体 + 逻辑实体的多次调用。
原因：逻辑代码重复，当需要修改该逻辑代码时，必须确保该逻辑代码的所有实体都是一致性。
这个一致性维护工作难度根据逻辑实体分散程度增加，逻辑实体的个数增加，出错的概率也不断提升。

2.move method(函数搬家，函数应该放在它使用的数据的对象内)
调整修改代码，让其使用新函数，旧函数如果是public则可以保留，如果其他类还引用了旧函数，不需要修改它们。
3.replace temp with query。（使用函数而不是临时变量）

阶段2：结合变化量，影片的类型，限制变化的影响范围。本例中将影片类型的变化影响限制到Movie类中。

如果getCharge()和getRentPoints()方法在Movie类中，我们只需要在Movie类中修改类型，getCharge() getRentPoints()方法。
如果getCharge()和getRentPoints()方法在Rental类中，当影片类型发生变化，我们需要在Movie类中需改类型，并在Rental类中修改这2个方法。
尽管修改的东西都是一样的，但是后者在2个类中修改，前者在1个类中修改。显然前者更好。
所以把Rental中的getCharge()方法和getRentPoints()方法搬到Movie类中

方式：将Rental中的getCharge()方法和getRentPoints()方法搬到Movie类中

阶段3：使用多态替换类型代码
1.replace type code with state/strategy
1.self encapsulate field
2.move method
3.replace conditional with polymorphism

重构的节奏：测试小修改，测试小修改。。。

重构(名词)：对软件内部结构的一种调整，目的是在不改变软件可观察行为的前提下，提高其可理解性，降低其修改成本。
重构(动词)：使用一系列重构手法，在不改变软件可观察行为的前提下，调整其结构。
重构是对软件的小修改，但重构之中还可以包含另一个重构。

重构的目的：在不改变软件可观察行为的前提下，调整内部结构，使其更容易被理解和修改。
性能优化：在不改变软件可观察行为的前提下，调整内部结构，提高性能，但往往代码会变得更难理解。

使用重构技术开发软件:
时间分配给2种截然不同的行为：添加新功能，以及重构。
添加新功能时不应修改代码，只管添加功能。
重构时不应添加新功能，只管调整结构。
经常变换帽子，但无论何时都应该清楚自己戴的是哪顶帽子。

重构把我带到更高的理解层次。

重构使程序拥有良好的设计，而良好的设计使快速开发的根本。
如果没有良好的设计，某段时间你可能进展迅速，但恶劣的设计很快就让你的速度慢下来。你会花时间在调试上面，修改的时间
越来越长，因为你必须花更多的时间取理解系统、寻找重复的代码，随着你给程序打上一个有一个补丁，新特性需要更多的代码来实现。
真是个恶性循环。

何时重构？
重构本来就不是一件应该特别拨出时间做的事情，随时随地都可以。
如果你想做某件事情，而重构可以帮你把它做得更好，就用重构。
如果你想理解代码，而重构可以帮助你理解，那就重构。
如果你想添加新功能，重构可以帮助你理解代码，并更容易的添加新功能。

package shop;

public class Movie {
    public static final int NEW_RELEASE = 0;
    public static final int REGULAR = 1;
    public static final int CHILDREN = 2;

//    private int priceCode;
    private String title;
    private Price price;

    public Movie() {
    }

    public Movie(String title, int priceCode) {
        setPriceCode(priceCode);
        this.title = title;
    }

    public int getPriceCode() {
        return price.getPriceCode();
    }

    public void setPriceCode(int priceCode) {
        switch (priceCode){
            case Movie.NEW_RELEASE:
                price = new NewPrice();
                break;
            case Movie.REGULAR:
                price = new RegularPrice();
                break;
            case Movie.CHILDREN:
                price = new ChildrenPrice();
                break;
            default:
                throw new IllegalArgumentException("非法的影片类型：" + priceCode);
        }
    }

    public String getTitle() {
        return title;
    }

    public void setTitle(String title) {
        this.title = title;
    }

    public double getCharge(int daysRent){
        return price.getCharge(daysRent);
//        double result = 0;
//        switch(getPriceCode()){
//            case Movie.REGULAR:
//                result += 2;
//                if(daysRent > 2){
//                    result += (daysRent - 2) * 1.5;
//                }
//                break;
//            case Movie.NEW_RELEASE:
//                result += daysRent * 3;
//                break;
//            case Movie.CHILDREN:
//                result += 1.5;
//                if(daysRent > 3){
//                    result += (daysRent - 3) * 1.5;
//                }
//                break;
//        }
//        return result;
    }

    public int getRentPoints(int daysRent){
        return price.getRentPoints(daysRent);
//        int rentPoints = 1;
//        if(getPriceCode() == Movie.NEW_RELEASE
//                && daysRent > 1){
//            rentPoints++;
//        }
//        return rentPoints;
    }
}

package shop;

public class Rental {
    private Movie movie;
    private int daysRent;

    public Rental(){}

    public Rental(Movie movie, int daysRent){
        this.movie = movie;
        this.daysRent = daysRent;
    }

    public Movie getMovie() {
        return movie;
    }

    public void setMovie(Movie movie) {
        this.movie = movie;
    }

    public int getDaysRent() {
        return daysRent;
    }

    public void setDaysRent(int daysRent) {
        this.daysRent = daysRent;
    }

    public int getRentPoints(){
        return getMovie().getRentPoints(getDaysRent());
//        int rentPoints = 1;
//        if(getMovie().getPriceCode() == Movie.NEW_RELEASE
//                && getDaysRent() > 1){
//            rentPoints++;
//        }
//        return rentPoints;
    }

    public double getCharge(){
        return getMovie().getCharge(getDaysRent());
//        double result = 0;
//        switch(this.getMovie().getPriceCode()){
//            case Movie.REGULAR:
//                result += 2;
//                if(this.getDaysRent() > 2){
//                    result += (this.getDaysRent() - 2) * 1.5;
//                }
//                break;
//            case Movie.NEW_RELEASE:
//                result += this.getDaysRent() * 3;
//                break;
//            case Movie.CHILDREN:
//                result += 1.5;
//                if(this.getDaysRent() > 3){
//                    result += (this.getDaysRent() - 3) * 1.5;
//                }
//                break;
//        }
//        return result;
    }
}

package shop;

import java.util.ArrayList;

public class CustomerOrder01 {
    private String name;
    private ArrayList<Rental> rentals = new ArrayList<>();

    public CustomerOrder01(String name){
        this.name = name;
    }

    public void addRental(Rental rental){
        rentals.add(rental);
    }

    public String statement(){
        String result ="Rental Record for " + getName() + "
";
        for (Rental rental : rentals) {
//            double thisAmount = 0;
//            switch(rental.getMovie().getPriceCode()){
//                case Movie.REGULAR:
//                    thisAmount += 2;
//                    if(rental.getDaysRent() > 2){
//                        thisAmount += (rental.getDaysRent() - 2) * 1.5;
//                    }
//                    break;
//                case Movie.NEW_RELEASE:
//                    thisAmount += rental.getDaysRent() * 3;
//                case Movie.CHILDREN:
//                    thisAmount += 1.5;
//                    if(rental.getDaysRent() > 3){
//                        thisAmount += (rental.getDaysRent() - 3) * 1.5;
//                    }
//                    break;
//            }
//            thisAmount = amountFor(rental);
//            thisAmount = rental.getCharge();
//            rentPoints++;
//            if(rental.getMovie().getPriceCode() == Movie.NEW_RELEASE
//                && rental.getDaysRent() > 1){
//                rentPoints++;
//            }
            result += "	" + rental.getMovie().getTitle() + "	" + String.valueOf(rental.getCharge()) + "
";
        }
        result += "Amount owed is " + String.valueOf(getTotalAmount()) + "
";
        result += "You earned " + String.valueOf(getTotalRentPoints()) + " frequent renter points";
        return result;
    }

    public double amountFor(Rental rental){
//        double result = 0;
//        switch(rental.getMovie().getPriceCode()){
//            case Movie.REGULAR:
//                result += 2;
//                if(rental.getDaysRent() > 2){
//                    result += (rental.getDaysRent() - 2) * 1.5;
//                }
//                break;
//            case Movie.NEW_RELEASE:
//                result += rental.getDaysRent() * 3;
//                break;
//            case Movie.CHILDREN:
//                result += 1.5;
//                if(rental.getDaysRent() > 3){
//                    result += (rental.getDaysRent() - 3) * 1.5;
//                }
//                break;
//        }
//        return result;
        return rental.getCharge();
    }

    public double getTotalAmount(){
        double totalAmount = 0;
        for (Rental rental : rentals) {
            totalAmount += rental.getCharge();
        }
        return totalAmount;
    }

    public int getTotalRentPoints(){
        int totalRentPoints = 0;
        for (Rental rental : rentals) {
            totalRentPoints += rental.getRentPoints();
        }
        return totalRentPoints;
    }
    public String getName(){
        return name;
    }
}

package shop;

public class CustomerTest01 {
    public static void main(String[] args) {
        Movie movie1 = new Movie("阿凡达", Movie.NEW_RELEASE);
        Movie movie2 = new Movie("僵尸世界大战", Movie.REGULAR);
        Movie movie3 = new Movie("熔炉", Movie.CHILDREN);
        Movie movie4 = new Movie("星际穿越", Movie.REGULAR);

        Rental rental1 = new Rental(movie1,3);
        Rental rental2 = new Rental(movie2,5);
        Rental rental3 = new Rental(movie3,5);

        CustomerOrder01 customerOrder = new CustomerOrder01("liubei");
        customerOrder.addRental(rental1);
        customerOrder.addRental(rental2);
        customerOrder.addRental(rental3);

        System.out.println(customerOrder.statement());
    }
}

package shop;

public abstract class Price {
    public abstract int getPriceCode();

    public abstract double getCharge(int daysRent);

    public int getRentPoints(int daysRent){
        return 1;
//        int rentPoints = 1;
//        if(getPriceCode() == Movie.NEW_RELEASE
//                && daysRent > 1){
//            rentPoints++;
//        }
//        return rentPoints;
    }
}

package shop;

public class NewPrice extends Price{
    @Override
    public int getPriceCode() {
        return Movie.NEW_RELEASE;
    }

    @Override
    public double getCharge(int daysRent) {
        return daysRent * 3;
    }

    @Override
    public int getRentPoints(int daysRent) {
        return daysRent > 1 ? 2 : 1;
    }
}

package shop;

public class ChildrenPrice extends Price {
    @Override
    public int getPriceCode(){
        return Movie.CHILDREN;
    }

    @Override
    public double getCharge(int daysRent) {
        double result = 1.5;
        if(daysRent > 3){
            result += (daysRent - 3) * 1.5;
        }
        return result;
    }
}

package shop;

public class RegularPrice extends Price {
    @Override
    public int getPriceCode() {
        return Movie.REGULAR;
    }

    @Override
    public double getCharge(int daysRent) {
        double result = 2;
        if(daysRent > 2){
            result += (daysRent - 2) * 1.5;
        }
        return result;
    }
}