Probleme avec un scanf

Version imprimable

29/03/2007, 09h35
johnson95

Probleme avec un scanf
Bonjour,

J'ai un petit probleme que je ne sais pas résoudre :
Code:

1 2 3 4 5 6 7 8 int test=0,i; while (test == 0){ printf("Lecture : 1 \n Ecriture : 2 \n"); scanf("%d",&i); if ((i == 1) || (i == 2)) test = 1 ; }
tout se passe bien si j'appuie sur 1 ou 2 par contre si j'appuie sur une autre touche il y a un bug, en effet, par exemple si jappuie "J", j'ai mon printf qui s'affiche deux fois !

Mon probleme vient du fait que la lors du scanf on appuie sur "J" puis entrée
Alors le prochain printf prend la valeur "entrée" et passe toute seule!

Comment puis résoudre ce probleme ?
Je pense qu'il faut vider le buffer clavier mais je ne sais pas comment faire sans utiliser fflush.
Merci
29/03/2007, 09h46
stephl
Citation:

Envoyé par johnson95

Comment puis résoudre ce probleme ?
Je pense qu'il faut vider le buffer clavier mais ...

Tout juste. Ajoutez ceci après scanf():
Code:

1 2 while ((c=getchar())!=EOF && c!='\n'); /*c doit être de type int*/
29/03/2007, 12h56
Emmanuel Delahaye

Citation:

Envoyé par johnson95

Mon probleme vient du fait que la lors du scanf on appuie sur "J" puis entrée
Alors le prochain printf prend la valeur "entrée" et passe toute seule!

Comment puis résoudre ce probleme ?
Je pense qu'il faut vider le buffer clavier mais je ne sais pas comment faire sans utiliser fflush.

Bah, je nai pas tout ces problèmes, car je n'utilise pas scanf(). Jamais.

http://emmanuel-delahaye.developpez.com/inputs.htm

Ensuite, pour les conversions, il y a sscanf(), strto{[u]l|d}(),

Regarde essaye, mais je suis pas 100% sur regarde.

Code:

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
#include <stdio.h>
#include <stdlib.h>
 
int main()
{
long test = 0;
long i = 0;
 
 
 
 
 
 
while (test == 0)
{
printf("Lecture : 1 \nEcriture : 2 \n");
scanf("%ld", &i);
if (i == 1 || i == 2)
 
test = 1 ;
 
printf("\nmaintenant test vaut %ld", test);
}
 
	return 0;
}

J'éspère que c'est ça si sa t'aide.

Citation:

Envoyé par mitherkiller

Regarde essaye, mais je suis pas 100% sur regarde.

Code:

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
#include <stdio.h>
#include <stdlib.h>
 
int main()
{
long test = 0;
long i = 0;
 
 
 
 
 
 
while (test == 0)
{
printf("Lecture : 1 \nEcriture : 2 \n");
scanf("%ld", &i);
if (i == 1 || i == 2)
 
test = 1 ;
 
printf("\nmaintenant test vaut %ld", test);
}
 
	return 0;
}

J'éspère que c'est ça si sa t'aide.

Si déjà tu proposes un code, essaye au moins de le rendre un peu présentable, c'est mieux pour la lecture. Moi ce que je propose et comme d'autre te proposerons, n'utilise pas scanf mais fgets, c'est mieux !

Si tu veux faire un menu, voici une solution avec fgets:

Code:

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
 
/* Buffer de 5 char, a changer pour un plus grand au cas ou ! */
#define BUFFER_SIZE 5
 
 
static int choice_menu (void)
{
   char buf [BUFFER_SIZE] = { 0 };
   int val = 0;
 
 
   printf ("Lecture : 1\nEcriture : 2\n");
   fflush (stdout);
 
 
   /* On recupere la saisie utilisateur : */
   if ((fgets (buf, BUFFER_SIZE, stdin)) != NULL)
   {
      /* Suppression du \n s'il y'a. */
      {
         char * p = NULL;
 
         if (buf != NULL)
         {
            p = strchr (buf, '\n');
 
            if (p != NULL)
            {
               *p = 0;
            }
         }
      }
 
      val = strtol (buf, NULL, 10);
   }
 
 
   return val;
}
 
 
int main (void)
{
   int val = 0;
 
 
   do
   {
      val = choice_menu ();
   }
   while (val != 1 && val != 2);
 
 
   return EXIT_SUCCESS;
}

Citation:
Envoyé par Franck.H
Si tu veux faire un menu, voici une solution avec fgets:
Code:

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 36 37 38 #include <stdlib.h> #include <stdio.h> #include <string.h> /* Buffer de 5 char, a changer pour un plus grand au cas ou ! */ #define BUFFER_SIZE 5 static int choice_menu (void) { char buf [BUFFER_SIZE] = { 0 }; int val = 0; printf ("Lecture : 1\nEcriture : 2\n"); fflush (stdout); /* On recupere la saisie utilisateur : */ if ((fgets (buf, BUFFER_SIZE, stdin)) != NULL) { /* Suppression du \n s'il y'a. */ { char * p = NULL; if (buf != NULL) { p = strchr (buf, '\n'); if (p != NULL) { *p = 0; } } } val = strtol (buf, NULL, 10); } return val; }
Ici, il n'y a pas d'intérêt à supprimer le '\n', surtout que tu n'en profites pas pour vider le flux en cas de saturation (4 caractères, c'est vite fait), ni pour tester le 0 en fin de chaine après la conversion par strtol(). C'est un peu bancal tout ça...

Il vaut se concentrer quelques heures à faire des fonctions de lecture de 1 caractère (pratique pour le menus) ou d'une valeur entière, une bonne fois pour toutes plutôt que de de compliquer inutilement.

Le fflush(stdout) est inutile à mois que tu ais oublié un 'prompt' ...