Помогите исправить баг...., WinAPI Опции

[blackhard]

В общем делаю программу чертеж в которой есть зум и все такое.........Так вот в чем проблема....Если пару десятков раз изменить размер окна или использовать зумм все окна(нетолько моей программы)перестают обнавляться(незнаю как подругому объяснить лучше сами посмотрите(после завершения моей программы все вновь начинает обнавляться))..Никак не могу понять с чем это связано???Ставил комментарии на все что только можно и всеравно тоже самое......Помогите найти причину....

P.S Заранее извиняюсь за код, это моя первая серьезная прога на WinAPI)))


Полностью не добавился поэтому пришлось разбить...

[volvo]

Хм...

Ну, как я и говорил, у тебя утечка памяти, причем очень большая. Попробуй запустить TaskManager при работе своей программы, и посмотреть число GDI Objects... А знаешь, почему? Потому, что все, что ты делаешь

// Описание переменных

... // <--- Вот тут, после описания переменных, но перед switch-ем

switch(msg) {

в WndProc выполняется при получении каждого сообщения Windows твоим приложением. Ты представляешь, сколько сообщений приложение получает? Сотни ежесекундно. И у тебя каждый раз создается новый Pen, который нигде не удаляется, и новый BITMAP, а это - утечка. В какой-то момент счетчик GDI-объектов просто зашкаливает, и ты получаешь то, что описано в первом посте... Перенеси создание этого карандаша (если он тебе нужен, конечно), куда-нибудь в другое место, скажем, в WM_CREATE...

Кстати. Совсем не обязательно на все события мыши вешать SetCursor(hcguetar), вместо этого достаточно опять же при создании окна в WM_CREATE один раз прописать установку курсора для твоего окна... Все остальные действия - лишние. Ну, в методе draw класса вообще полный бардак, проверь, все ли созданные объекты там удаляются.

Сообщение отредактировано: volvo - 8.10.2008 19:18

[Гость]

Хм...

Ну, как я и говорил, у тебя утечка памяти, причем очень большая. Попробуй запустить TaskManager при работе своей программы, и посмотреть число GDI Objects... А знаешь, почему? Потому, что все, что ты делаешь

// Описание переменных

... // <--- Вот тут, после описания переменных, но перед switch-ем

switch(msg) {

в WndProc выполняется при получении каждого сообщения Windows твоим приложением. Ты представляешь, сколько сообщений приложение получает? Сотни ежесекундно. И у тебя каждый раз создается новый Pen, который нигде не удаляется, и новый BITMAP, а это - утечка. В какой-то момент счетчик GDI-объектов просто зашкаливает, и ты получаешь то, что описано в первом посте... Перенеси создание этого карандаша (если он тебе нужен, конечно), куда-нибудь в другое место, скажем, в WM_CREATE...

Кстати. Совсем не обязательно на все события мыши вешать SetCursor(hcguetar), вместо этого достаточно опять же при создании окна в WM_CREATE один раз прописать установку курсора для твоего окна... Все остальные действия - лишние. Ну, в методе draw класса вообще полный бардак, проверь, все ли созданные объекты там удаляются.

Спасибо за помощь)Блин глупая ошибка.....Начну разгребать свой бардак)