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

[blackhard]

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

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


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

[volvo]

лучше сами посмотрите

Где посмотреть-то?

Теоретически - возможна утечка памяти (что-нибудь не освобождаешь, хотя нужно), приводящая к нехватке ресурсов. Чтобы сказать точнее - нужен код...

[blackhard]

Где посмотреть-то?

Теоретически - возможна утечка памяти (что-нибудь не освобождаешь, хотя нужно), приводящая к нехватке ресурсов. Чтобы сказать точнее - нужен код...

Прикрепил файлы....

[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 класса вообще полный бардак, проверь, все ли созданные объекты там удаляются.

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

[blackhard]

Очередная проблема(ну она была и раньше)иконки на кнопках тулбара исчезают после нажатия на них или после изменения размера экрана...Думал это связано с применением двойной буферизации но без нее все также......Делал простые примерчики где токо тулбар был все работало нориально(те использовал туже функцию void CreateTB(HWND hw,DWORD tbid) что и в этой программе)

Сообщение отредактировано: blackhard - 15.10.2008 20:36

[volvo]

иконки на кнопках тулбара исчезают после нажатия на них или после изменения размера экрана

Некорректно созданный toolbar дает тебе глюки... Вот такое работает в твоем же проекте под GCC (VisualStudio нет в наличии):

HWND CreateMyToolbar(HWND hWndParent)
{
    const int num_buttons = 4;
    const int imgListID = 0;
    const DWORD buttonStyles = BTNS_BUTTON | TBSTYLE_AUTOSIZE | TBSTYLE_BUTTON | TBSTYLE_TRANSPARENT;

    HWND hWndToolbar =
        CreateWindowEx(0, TOOLBARCLASSNAME, NULL,
            WS_CHILD | TBSTYLE_WRAPABLE, 0, 0, 0, 0,
            hWndParent, NULL, hInst, 0);
    if(hWndToolbar == NULL) return NULL;

    HIMAGELIST hImageList =
        ImageList_Create(16, 16, ILC_COLOR16 | ILC_MASK,
            num_buttons, 0);
    SendMessage(hWndToolbar, TB_SETIMAGELIST, (WPARAM)imgListID, (LPARAM)hImageList);

    // Ну, тут подставишь свои картинки и подписи
    TBBUTTON tbButtons[num_buttons] = {
        {
            MAKELONG(STD_FILEOPEN, imgListID), IDM_OPEN, TBSTATE_ENABLED,
            buttonStyles, {0}, 0, (INT_PTR)"Open"
        },
        {
            MAKELONG(STD_FILESAVE, imgListID), IDM_SAVE, TBSTATE_ENABLED,
            buttonStyles, {0}, 0, (INT_PTR)"Save"
        },
        {
            MAKELONG(HIST_ADDTOFAVORITES, imgListID), IDM_SAVE, 0,
            buttonStyles, {0}, 0, (INT_PTR)"Save"
        },
        {
            MAKELONG(HIST_ADDTOFAVORITES, imgListID), IDM_SAVE, 0,
            buttonStyles, {0}, 0, (INT_PTR)"Save"
        }
    };

    SendMessage(hWndToolbar, TB_LOADIMAGES, (WPARAM)IDB_STD_SMALL_COLOR, (LPARAM)HINST_COMMCTRL);
    SendMessage(hWndToolbar, TB_BUTTONSTRUCTSIZE, (WPARAM)sizeof(TBBUTTON), 0);
    SendMessage(hWndToolbar, TB_ADDBUTTONS, (WPARAM)num_buttons, (LPARAM)&tbButtons);

    SendMessage(hWndToolbar, TB_AUTOSIZE, 0, 0);
    ShowWindow(hWndToolbar, TRUE);
    return hWndToolbar;
}


// создавать не так:
// CreateTB(hWnd,IDTB_TB);

// а вот так
htb = CreateMyToolbar(hWnd);

Ты поправил свой класс? Хендлы удаляются, как положено? А то это ведь тоже может послужить причиной глюков...

[blackhard]

Некорректно созданный toolbar дает тебе глюки...

Мне всетаки удалось отыскать ошибку...функция удаления списка иконок залезла нетуда куда нужно и удаляла их.Спасибо за пример создания тулбара он явно выглядит лучше чем мой))

[blackhard]

Мне вот еще в этом задании нужно сделать сохранение и чертежа в файл и чтение из него.Можно както сохранить описатель кисти или пера?Иначе придется в файле хранить токо координаты чертежа да цвета.....

[volvo]

Можно както сохранить описатель кисти или пера?

Можно, GetObject-ом заполнить структуру LOGBRUSH или LOGPEN, и сохранить ее... Восстанавливать - через CreateBrushIndirect/CreatePenIndirect

[blackhard]

Можно, GetObject-ом заполнить структуру LOGBRUSH или LOGPEN, и сохранить ее... Восстанавливать - через CreateBrushIndirect/CreatePenIndirect

Спасибо за помощь!И последнее что мне осталось сдлать это масштабируемый шрифт и вот тут я опять застрял
Никак не получается сделать
Вот как я попробовал

if(dz<0)//если уменьшаем чертеж
{
    cof=last/di.h;//делим высоту до уменьшения на длинну после
    DraftFont.lfHeight/=cof;востолькоже уменьшаем текст
......................................................................................
}
.......................................................................................

[volvo]

//делим высоту до уменьшения на длинну после

Ты должен делить высоту до уменьшения на высоту после. Тогда получаешь коэффициент уменьшения высоты формы. Скажем, было 500, стало 400, коэффициент = 500/400 = 1.25. А вот теперь делишь высоту шрифта на эти самые 1.25