Discussion :

[kharraz]

bonjour les informaticiens;
j'ai cree une enumeration couleur, et je veux demander a l'utilisateur de taper un nombre et lui donner la reponse correspondanre.Apres lui demander s'il veut continuer.
le probleme c'est qu'il me donne pas a taper, et qu'il m'affiche le message du choix
voici le code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
#include<stdio.h>
#include<stdlib.h>
typedef enum
{
    rouge,
    jaune,
    vert
}couleur;
 
 
main()
{
    char again;
    couleur c;
 
 
        do
    {
 
    printf("--MENU--\n0-rouge\n1-jaune\n2-vert\n");
        scanf("%d",&c);
        switch(c)
        {
            case 0 : printf("STOP"); break;
            case 1 : printf("ATTENTION");break;
            case 2 : printf("GO");break;
            default : printf("choix invalide");
        }
 
 
        printf("\ntapez n'importe quelle touche pour continuer ou n pour sortir");
        scanf("%c",&again);
    }while (again != 'n');
    return 0;
}

merci

[diogene]

1- lorsque tu fais le scanf("%d",&c) , il lit la valeur numérique mais laisse dans le buffer d'entrée le reste de la ligne soit au minimum le '\n' qui a terminé la lecture.
Lorsque tu fais le scanf("%c",&again) , il trouve au moins le '\n' dans le buffer, le prend, continue donc sans demander de nouvelle entrée et reboucle tout seul.
Il faut donc vider le buffer après la lecture en "%d". Par exemple :scanf("%d%*[^\n]%*c",&c) .

2- Sur le scanf("%d",&c) , c est de type couleur, un type entier. Mais rien ne dit que ce type entier est un int (ça peut dépendre des termes de l'énumération ou d'un choix à son goût du compilateur). Les valeurs de l'énumération (rouge, jaune, vert) sont des int, mais une variable de type couleur n'est pas forcément un int. En conclusion, il ne faut pas faire de lecture directement dans une variable de type enum.
Par contre, on peut faire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
    int c;
    scanf("%d",&c);
    switch(c)
    {
       case rouge : printf("STOP"); break;
       case jaune : printf("ATTENTION");break;
       case vert  : printf("GO");break;
       default    : printf("choix invalide");
   }

[kharraz]

merci beaucoup, c’était vraiment très instructif.

[gangsoleil]

3/ Lire les entrees clavier avec scanf, c'est mal. Il suffit, par exemple, que l'utilisateur rentre une lettre, et hop, plus de programme.
http://c.developpez.com/cours/