Question

我已经广泛使用了命令模式，并且效果很好。但是，通常没有讨论的是其中创建了命令的实例。

以下示例说明了此问题：Document具有设置文本的函数setText()：

class Document {
public:
    void setText(const std::string text) {
        if (commandManager()->isActive()) {
            // called by SetTextCommand
            m_text = text;
        } else {
            // called somewhere in the application
            commandManager()->addAndExecute(new SetTextCommand(this, text));
        }
    }
    std::string text() const { return m_text; }

    CommandManager * commandManager() const { return m_commandManager; }
private:
    std::string m_text;
    CommandManager * m_commandManager;
}

此处，SetTextCommand会执行document->setText(text)，如下所示：

class SetTextCommand : public Command {
public:
    SetTextCommand(Document * doc, const std::string & text)
        : Command(), m_doc(doc), m_oldText(doc->text()), m_text(text)
    {}
    void redo() override {
        m_doc->setText(m_text);
    }
    void undo() override {
        m_doc->setText(m_oldText, false);
    }
}

SetTextCommand由CommandManager处理，如下所示：

CommandManager::addAndExecute(Command * command) {
    m_doc->commandManager()->setActive(true); // THIS IS THE TRICK
    command->redo();
    m_doc->commandManager()->setActive(false);  // THIS IS THE TRICK
    m_stack->push_back(command);
}

这里的诀窍是，在运行redo()时，CommandManager::isActive()设置为 true 。因此，Document::setText()将设置m_text。

显然，所有Document setter函数都必须遵循if (commandManager()->isActive()) { ... } else { ... }范例。这是因为命令本身是在setter函数中创建的。

现在问题：这是实现命令模式的好方法吗？或者是否有更清晰的解决方案来创建命令，同时拥有一个很好的API？

请详细说明你的答案。

Answer 1

我认为必须在任何地方复制if (commandManager()->isActive())非常难看......让setText始终执行SetTextCommand路径可能更好，并创建一个新的{{ 1}} setTextImmediate可以使用的方法。

命令模式：在哪里创建命令项？

1 个答案: