Trying to figure out a simple loop....

I am trying to set up an ap that will loop through a messagebox until the user click cancel. As of now I have

Code:

---

#include <windows.h>


int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
        LPSTR lpCmdLine, int nShowCmd)
{

        int check = MessageBox(NULL, "Click an Option!",
                    "MessageBox Example!!", MB_ICONQUESTION |MB_YESNOCANCEL);

        while (check != IDCANCEL)
        {
                if (check == IDYES)
                {
                        MessageBox(NULL, "You Hit YES!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }
                else if (check == IDNO)
                {
                        MessageBox(NULL, "You Hit NO!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }
                else if (check == IDCANCEL)
                {
                        MessageBox(NULL, "You Hit CANCEL - Goodbye!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }
        }

    return 0;
}

---

You might notice that once the user clicks anything the ap goes into a continues loop that you can't exit. So basically in short this is what I am trying to accomplish....

Mesasgebox pops up...
User clicks one of three options (YES, NO, Cancel)

If user clicks Yes then another message box pops up saying you have hit yes. User then hits ok and the loop starts again

If user clicks NO then another message box pops up saying you have hit no. User then hits ok and the loop starts again

If user clicks Cancel then another message box pops sayin you have hit cancel. User the hits ok and the loop exits and ap closes.

In your while loop you don' assign anything to check. So once the loop is entered it will never leave anymore.
maybe this is what you want

Code:

---

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
        LPSTR lpCmdLine, int nShowCmd)
{

        int check = MessageBox(NULL, "Click an Option!",
                    "MessageBox Example!!", MB_ICONQUESTION |MB_YESNOCANCEL);

        while (check != IDCANCEL)
        {
                if (check == IDYES)
                {
                        MessageBox(NULL, "You Hit YES!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }
                else if (check == IDNO)
                {
                        MessageBox(NULL, "You Hit NO!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }
                else if (check == IDCANCEL)
                {
                        MessageBox(NULL, "You Hit CANCEL - Goodbye!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }
              check = MessageBox(NULL, "Click an Option!",
                    "MessageBox Example!!", MB_ICONQUESTION |MB_YESNOCANCEL);
        }

    return 0;
}

---

Kurt

Quote:

---

Originally Posted by ZuK

In your while loop you don' assign anything to check. So once the loop is entered it will never leave anymore.
maybe this is what you want

Code:

---

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
        LPSTR lpCmdLine, int nShowCmd)
{

        int check = MessageBox(NULL, "Click an Option!",
                    "MessageBox Example!!", MB_ICONQUESTION |MB_YESNOCANCEL);

        while (check != IDCANCEL)
        {
                if (check == IDYES)
                {
                        MessageBox(NULL, "You Hit YES!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }
                else if (check == IDNO)
                {
                        MessageBox(NULL, "You Hit NO!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }
                else if (check == IDCANCEL)
                {
                        MessageBox(NULL, "You Hit CANCEL - Goodbye!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }
              check = MessageBox(NULL, "Click an Option!",
                    "MessageBox Example!!", MB_ICONQUESTION |MB_YESNOCANCEL);
        }

    return 0;
}

---

Kurt

---

Yeah I had just figured this out before you posted and it works so thanks......but now I need to figure out how to display a MessageBox after I hit cancel....the while loop is not letting the nested if statment run one lst time when I hit cancel...

Maybe a do/while loop?...I am going to go try that...

does the user get a chance to back out of the cancel and not close the app?

Quote:

---

Originally Posted by Syneris

does the user get a chance to back out of the cancel and not close the app?

---

Nope........

then you could do one last MessageBox for the cancel outside the loop since you know it will be cancel from the loop?

I think you want to use a do while:

Code:

---

do {

    ...

} while (check != IDCANCEL);

---

The only difference in a while and a do/while is that do/while will go through the loop atleast once, as opposed to a while which will not run at all if the criteria for the loop is not met when it goes to start the loop. There is no difference between exiting a do/while and a while. I haven't learned any win32 programming, but it looks like if IDCANCEL is return from the first MessageBox the loop does not run.

try

Code:

---

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
                                  LPSTR lpCmdLine, int nShowCmd)
{

        int check;
        do
        {
                check = MessageBox(NULL, "Click an Option!",
                        "MessageBox Example!!", MB_ICONQUESTION |MB_YESNOCANCEL);
                if (check == IDYES)
                {
                        MessageBox(NULL, "You Hit YES!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }
                else if (check == IDNO)
                {
                        MessageBox(NULL, "You Hit NO!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }
                else if (check == IDCANCEL)
                {
                        MessageBox(NULL, "You Hit CANCEL - Goodbye!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }

        }
        while(check != IDCANCEL);

        return 0;
}

---

Sorry, in my previous post on adding a MessageBox after the loop I did not see that check was being initialized to the result of the first MessageBox.

Quote:

---

Originally Posted by Syneris

try

Code:

---

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
                                  LPSTR lpCmdLine, int nShowCmd)
{

        int check;
        do
        {
                check = MessageBox(NULL, "Click an Option!",
                        "MessageBox Example!!", MB_ICONQUESTION |MB_YESNOCANCEL);
                if (check == IDYES)
                {
                        MessageBox(NULL, "You Hit YES!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }
                else if (check == IDNO)
                {
                        MessageBox(NULL, "You Hit NO!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }
                else if (check == IDCANCEL)
                {
                        MessageBox(NULL, "You Hit CANCEL - Goodbye!",
                                "MessageBox Example!!", MB_ICONASTERISK);
                }

        }
        while(check != IDCANCEL);

        return 0;
}

---

Sorry, in my previous post on adding a MessageBox after the loop I did not see that check was being initialized to the result of the first MessageBox.

---

No problem...this is perfect....I am going to go and try and implement this in my ap.....thanks again everyone....