Пакка, я видел твою ПМ-ку, но посмотреть как следует всё никак не могу - некогда
Через неделю отпуск, тогда уж точно гляну
(а может, и раньше).
Пока лишь бросилась в глаза пара нюансов:
- форматирование кода. Тело какого-то блока
всегда должно отделяться табуляцией от его заголовка, а у тебя даже на последнем скриншоте оно стоит на той же самой вертикали. Да и большинство программистов
(в том числе и я) всё-таки отдают "открывающей скобке" целую строку, хотя это дело вкуса.
- объявление переменных. На кой чёрт они объявлены внутри
h-файла, когда больше никаких
cpp-шников у тебя нету? Ты ведь явно не библиотеку пишешь. А то я при чтении основного исходника никак не мог понять, что же значит та или иная переменная... Не говоря уже про то, что глобальные переменные сами по себе не есть гут.
- утечки памяти. Твоя программа вылетает через десяток-другой секунд после запуска, всё увеличивая размер выделяемой для себя памяти. Глянул код - и точно, в нём в четырёх местах используется
new, но нету ни одного
delete...