不良代码展示-父类中可以抽象出子类的行为

原创文章，如有转载，请注明出处：http://blog.csdn.net/yihui823/article/details/6932952

把一段代码，业务逻辑剥离后，给大家看看。

基类：

package testjava;

/**
 * 书的基类
 */
public class BaseBook {
    private int type = -1;
    
    public BaseBook(int type) {
        this.type = type;
    }
    
    /**
     * 打印页面
     */
    public void printPage() {
        if (type == 1) {
            System.out.println("Print with the green color.");
        } else if (type == 2) {
            System.out.println("Print with the blue color.");
        } else {
            System.out.println("Print with the white color.");
        }
    }
}

子类1：

package testjava;

/**
 * 绿色的书
 */
public class GreenBook extends BaseBook {
    public GreenBook(int type) {
        super(type);
    }
    
    /**
     * 得到颜色
     * @return 颜色 
     */
    public String getColor() {
        return "Green";
    }
}

子类2：

package testjava;

/**
 * 蓝色的书
 */
public class BlueBook extends BaseBook {
    
    public BlueBook(int type) {
        super(type);
    }
    
    /**
     * 得到颜色
     * @return 颜色 
     */
    public String getColor() {
        return "Blue";
    }
}

调用的类：

package testjava;

/**
 * 执行类
 */
public class Testjava {

    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        BlueBook bb = new BlueBook(2);
        bb.printPage();
        
        GreenBook gb = new GreenBook(1);
        gb.printPage();
    }
}

运行结果当然是对的。可是，子类的行为，却定义在了父类之中了。也就是说，基类的“书”，却根据每本书的颜色去做行为判断的。

        if (type == 1) {
            System.out.println("Print with the green color.");
        } else if (type == 2) {
            System.out.println("Print with the blue color.");
        } else {
            System.out.println("Print with the white color.");
        }

这个结构，扩展性不好，也不好维护，也不好阅读。

问当事人，当事人说，我在父类没法知道是哪个子类啊。但是这个业务是可以抽取的，所以就写成这个样子了。

其实，可以这么写的。

基类：

package testjava.right;

/**
 * 书的基类
 */
public abstract class BaseBook {
    
    public abstract String getColor();
    
    /**
     * 打印页面
     */
    public void printPage() {
        System.out.println("Print with the " + getColor() + " color.");
    }
}

子类1：

package testjava.right;

/**
 * 蓝色的书
 */
public class BlueBook extends BaseBook {
    
    /**
     * 得到颜色
     * @return 颜色 
     */
    public String getColor() {
        return "Blue";
    }
}

子类2：

package testjava.right;

/**
 * 绿色的书
 */
public class GreenBook extends BaseBook {

    /**
     * 得到颜色
     * @return 颜色 
     */
    public String getColor() {
        return "Green";
    }
}

执行类：

package testjava.right;

/**
 * 执行类
 */
public class Testjava {

    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        BlueBook bb = new BlueBook();
        bb.printPage();
        
        GreenBook gb = new GreenBook();
        gb.printPage();
    }
}

把属于子类的行为，定义成抽象函数，由子类去实现。OK