Какой самый безопасный способ хранить объекты классов, производных от общего интерфейса, в общем контейнере?

Как вы уже догадались, вам нужно хранить объекты как указатели.
Я предпочитаю использовать контейнеры указателей повышения (а не обычные контейнеры интеллектуальных указателей).

Причина этого в том, что контейнер boost ptr обращается к объектам, как если бы они были объектами (возвращая ссылки), а не указатели. Это упрощает использование стандартных функторов и алгоритмов в контейнерах.

Недостатком интеллектуальных указателей является то, что вы разделяете право собственности.
Это не то, что вам действительно нужно. Вы хотите, чтобы право собственности было в одном месте (в данном случае в контейнере).

boost::ptr_vector<IActor> ActorList; 
ActorList.push_back(new Enemy()); 
ActorList.push_back(new Civilian());

а также

std::for_each(ActorList.begin(), 
              ActorList.end(),
              std::mem_fun_ref(&IActor::updateDraw));

Чтобы решить проблему, которую вы упомянули, хотя вы идете в правильном направлении, но вы делаете это в неправильном направлении. Это то, что вам нужно сделать

• Определите базовый класс (который вы уже делаете) с виртуальными функциями, которые в вашем случае будут переопределены производными классами Enemy и Civilian.
• You need to choose a proper container with will store your object. You have taken a std::vector<IActor> which is not a good choice because
  • Firstly when you are adding objects to the vector it is leading to object slicing. This means that only the IActor part of Enemy or Civilian is being stored instead of the whole object.
  • Во-вторых, вам нужно вызывать функции в зависимости от типа объекта (virtual functions), что может произойти только в том случае, если вы используете указатели.

Обе приведенные выше причины указывают на то, что вам нужно использовать контейнер, который может содержать указатели, например std::vector<IActor*> . Но лучшим выбором было бы использовать container of smart pointers, который избавит вас от головной боли с управлением памятью. Вы можете использовать любой из интеллектуальных указателей в зависимости от ваших потребностей (но не auto_ptr)

Вот как будет выглядеть ваш код

// somewhere during init
std::vector<some_smart_ptr<IActor> > ActorList;
ActorList.push_back(some_smart_ptr(new Enemy()));
ActorList.push_back(some_smart_ptr(new Civilian()));

а также

// main loop
while(!done) 
{
    BOOST_FOREACH(some_smart_ptr<IActor> CurrentActor, ActorList) 
    {
        CurrentActor->Update();
        CurrentActor->Draw();
     }
}

Что очень похоже на ваш исходный код, за исключением части интеллектуальных указателей.

Моя мгновенная реакция заключается в том, что вы должны хранить интеллектуальные указатели в контейнере и убедиться, что базовый класс определяет достаточно (чистых) виртуальных методов, которые вам никогда не понадобятся dynamic_cast обратно в производный класс.

Если вы хотите, чтобы контейнер эксклюзивно владел элементами в нем, используйте контейнер указателя Boost: они предназначены для этой работы. В противном случае используйте контейнер shared_ptr<IActor> (и, конечно, используйте их правильно, а это означает, что все, кому нужно разделить владение, используют shared_ptr).

В обоих случаях убедитесь, что деструктор IActor является виртуальным.

void* требует от вас ручного управления памятью, так что это исключено. Boost.Any является излишним, когда типы связаны наследованием - стандартный полиморфизм делает свою работу.

Нужен ли вам dynamic_cast или нет, является ортогональной проблемой - если пользователям контейнера нужен только интерфейс IActor, и вы либо (а) делаете все функции интерфейса виртуальными, либо (б) используете идиому невиртуального интерфейса , тогда вам не нужен dynamic_cast. Если пользователи контейнера знают, что некоторые из объектов IActor являются «действительно» гражданскими, и хотят использовать вещи, которые есть в интерфейсе Civilian, но не в IActor, тогда вам понадобятся приведения (или редизайн).