Форум программистов, компьютерный форум, киберфорум
Наши страницы
C++: WinAPI
Войти
Регистрация
Восстановить пароль
 
Unsigned
1 / 1 / 0
Регистрация: 21.12.2014
Сообщений: 67
1

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

14.07.2015, 21:34. Просмотров 312. Ответов 1
Метки нет (Все метки)

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

Основной файл(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
}
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
14.07.2015, 21:34
Ответы с готовыми решениями:

Форма начинает глючить после добавления кнопки вызова подчиненной формы
Вот уже который раз, после попытки добавить кнопку на форму и прохождения...

После 5-10 минут работы комп начинает глючить
После 5-10 минут работы комп начинает глючить,окна заторможено закрываются и...

После включение компьютер начинает свистеть и сильно глючить
Я не давно купил Комп , и в нём была и до сих пор есть такая хигня . При...

После некоторого времени в игре она начинает глючить.
Появилась проблема такая: после некоторого времени проведения в онлайновых...

Ноутбук начинает глючить через 15-20 минут после использования
Смотрите какая ситуация. Запускаю ноутбук, пользуюсь им в течении 15-20 минут....

1
jonson
235 / 208 / 84
Регистрация: 18.03.2010
Сообщений: 750
14.07.2015, 23:28 2
Цитата Сообщение от 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.
0
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
14.07.2015, 23:28

После установки новых драйверов GTS 250 начинает глючить
Проблемы начались после перехода с Windows XP SP3 (x32) на Windows 7 (x32, тоже...

Acer Acpire 4720z после второго запуска начинает "глючить"
Тут такая ситуация.Ноутбук был сильно забит пылью,отключался .Прочистил,заменил...

Сохранение xml после n нажатий кнопки
есть код, который сохраняет xml документ с данными, но необходимо произвести...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
2
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2018, vBulletin Solutions, Inc.
Рейтинг@Mail.ru