Java - 多个装饰器不起作用

Question

我目前正在尝试装饰器。我创建了一个 Tank 类和两个装饰器:DoubleGunTank(射击更强大)和 FasterTank(开得更快)。它们在这里:

public class Tank {
    public int shoot() {
        return 100;
    }

    public int drive() {
        return 10;
    }
}
public class FasterTank extends Tank {

    protected Tank fTank;

    public FasterTank(Tank tank) {
        fTank = tank;
    }

    public int drive() {
        return fTank.drive() * 2;
    }
}

public class DoubleGunTank extends Tank {

    protected Tank fTank;

    public DoubleGunTank(Tank tank) {
        fTank = tank;
    }
    public int shoot() {
        return fTank.shoot() * 2;
    }   
}

我想做的是用双炮和超高速装饰一辆坦克。所以我这样做:

Tank czolg = new Tank();
czolg = new FasterTank(czolg);
czolg = new DoubleGunTank(czolg);
System.out.println("Shoot: "+czolg.shoot());
System.out.println("Drive: "+czolg.drive());

但结果是:

Shoot: 200
Drive: 10

似乎只有一个装饰器激活 DoubleGunTank 类中的两种方法。所以我的问题是:如何让坦克射击威力更大，同时行驶得更快？

Answer 1

所有装饰器都需要重写所有装饰对象的方法:

class FasterTank extends Tank {

    protected Tank fTank;

    public FasterTank(Tank tank) {
        fTank = tank;
    }

    public int drive() {
        return fTank.drive() * 2;
    }

    //crucial!  
    public int shoot() {
        return fTank.shoot();
    }
}

class DoubleGunTank extends Tank {

    protected Tank fTank;

    public DoubleGunTank(Tank tank) {
        fTank = tank;
    }
    public int shoot() {
        return fTank.shoot() * 2;
    }

    //crucial!  
    public int drive() {
        return fTank.drive();
    }

}

原因如下:当您有:

Tank czolg = new DoubleGunTank(new FasterTank(new Tank()));

并且您调用czolg.drive()，它实际上调用了DoubleGunTank类的方法 - 该方法是从Tank继承的，没有任何更改。因此，您不是使用目标 fTank 的修饰方法，而是调用 DoubleGunTank 的未修改方法。

请注意，您可以通过使用 Tank 接口(interface)来避免此类问题 - 这将迫使您始终装饰所有方法。此外，如果您的目标 Tank 类具有某些状态或在构造函数中执行某些操作，则每个装饰器(继承自它)将复制此状态，并将在构造函数中调用相同的代码。

更新(OP本人建议):

或者，您可以使用抽象 TankDecorator 类，如下所示:

abstract class TankDecorator extends Tank {
    protected final Tank fTank;

    protected TankDecorator(Tank fTank) {
        this.fTank = fTank;
    }

    @Override
    public int shoot() {
        return fTank.shoot();
    }

    @Override
    public int drive() {
        return fTank.drive();
    }
}

class FasterTank extends TankDecorator {

    public FasterTank(Tank tank) {
        super(tank);
    }

    public int drive() {
        return fTank.drive() * 2;
    }

}

class DoubleGunTank extends TankDecorator {

    public DoubleGunTank(Tank tank) {
        super(tank);
    }
    public int shoot() {
        return fTank.shoot() * 2;
    }

}

<小时/>

我在使用cglib时遇到了这个问题spring中的代理- 还通过继承我的类来利用装饰器模式。基类构造函数被调用两次。请参阅:CGLIB proxy method calls constructor twice?和 Spring AOP creates extra bean .