Избегание нескольких почти идентичных объявлений для классов, которые реализуют интерфейс - PullRequest
Новая
       19

Избегание нескольких почти идентичных объявлений для классов, которые реализуют интерфейс

1 голос
/ 01 апреля 2019

У меня есть абстрактный базовый класс с именем 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?

Ответы [ 2 ]

2 голосов
/ 01 апреля 2019

Все зависит от того, что вам нужно сделать в вашем operator().

Если вам не нужен доступ к состоянию Command, вы можете передать вызываемую функцию в конструктор.Например, здесь: 

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(); }
};

Вы можете вызвать его, предоставив свободную функцию, лямбду или любой объект std::function.Например: 

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

Но если вам нужен доступ к контексту класса, то я боюсь, что реальное переопределение, как и вы, никак не обойтись (а макрос - ужасная работа для стандартного кода),

1 голос
/ 01 апреля 2019

Ответ похож на тот, на который ответил Кристоф , но не использует наследование.

Возможно, вам не нужно наследование.Вместо этого вы можете использовать выражения 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.

...