C - le programme de menu utilisant while et switch / case renvoie le menu en double avec une sélection invalide

Vous devez conserver ce que votre fonction d'impression renvoie.

      default:
        printf ("Invalid Selection");
        break;
    };
    selection = print_menu();
}; /* end of while */

Le deuxième problème est que votre appel getchar () prendra également le retour après votre sélection. Ajoutez un autre getchar () pour le consommer.

int selection = getchar();
(void) getchar(); /* ignore enter key */
return selection;

En passant, ne mettez pas de code dans les en-têtes, uniquement les déclarations.
Sinon, le code sera compilé dans chaque fichier de code (parmi plusieurs) qui comprend l'en-tête et vous obtiendra plusieurs erreurs de définition. Ce n'est pas apparent si vous n'avez qu'un seul fichier de code avec l'en-tête, mais vous devriez prendre la bonne habitude tôt.

Enfin, vous devez toujours relire, pas seulement dans le cas de 5. p>

      default:
        printf ("Invalid Selection");
        selection = print_menu();
        break;

Ie faites-le dans la boucle mais en dehors des instructions case, car la branche par défaut n'est prise que si aucune des autres n'a été exécutée.

Essayer

  while(1)
  {

      int selection = print_menu();
      switch (selection)
      ....
  }

getchar () a pour effet secondaire de supprimer le caractère suivant du tampon d'entrée.

Puisque le scanf () est invité à lire un et un seul caractère (% c) , cela a pour effet d'ignorer tout le reste sur cette ligne d'entrée. p>

Effacez également la console avant de réimprimer le menu.

#include <stdio.h>
#include <stdlib.h>
#include <stdio.h>

   int print_menu()
   {
    char selection;

    printf("MENU\n");
    printf("1. Total of All Top Scores for the Week\n");
    printf("2. Total of All High Scores for the Week\n");
    printf("3. Total Machine High Scores for the Week\n");
    printf("4. Machine High Score for the Week\n");
    printf("5. EXIT\n");
    printf("Enter Selection:");

    scanf("%c", &selection);

    return selection;
   }

   int main(int argc, char *argv[])
   {
    int selection;

    while(1) {
      selection = print_menu();

      switch (selection)
      {
        case '1':
          printf("\nselected 1\n");
          exit(selection);
          break;
        case '2':
          printf("\nselected 2\n");
          exit(selection);
          break;
        case '3':
          printf("\nselected 3\n");
          exit(selection);
          break;
        case '4':
          printf("\nselected 4\n");
          exit(selection);
          break;
        case '5':
          printf ("\nExit\n");
          exit(0);
          break;
        default:
          system("@cls||clear");
          printf("\nInvalid Selection\n");
          break;
      };
    };
   }