Малък фикс в TimerMgr

bonev_st 13 Точки

Малък фикс в TimerMgr

Проблем: Когато таймера е стопиран в предишния цикъл на main loop-a на Engine чрез TimerClient::stopTimer() и изтече в този - ще се генерира onTimerTimeout с Id на таймер който вече е спрян, също така ще намали броя на итерации в цикъл

for(auto& [id, data] : m_TimerMap) {
       data.m_Remaining -= elapsed_ms;
       if (0 > data.m_Remaining) {
           onTimerTimeout(id, data);
       }
}

Вариант 1:

Предлагам малка промяна в реда на TimerMgr::process() метода - removeTimers() да се премети в началото на фунията.

Вариант 2:

В stopTimer и в detachTimerClient да се извиква removeTimers() преди края на методите.

Тагове:

07/11/2021 13:26:47 C++ Applications Development

Виж всички отговори

MartinBG 4803 Точки

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

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

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

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

07/11/2021 20:23:23 07/11/2021 21:25:14

bonev_st 13 Точки

Да, абасолютно си прав!

07/11/2021 20:46:53

j.petrov_90 373 Точки

Супер сте, колеги :)

08/11/2021 10:51:07

Виж всички отговори