Message Boxes executing one after the other

Heya there,

I have been learning C++ for a few weeks now, and I am kind of stuck on creating more than one message box successfully.

I have created a simple menu, and a single button. I have created IDs for both the exit button (file > exit) in the program and the button as well. If I test them on their own with 1 message box at a time...it works fine.

However...this is where the problem starts. It is when I add a second message box to the WM_COMMAND case. What is does is for the exit in the file > exit scenario, carry out the message box for the button and then followed by the one intended for it. It does the same exact procedure for the button as well.

What exactly am I doing wrong? Any help or explanation would be appreciated.

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
#include <windows.h>

/*  Declare Windows procedure  */
LRESULT CALLBACK WindowProcedure (HWND, UINT, WPARAM, LPARAM);

/*  Make the class name into a global variable  */
char szClassName[ ] = "WindowsApp";

int WINAPI WinMain (HINSTANCE hThisInstance,
                    HINSTANCE hPrevInstance,
                    LPSTR lpszArgument,
                    int nFunsterStil)

{
    HWND hwnd;               /* This is the handle for our window */
    MSG messages;            /* Here messages to the application are saved */
    WNDCLASSEX wincl;        /* Data structure for the windowclass */

    /* The Window structure */
    wincl.hInstance = hThisInstance;
    wincl.lpszClassName = szClassName;
    wincl.lpfnWndProc = WindowProcedure;      /* This function is called by windows */
    wincl.style = CS_DBLCLKS;                 /* Catch double-clicks */
    wincl.cbSize = sizeof (WNDCLASSEX);

    /* Use default icon and mouse-pointer */
    wincl.hIcon = LoadIcon (NULL, IDI_APPLICATION);
    wincl.hIconSm = LoadIcon (NULL, IDI_APPLICATION);
    wincl.hCursor = LoadCursor (NULL, IDC_ARROW);
    wincl.lpszMenuName = NULL;                 /* No menu */
    wincl.cbClsExtra = 0;                      /* No extra bytes after the window class */
    wincl.cbWndExtra = 0;                      /* structure or the window instance */
    /* Use Windows's default color as the background of the window */
    wincl.hbrBackground = (HBRUSH) COLOR_BACKGROUND;

    /* Register the window class, and if it fails quit the program */
    if (!RegisterClassEx (&wincl))
        return 0;

    /* The class is registered, let's create the program*/
    hwnd = CreateWindowEx (
           0,                   /* Extended possibilites for variation */
           szClassName,         /* Classname */
           "Mini-Test",       /* Title Text */
           WS_OVERLAPPEDWINDOW, /* default window */
           CW_USEDEFAULT,       /* Windows decides the position */
           CW_USEDEFAULT,       /* where the window ends up on the screen */
           600,                 /* The programs width - default 544 */
           400,                 /* and height in pixels - default 375 */
           HWND_DESKTOP,        /* The window is a child-window to desktop */
           NULL,                /* No menu */
           hThisInstance,       /* Program Instance handler */
           NULL                 /* No Window Creation data */
           );

    /* Make the window visible on the screen */
    ShowWindow (hwnd, nFunsterStil);

    /* Run the message loop. It will run until GetMessage() returns 0 */
    while (GetMessage (&messages, NULL, 0, 0))
    {
        /* Translate virtual-key messages into character messages */
        TranslateMessage(&messages);
        /* Send message to WindowProcedure */
        DispatchMessage(&messages);
    }

    /* The program return-value is 0 - The value that PostQuitMessage() gave */
    return messages.wParam;
}

#define ID_BUTTON 1
#define ID_EXIT 1

/*  This function is called by the Windows function DispatchMessage()  */

LRESULT CALLBACK WindowProcedure (HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
{
    switch (message)                  /* handle the messages */
    {
        case WM_CREATE:{
             /* Button */
             CreateWindow(TEXT("button"), TEXT("Click Me"),
                 WS_VISIBLE | WS_CHILD,
                 10, 40, 80, 20,      /* x y width height */
                 hwnd, (HMENU) ID_BUTTON, NULL, NULL
             );
             
            /* Menu */ 
            HMENU hMenuBar = CreateMenu();
            HMENU hFile = CreateMenu();
            HMENU hOptions = CreateMenu();
            
            AppendMenu(hMenuBar, MF_POPUP, (UINT_PTR)hFile, "File");
            AppendMenu(hMenuBar, MF_POPUP, NULL, "Edit");
            AppendMenu(hMenuBar, MF_POPUP, NULL, "Options");
            
            AppendMenu(hFile, MF_STRING, ID_EXIT, "Exit");
            
            SetMenu(hwnd, hMenuBar);
            
            break;  
        }
        case WM_COMMAND:{
            
            if (LOWORD(wParam) == ID_BUTTON){
                  MessageBox(hwnd, "You clicked me", "Success!", MB_ICONINFORMATION);
            }
            
            if (LOWORD(wParam) == ID_EXIT){
                  MessageBox(hwnd, "You cannot exit yet", "Test", MB_ICONHAND);
            }
            
            break;
        }
        case WM_DESTROY:
            PostQuitMessage (0);       /* send a WM_QUIT to the message queue */
            break;
        default:                      /* for messages that we don't deal with */
            return DefWindowProc (hwnd, message, wParam, lParam);
    }

    return 0;
}
closed account (DSLq5Di1)
Your problem is at line 72/73.
Thanks for pointing that out for me...

But where do I define them then?
closed account (DSLq5Di1)
Where you've defined them isn't the problem, the values assigned to them is.

Identifiers should be unique, ID_BUTTON & ID_EXIT have the same value, hence, whether you click your button, or the exit.. both if statements in the WM_COMMAND block are true.
u defined same IDs for both controls
Topic archived. No new replies allowed.