復(fù)合優(yōu)先于繼承（重寫equals方法引出的建議）

villainhr 發(fā)布于2019-08-19 11:02 / 2315人閱讀

摘要：即不使用繼承方式，而將父類包含在子類中重寫只有當(dāng)坐標(biāo)和顏色都相同才返回當(dāng)然，重寫的時(shí)候一定記得重寫重寫重寫參考文獻(xiàn)第二版第條覆蓋方法時(shí)請遵守通用約定

問題復(fù)現(xiàn)：
首先有一個(gè)Point類，重寫了equals方法：

public class Point{
    private final int x;
    private final int y;
    public Point(x,y){
        this.x=x;
        this.y=y;
    }
    @Override public boolean queals(Object o){
        if(!(o instanceof Point){
            return false;
        }
        Point p = (Point)o;
        return p.x == x && p.y == y;
    }
}

另有一個(gè)擴(kuò)展類，ColorPoint繼承Point類

public class ColorPoint{
    private final Color color;
    public ColorPoint(int x,int y,Color color){
        super(x,y);
        this.color=color;
    }
}

這時(shí)候比較兩個(gè)點(diǎn)的時(shí)候就有個(gè)問題：

Point point = new Point(1, 2);
ColorPoint cPoint = new ColorPoint(1, 2, Color.RED);
System.out.println(point.equals(cPoint)); // true
System.out.println(cPoint.equals(point); // false

可以發(fā)現(xiàn)equals方法違背了對稱性原則，原因是Point的equals方法在接收ColorPoint類型的參數(shù)時(shí)，會(huì)將其當(dāng)做Point進(jìn)行比較，忽略了顏色的判斷，認(rèn)定兩個(gè)類是相等的。
對此我們做出一些修改：

此時(shí)可修改equals方法,加入對顏色的判斷：
if(!(o.instanceOf(Point))
    return false;
//if o is a normal point,ignore color
if(!(o.instanceOf(ColorPoint))
    return o.equals(this);
//if o is a colorPoint .do a full compation
return super.equals(o) && ((ColorPoint)o).equals(this.color);

這段代碼修復(fù)了違背對稱性的錯(cuò)誤，但兩個(gè)以上的點(diǎn)會(huì)有錯(cuò)誤，比如兩個(gè)ColorPoint和一個(gè)Point作比較，如下：

ColorPoint cPointRed = new ColorPoint(1, 2, Color.RED);
Point point = new Point(1, 2);
ColorPoint cPointBlue = new ColorPoint(1, 2, Color.BLUE);
System.out.println(cPointRed.equals(point)); // true
System.out.println(point.equals(cPointBlue)); // true
System.out.println(cPointRed.equals(cPointBlue)); // false

這樣又違背了傳遞性原則。這時(shí)候就有個(gè)建議：復(fù)合優(yōu)先于繼承。
即不使用繼承方式，而將"父類"包含在"子類"中

public class ColorPoint{
    private final Point point;
    private final Color color;
    public ColorPoint(int x,int y,Color color){
        point.x = x;
        point.x = x
        this.color = color;
    }
    public Point getPoint(){
        return this.point;
    }
    //重寫equals
    @Override public boolean equals(Object o){
        if(!(o instanceof ColorPoint){
            return false;
        }
        ColorPoint cp = (ColorPoint)o;
        //只有當(dāng)坐標(biāo)和顏色都相同才返回true;
        return cp.point.equals(this.point) && cp.Color.equals(color);
    }
}

當(dāng)然，重寫equals的時(shí)候一定記得重寫hashCode~重寫hashCode~重寫hashCode~~de~de~de~

參考文獻(xiàn)：《Effective Java》第二版第8條 “覆蓋equals方法時(shí)請遵守通用約定”