do / while loop

How should this switch statement be correctly rapped into a do while loop? I need the switch statement to loop until the user selects a correct option.

Code:

---

                scanf("%d", &options);
       
        switch (options)
        {
                case 1 :
                {
                        sumIntegers ();
                        break;
                }
                case 2 :
                {
                        sumDoubles ();
                        break;
                }
                case 3 :
                {
                        calcFactorial ();
                        break;
                }
                default :
                {
                        printf("invalid choice. Please choose correctly.\n");
                }
        }

---

deleted

Code:

---

do
        {
        unknownoption=0;
        scanf("%d", &options);
       
        switch (options)
        {
                case 1 :
                {
                        sumIntegers ();
                        break;
                }
                case 2 :
                {
                        sumDoubles ();
                        break;
                }
                case 3 :
                {
                        calcFactorial ();
                        break;
                }
                default :
                {
                        printf("invalid choice. Please choose correctly.\n");
                        unknownoption=1;
                }
            }
            }while( unknownoption );

---

One way I like to do it:

Code:

---

do {
        scanf("%d", &options);
        switch (options)
        {
                case 1 :
                {
                        sumIntegers ();
                        break;
                }
                case 2 :
                {
                        sumDoubles ();
                        break;
                }
                case 3 :
                {
                        calcFactorial ();
                        break;
                }
                default:
                        printf("invalid choice. Please choose correctly.\n");
                        continue;
        }
} while (0);

---

Otherwise known as a glorified goto, but hey works for me.

Quote:

---

Originally Posted by iMalc

One way I like to do it:

Code:

---

do {
        scanf("%d", &options);
        switch (options)
        {
                case 1 :
                {
                        sumIntegers ();
                        break;
                }
                case 2 :
                {
                        sumDoubles ();
                        break;
                }
                case 3 :
                {
                        calcFactorial ();
                        break;
                }
                default:
                        printf("invalid choice. Please choose correctly.\n");
                        continue;
        }
} while (0);

---

Otherwise known as a glorified goto, but hey works for me.

---

I'd prefer to refactor the menu driver into a separate function that validates input and returns the validated input. Then this becomes a simple switch statement without the need of looping. That's more reusable too - if you make a generic menu driver, you can reuse it for other menus, plus it separates the different concerns of the program (collecting and validating input is separate from acting on the input).