Отличнейший баг

[metaclass]

Откопал только что в проге сабж. Совершенно безумный.

Есть у меня в проге отображение прогресса для длинных операций. Чтобы его постоянно не обновлять, я вывожу изменения только через каждые 0.3 секунды. А проверяю это дело, использую функцию GetTickCount, сохраняя ее предыдущее значение и сравнивая с текущим. Если разница >300 то прогресс обновляется.

Оказалось, что комп на работе не выключался столько, что GetTickCount стал меньше нуля (использовался integer). И сей алгоритм сломался, т.е. прогресс перестал показываться вообще. Дома показывается, тут нет.

Comments

[eu3eu.livejournal.com]

А я пользую такой способ.
Результат DWORD-DWORD будет DWORD. Поэтому вычисление:

DWORD tstart = GetTickCount();
...
if ( ( GetTickCount() - tstart ) >= 300 ) ...

будет всегда нормально работать, даже, когда GetTickCount() перейдёт через ноль.

[metaclass.livejournal.com]

Хехе, у меня там было более хитро. tstart назначался не всегда, в чем собственно и баг. Когда он был 0 и GetTickCount>0 - все работало. А когда GetTickCount cтал меньше нуля, все сломалось :)

[eu3eu.livejournal.com]

Лично я стараюсь не пользовать типы со знаком там, где в этом нет надобности - в индексах, в счётчиках на инкремент.

[eu3eu.livejournal.com]

А GetTickCount возвращает DWORD (unsigned long), зачем было делать каст в int?

[metaclass.livejournal.com]

а, осталось от каких-то дико стародавних пор