Какой это будет шаблон проектирования и подойдет ли он для выбора во время выполнения? ⇐ C++

[Anonymous]

У нас есть очень распространенное вычисление Compute, которое можно выполнить несколькими способами. Они наследуются от некоторого интерфейса IFoo. Мы назовем их FooRed и FooBlue.

Код: Выделить всё

class FooRed : public IFoo
{
public:
FooRed() = default;
virtual double Compute() override final; //Inherited from IFoo.
}

Shared_ptr распределяется между многими объектами, поэтому, занимаясь своими делами, они могут выполнять вычисления одинаковым образом. Исходный файлshared_ptr foo = make_shared() находится выше и передается объектам по мере их создания: SubObject(myFoo).
На более позднем этапе разработки было решено, что для расчета может быть полезно изменить режимы с FooRed на FooBlue во время выполнения. Для этого пишется шаблонный класс

Код: Выделить всё

template
class Toggleable:
{
protected:
std::shared_ptr _instance;
bool _state;
public:
Toggleable();
T* const Get() const {return _instance.get()};
const shared_ptr& GetRef() const {return _instance};
virtual void Toggle(); //sets _instance to the correct type.
}

И соответствующий FooManager

Код: Выделить всё

class FooManager : public Toggleable
{
public:
virtual void Toggle() override; //Inherited from Toggleable
}

Итак, теперь пользовательский интерфейс или что-то еще, имеющее доступ к исходному FooManager, может переключать его, и ссылки на внутреннее состояние передаются объектам по мере необходимости: Подобъект(myFoo->GetRef()). У всех одна и та же ссылка, поэтому все они вычисляют одинаково.
Однако моим первым желанием было сделать так, чтобы FooManager также реализовывал IFoo, что-то вроде шаблона контейнера:

Код: Выделить всё

class FooManager : public Toggleable, public IFoo
{
public:
virtual void Toggle() override; //Inherited from Toggleable
virtual double Compute() override {return Get()->Compute();} //Inherited from IFoo
}

Таким образом, вам не придется менять вызовы конструктора, а реализация переключателя меняет как можно меньше базового кода. Все, кому уже был или был передан IFoo, по-прежнему видят его как таковой, и только части, которые сейчас обрабатывают переключение, должны видеть этот объект как FooManager или Toggleable.
Я также считаю, что это более логично для конструкторов, поскольку, если Foo является переключаемым, а Bar нет, то подобъект, который должен выполнять и то, и другое, создается с помощью SubObject(MyFoo->GetRef(), MyBar) в неуклюжей смеси ссылок на общие указатели и общие указатели. Но мой коллега сказал, что такого рода множественное наследование — плохая практика, и что эта структура запутывает поведение менеджера и настоящего Foo.
Это хорошая практика? Как правильно реализовать такой переключатель во время выполнения, который должен переключаться между несколькими подэлементами?

Подробнее здесь: https://stackoverflow.com/questions/783 ... -selection