После нескольких нажатий на кнопки приложение начинает глючить

@Unsigned · Регистрация: 21.12.2014

Студворк — интернет-сервис помощи студентам

Всем привет, возникла такая проблема. Есть приложение с несколькими кнопками, суть в том, что при нажатии они меняют цвет прямоугольника, который тоже выведен на экран. Так вот, после нескольких нажатий на эти кнопки диалог перестает отвечать. Кто-нибудь знает, в чем дело? Если да - помогите пожалуйста)

Основной файл(main.cpp):

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
#include "resources.h"
#include <windows.h>
#include <tchar.h>
 
#if defined(UNICODE) && !defined(_UNICODE)
    #define _UNICODE
#elif defined(_UNICODE) && !defined(UNICODE)
    #define UNICODE
#endif
 
LPARAM CALLBACK WindowProcedure(HWND, UINT, WPARAM, LPARAM);
BOOL   CALLBACK DialogProcedure(HWND, UINT, WPARAM, LPARAM);
VOID   DefaultMessageBox(HWND hWnd) { MessageBox(hWnd, _T("In progress..."), _T(""), MB_OK | MB_ICONINFORMATION); }
 
TCHAR szClassName[] = { _T("WIN_FRAME") };
HINSTANCE hGlobInstance;
HBRUSH color; HWND handleWND;
 
INT APIENTRY _tWinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR szCmdLine, INT nCmdShow)
{
    UNREFERENCED_PARAMETER(hPrevInstance);
    UNREFERENCED_PARAMETER(szCmdLine);
    UNREFERENCED_PARAMETER(nCmdShow);
 
    TCHAR szAppName[] = { _T("Colors") };
    WNDCLASSEX exwc; HACCEL hAccel;
    MSG messages; HWND hwnd;
 
    exwc.cbClsExtra = 0;
    exwc.cbWndExtra = 0;
    exwc.cbSize = sizeof(exwc);
    exwc.hbrBackground = static_cast <HBRUSH> (GetStockObject(WHITE_BRUSH));
    exwc.hCursor = LoadCursor(NULL, IDC_ARROW);
    exwc.hIcon = LoadIcon(hInstance, "OWNICON");
    exwc.hIconSm = exwc.hIcon;
    exwc.hInstance = hInstance;
    exwc.lpfnWndProc = WindowProcedure;
    exwc.lpszClassName = szClassName;
    exwc.lpszMenuName = _T("highMenu");
    exwc.style = CS_HREDRAW | CS_VREDRAW;
 
    if(!RegisterClassEx(&exwc)) return 0x1;
    hGlobInstance = hInstance;
 
    hAccel = LoadAccelerators(hInstance, _T("highMenu"));
 
    hwnd = CreateWindow(szClassName, szAppName, WS_OVERLAPPEDWINDOW,
                        CW_USEDEFAULT, 0, CW_USEDEFAULT, 0,
                        HWND_DESKTOP, NULL, hInstance, NULL);
 
    if(!hwnd) return 0x2;
 
    handleWND = hwnd;
    ShowWindow(hwnd, SW_MAXIMIZE);
    UpdateWindow(hwnd);
 
    while(GetMessage(&messages, NULL, 0, 0))
    {
        if(!TranslateAccelerator(hwnd, hAccel, &messages))
        {
            TranslateMessage(&messages);
            DispatchMessage(&messages);
        }
    }
 
    return messages.wParam;
}
 
BOOL CALLBACK DialogProcedure(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
{
    HBRUSH color_table[7] = {
        CreateSolidBrush(RGB(0x0, 0x0, 0x0)),    //! Black  [0]
        CreateSolidBrush(RGB(0xFF, 0xFF, 0xFF)), //! White  [1]
        CreateSolidBrush(RGB(0x0, 0x0, 0xFF)),   //! Blue   [2]
        CreateSolidBrush(RGB(0xFF, 0x0, 0x0)),   //! Red    [3]
        CreateSolidBrush(RGB(0x0, 0xFF, 0x0)),   //! Green  [4]
        CreateSolidBrush(RGB(0xFF, 0xFF, 0x0)),  //! Yellow [5]
        CreateSolidBrush(RGB(0xFF, 0xA5, 0x0))   //! Orange [6]
    };
 
    switch(uMsg)
    {
        case WM_COMMAND:
            switch(LOWORD(wParam))
            {
                case IDD_WHITE:
                    color = color_table[1];
                    InvalidateRect(handleWND, NULL, 1);
                    return 1;
                case IDD_BLACK:
                    color = color_table[0];
                    InvalidateRect(handleWND, NULL, 1);
                    return 1;
                case IDD_BLUE:
                    color = color_table[2];
                    InvalidateRect(handleWND, NULL, 1);
                    return 1;
                case IDD_GREEN:
                    color = color_table[4];
                    InvalidateRect(handleWND, NULL, 1);
                    return 1;
                case IDD_RED:
                    color = color_table[3];
                    InvalidateRect(handleWND, NULL, 1);
                    return 1;
                case IDD_YELLOW:
                    color = color_table[5];
                    InvalidateRect(handleWND, NULL, 1);
                    return 1;
                case IDD_ORANGE:
                    color = color_table[6];
                    InvalidateRect(handleWND, NULL, 1);
                    return 1;
                case IDD_EXIT:
                    EndDialog(hWnd, 0);
                    break;
            }
            break;
        case WM_CTLCOLORDLG:
            return reinterpret_cast <BOOL> (GetStockObject(WHITE_BRUSH));
    }
 
    return 0;
}
 
LPARAM CALLBACK WindowProcedure(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
{
    PAINTSTRUCT ps;
    RECT rect; HDC hdc;
    HBRUSH hBr;
 
    switch (uMsg)
    {
        case WM_CREATE:
            CreateDialog(hGlobInstance, _T("colorDialog"), hWnd, DialogProcedure);
            color = CreateSolidBrush(RGB(0xFF, 0xFF, 0xFF));
            break;
        case WM_COMMAND:
            switch(LOWORD(wParam))
            {
                case IDM_RANDCOLOR:
                    DefaultMessageBox(hWnd);
                    break;
                case IDM_EXIT:
                    DestroyWindow(hWnd);
                    break;
            }
            break;
        case WM_PAINT:
            hdc = BeginPaint(hWnd, &ps);
 
            rect.left   = 700;
            rect.right  = 900;
            rect.top    = 200;
            rect.bottom = 400;
 
            hBr = color;
            SelectObject(hdc, hBr);
 
            Rectangle(hdc, rect.left, rect.top, rect.right, rect.bottom);
 
            DeleteObject(hBr);
            EndPaint(hWnd, &ps);
            return 0;
        case WM_DESTROY:
            EndDialog(hWnd, 0);
            PostQuitMessage(0);
            break;
        default:
            return DefWindowProc(hWnd, uMsg, wParam, lParam);
    }
 
    return 0;
}

Заголовочный файл(Resource.h):

C++
1
2
3
4
5
6
7
8
9
10
11
12
#define STANDARD_BUTTON WS_CHILD | WS_TABSTOP | WS_VISIBLE
#define IDM_RANDCOLOR 0x000A
#define IDM_EXIT      0x000B
#define IDD_WHITE     0x000C
#define IDD_BLACK     0x000D
#define IDD_EXIT      0x000E
#define IDD_GREEN     0x000F
#define IDD_RED       0x0010
#define IDD_YELLOW    0x0011
#define IDD_BLUE      0x0012
#define IDD_ORANGE    0x0013
#define IDD_OWNCOLOR  0x0014

Файл ресурсов(res.RC)

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
#include "resources.h"
#include <windows.h>
 
OWNICON ICON icon.ICO
 
highMenu MENU
{
    POPUP "New"
    {
        MENUITEM "Random color", IDM_RANDCOLOR
        MENUITEM "Exit", IDM_EXIT
    }
}
 
colorDialog DIALOG 10, 10, 200, 150
STYLE WS_POPUP | WS_VISIBLE | DS_SETFOREGROUND
{
    DEFPUSHBUTTON "White", IDD_WHITE, 20, 20, 40, 20, STANDARD_BUTTON
    PUSHBUTTON "Black", IDD_BLACK, 70, 20, 40, 20, STANDARD_BUTTON
    PUSHBUTTON "Green", IDD_GREEN, 120, 20, 40, 20, STANDARD_BUTTON
    PUSHBUTTON "Red", IDD_RED, 20, 60, 40, 20, STANDARD_BUTTON
    PUSHBUTTON "Yellow", IDD_YELLOW, 70, 60, 40, 20, STANDARD_BUTTON
    PUSHBUTTON "Blue", IDD_BLUE, 120, 60, 40, 20, STANDARD_BUTTON
    PUSHBUTTON "Orange", IDD_ORANGE, 20, 100, 40, 20, STANDARD_BUTTON
    PUSHBUTTON "Exit", IDD_EXIT, 70, 100, 40, 20, STANDARD_BUTTON
}
 
 
highMenu ACCELERATORS
{
    VK_F1, IDM_RANDCOLOR, VIRTKEY
    VK_F2, IDM_EXIT, VIRTKEY
}

@jonson · 14.07.2015, 23:28

Сообщение от Unsigned

Кто-нибудь знает, в чем дело?

первое что бросается в глаза, большая утечка ресурсов.

Сообщение от Unsigned

HBRUSH color_table[7] = {
CreateSolidBrush(RGB(0x0, 0x0, 0x0)), * *//! Black *[0]
CreateSolidBrush(RGB(0xFF, 0xFF, 0xFF)), //! White *[1]
CreateSolidBrush(RGB(0x0, 0x0, 0xFF)), * //! Blue * [2]
CreateSolidBrush(RGB(0xFF, 0x0, 0x0)), * //! Red * *[3]
CreateSolidBrush(RGB(0x0, 0xFF, 0x0)), * //! Green *[4]
CreateSolidBrush(RGB(0xFF, 0xFF, 0x0)), *//! Yellow [5]
CreateSolidBrush(RGB(0xFF, 0xA5, 0x0)) * //! Orange [6]
};

При каждом вхождении в диалоговую процедуру создаются все новые и новые объекты GDI. Так делать нельзя. Инициализация должна быть выполнена только раз, к примеру в WM_INITDIALOG. Предварительно объяви массив как static.

Новые блоги и статьи Все статьи Все блоги /
http://iceja.net/ математические сервисы iceja 20.01.2026 Обновила свой сайт http:/ / iceja. net/ , приделала Fast Fourier Transform экстраполяцию сигналов. Однако предсказывает далеко не каждый сигнал (см ограничения http:/ / iceja. net/ fourier/ docs ). Также. . .	http://iceja.net/ сервер решения полиномов iceja 18.01.2026 Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит переходные токи и напряжения на элементах схемы. . . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .
Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .