使用带参数的命令模式

Question

我有一个这样的可重装武器类:

public class ReloadableWeapon {

    private int numberofbullets;
    
    public ReloadableWeapon(int numberofbullets){
        this.numberofbullets = numberofbullets;
    }
    
    public void attack(){
        numberofbullets--;
    }
    
    public void reload(int reloadBullets){
        this.numberofbullets += reloadBullets;
    }
}

具有以下接口：

public interface Command {
    void execute();
}

并像这样使用它：

public class ReloadWeaponCommand implements Command {

    private int reloadBullets;
    private ReloadableWeapon weapon;
    
    // Is is okay to specify the number of bullets?
    public ReloadWeaponCommand(ReloadableWeapon weapon, int bullets){
        this.weapon = weapon;
        this.reloadBullets = bullets;
    }
    
    @Override
    public void execute() {
        weapon.reload(reloadBullets);
    }
}

客户：

ReloadableWeapon chargeGun = new ReloadableWeapon(10);
Command reload = new ReloadWeaponCommand(chargeGun,10);
ReloadWeaponController controlReload = new  ReloadWeaponController(reload);
controlReload.executeCommand();

我想知道，对于命令模式，对于我看到的示例，除了命令正在操作的对象之外，没有其他参数。

此示例更改执行方法以允许使用参数。

另一个示例，更接近我在这里拥有的，在构造函数中使用参数。

在命令模式中包含参数是不是不好的做法/代码气味，在这种情况下是带有项目符号数的构造函数？

Answer 1

假设一个案例，你有95个子弹，你用10个子弹做了9个命令，用5个子弹做了1个命令。你已经向Invoker提交了这些命令，现在调用者不必担心还剩多少子弹。他将执行命令。另一方面，如果调用者必须在运行时提供没有子弹，那么可能是提供的子弹数量不可用的情况。

我的观点是，调用者不必担心执行命令所需的任何额外信息。正如wiki中提到的“对象用于封装稍后执行操作或触发事件所需的所有信息”

Answer 2

此模式的目的是允许定义动作，并在以后执行一次或多次。

您提供的代码是这种模式的一个很好的例子:您定义了“reload”动作，该动作为< code>gun装填一定数量的< code >子弹=10弹药。

现在，如果您决定修改此代码以添加子弹作为参数，那么您将完全失去此模式的目的，因为您每次都必须定义弹药量。

恕我直言，您可以保持代码原样。您必须定义多个具有不同项目符号值的 ReloadWeaponCommand 实例。然后，您可能必须使用另一种模式（例如策略）在命令之间切换。

Answer 3

我不认为在执行中添加参数将是糟糕的设计或违反命令模式。

这完全取决于您希望如何使用命令对象：Singleton或Prototype范围。

如果使用Prototype范围，则可以在构造函数中传递命令参数。每个命令实例都有自己的参数。

如果使用单一实例作用域（共享/重用实例），则可以在执行方法中传递命令参数。在这种情况下，命令的单一实例应该是线程安全的。这个解决方案也是IoC / DI框架的朋友。