Привет, bonev_st,

Не съм го пробвал с код, но като го "разиграх на ум" наистина достигнах до посочения проблем от теб.

Вариант 1 ми се струва добро предложение.
Жив и здрав :)

Вариант 2 или не го разчитам правилно или мисля, че би имало проблем с него.
Ако чрез stopTimer директно триеш таймера от map-а с removeTimers() ще попаднеш в проблема, заради който съществува онзи std::set както и функцията removeTimers() на първо време.

Казуса беше, че при стечение на обстоятелствана е съвсем нормално в един дълъг call stack да имаш 2 извикватия на stopTimer с едно и също id.
И двете са били валидни в момента на извикването им.
Резултата е, че единият би изтрил таймера, а за другия би се хвърлило грешка - "такъв таймер няма".

Разбира се, може и да не съм разбрал правилно.

Поздрави 

При вариант 1 може да изпаднем в следната ситуация:

- oneshot таймер изтича по време на цикъла и съответно се добавя в сета за изтриване, но не е изтрит (защото removeTimers() е в началото на TimerManager::process())

- ако в следващия цикъл от Game или някъде другаде стартираме нов таймер със същото ID, този таймер ще бъде създаден успешно (защото isActiveTimerId ще върне false), но ще бъде изтрит при първото извикване на TimerManager::process(), защото id-то му е маркирано за изтриване

Вариант за решаване на този проблем е да извикваме removeTimers() 2 пъти в TimerManager::process() - един път в началото (премахваме спрените таймери) и веднъж в края на метода (премахваме изтеклите таймери).