Menu acceptant un seul caractère en langage C

Question

Je souhaite créer un menu simple dans un programme C acceptant un seul caractère. Le menu sera comme ceci:

1. [S] comment
2. [E] xit

Si l'utilisateur d'entrer '1', 'l' ou 'S' le programme affichera " Bonjour "et demander à nouveau pour l'entrée sinon si l'utilisateur entre" 2 ", E ou" E "le programme se termine. sinon il devrait imprimer "entrée invalide" et demander à nouveau.

Je suis capable de créer le programme mais le problème est que lorsque l'utilisateur entre 12, 13, 14, 15, 16 ..... ainsi de suite en commençant par 1, il affiche Hello et même pour les autres options.

Mon code est:

#include <stdio.h> 

void clearBuffer(); 


int main() { 

    int i = 0; 
    char selection; 

    do 
    { 
     printf("\t1. [S]how\n"); 
     printf("\t2. [E]xit\n"); 
     printf("Enter your selection from the number or character noted above: "); 
     scanf("%s", &selection); 
     clearBuffer(); 

     if (selection == '1' || selection == 's' || selection == 'S') 
      printf("Hello"); 
     else if (selection == '2' || selection == 'E' || selection == 'x') 
      i = 0; 

} while(i != 0); 



} 

void clearBuffer() 
{ 

    while(getchar() != '\n'); 

} 

Answer 1

Vous pouvez utiliser strlen, qui fait partie de la bibliothèque standard C, pour vérifier la longueur de la chaîne retournée par scanf et rejeter les entrées plus d'un caractère:

if (strlen(selection) > 1) 
{ 
    printf("Invalid selection."); 
} 

Alternativement, je pense que vous pourriez utiliser getchar() pour n'accepter qu'un seul caractère de l'utilisateur, ce qui signifie qu'il n'aurait pas besoin d'appuyer sur Entrée.

Answer 2

Si vous allez recevoir un seul caractère envisager de remplacer la fonction pour getchar() fonction scanf():

printf("Enter your selection from the number or character noted above: "); 
selection = getchar(); 

Answer 3

Comme déjà mentionné, vous devez utiliser getchar() si vous voulez un seul caractère. Si vous voulez toujours utiliser scanf() pour une raison quelconque, le format correct est "%c", et non "%s".

Je suggère également que si vous cherchez un seul caractère, le bloc if semble un peu "occupé" (lire, maladroit) ... un commutateur serait un moyen plus propre et plus élégant de le faire (IMHO).

/* something like this ... */ 
switch (selection) { 
    case '1': 
    case 's': 
    case 'S': 
    printf ("Hello\n"); 
    break; 

    case '2': 
    case 'e': 
    case 'E': 
    i = 0; 
    break; 
} 

Autres deux choses ... si vous ne vous souciez pas le cas du caractère en cours de lecture (qui est, 's' et 'S' fera la même chose), vous pouvez convertir en majuscules selection avant if -bloc ou switch -bloquer en utilisant toupper(). En outre, et ceci est juste une suggestion de style, n'utilisez pas i pour votre drapeau de sortie. La pratique générale est d'utiliser des choses comme i et j pour les compteurs ou les index - vous pouvez utiliser quelque chose comme quit_now ou user_done qui permettrait de transmettre plus précisément ce que signifie la variable.