Discussion :

[johnson95]

Bonjour,

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

Code :

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

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

[stephl]

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

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

Code :

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

1
2
while ((c=getchar())!=EOF && c!='\n');
/*c doit être de type int*/

[Emmanuel Delahaye]

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}(),

[mitherkiller]

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

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
#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.

[Franck.H]

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

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
#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 :

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
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;
}

[Emmanuel Delahaye]

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

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
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' ...