Question

我有一个称为Command的抽象基类，它充当可放入队列的命令的接口：

class Command
{
public:

    Command(Dependency1& d1, Dependency2& d2);

    //...Irrelevant code removed for simplicity...

private:

    //Implementations do their work in their override of operator()
    virtual void operator()() = 0;
};

然后我在头文件中包含实现的声明：

class FooCommand : public Command
{
public:

    using Command::Command;

private:

    void operator()() override; 
};

class BarCommand : public Command
{
public:

    using Command::Command;

private:

    void operator()() override; 
};

class BazCommand : public Command
{  
public:

    using Command::Command;

private:

    void operator()() override; 
};

//...And many more...

所以现在我有一长串几乎相同的类声明，只是名称有所不同。除了C样式巨集外，还有什么更可取的方法来清除这个问题？

Answer 1

这完全取决于您在operator()中需要执行的操作。

如果不需要访问Command的状态，则可以选择将Callable传递给构造函数。就像这里：

class CallCommand : public Command { 
    std::function<void()> f;
public:
    CallCommand(Dependency1& d1, Dependency2& d2, std::function<void()> f) : Command(d1,d2), f(f) {
    }
private:
    void operator()() override { f(); }
};

然后可以通过提供自由函数，lambda或任何std::function对象来调用它。例如：

CallCommand c(d1,d2, [](){ cout<<"Hello world"<<endl;});

但是，如果您需要访问类上下文，那么我深感无法绕开像您所做的那样的真正覆盖（宏对于样板代码来说是很糟糕的工作）。

Answer 2

答案类似于Cristophe所回答的，但不使用继承。

您可能不需要继承。相反，您可以使用std::function或lambda表达式。参见，例如：

class Command
{
private:
  std::function<void()> f;
public:
  Command(Dependency1& d1, Dependency2& d2, std::function<void()> f): d1(d1), d2(d2), f(f) {}

  void operator()() {
    f();
  }
};

然后，您可以创建Command的对象，并根据需要在queue或vector中的任何位置使用它们。

Command call1(0, 0, []() {
  std::cout << "call1" << std::endl;
});
Command call2(0, 0, []() {
  std::cout << "call2" << std::endl;
});
std::vector<Command> v;
v.emplace_back(call1);
v.emplace_back(call2);
for (size_t i = 0; i < v.size(); i++) {
  v[i]();
}

在c++11之前，您可以通过为每个函数创建函数指针并将这些指针传递给Command构造函数来避免继承。

为实现接口的类避免多个几乎相同的声明

2 个答案: