Discussion :

[dr4g0n]

Bonjour a tous.
Voila j'ai un petit souci
je demande un numéro a l'utilisateur compris entre 1 et 1000
je test la valeur entrée et je lui répond
Seul problème lorsque l'on rentre une lettre au lieu d'un chiffre mon programme boucle et plante ... il ne redemande plus la valeur et boucle
une idée pour eviter cela ?
Pour complément je suis sous un cygwin et je compile avec gcc.
ci dessous mon 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
36
37
 
 
 
#include <stdio.h> 
#include <signal.h> 
int i; 
int nbuser; 
 
verifentree(nbuser,petit,grand){ 
 
if ( nbuser < 1 || nbuser > 1000){ 
printf(" votre nombre n'est pas dans la fourcette \n" ); 
//comptcoup(); 
i = 0; 
return i, nbuser; 
} 
 
else { 
printf("votre nb est bien dans la fourchette attendu \n il va etre envoyé au serveur \n" ); 
//comptcoup(); 
i = 1; 
return i; 
} 
} 
 
main() { 
//int nbuser; 
do{ 
printf(" PERE >> Entrez votre nombre entre 1 et 1000 \n" ); 
scanf("%d", &nbuser); 
printf(" vous avez saisie la valeur %d \n", nbuser); 
verifentree(nbuser); 
//printf(" coup = %i",coup); 
} 
while (i != 1); 
printf("PERE >> ok \n" ); 
}

[diogene]

La lettre tapée reste dans le buffer du clavier.

Solution : FAQ ici

[Trap D]

Il faut tester le retour de scanf :

Code :

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

1
2
3
4
5
		if (scanf("%d", &nbuser)!= 1)
		{
			puts("Erreur de saisie");
			purge();
		}

avec comme code de purge ce qui est écrit dans la FAQ :

Code :

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

1
2
3
4
5
void purge(void)
{
	int c;
	while ((c = getchar()) != '\n' && c != EOF); /* mange le tampon */
}

!= 1 puisqu'une seule lecture est demandée.

D'autre part je voudrais bien savoir comment tu compiles, ton code est bourré d'erreurs !
Ton code (un peu) corrigé

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
#include <stdio.h> 
#include <signal.h> 
 
void purge(void)
{
	int c;
	while ((c = getchar()) != '\n' && c != EOF); /* mange le tampon */
}
int verifentree(int nbuser)
{ 
	int i;
	if ( nbuser < 1 || nbuser > 1000)
	{ 
		printf(" votre nombre n'est pas dans la fourcette \n" ); 
		//comptcoup(); 
		i = 0; 
	} 
	else 
	{ 
		printf("votre nb est bien dans la fourchette attendu \n il va etre envoyé au serveur \n" ); 
		//comptcoup(); 
		i = 1; 
	} 
	return i; 
} 
 
int main(void) 
{ 
	int nbuser; // c'était très bien ici, pas de variable globale
	do
	{ 
		printf(" PERE >> Entrez votre nombre entre 1 et 1000 \n" ); 
		if (scanf("%d", &nbuser)!= 1)
		{
			puts("Erreur de saisie");
			purge();
		}
		else
		{
			printf(" vous avez saisie la valeur %d \n", nbuser); 
			verifentree(nbuser); 
		}
		//printf(" coup = %i",coup); 
	} 
	while (i != 1); 
	printf("PERE >> ok \n" ); 
	return 0;
}

[Emmanuel Delahaye]

Bonjour a tous.
Voila j'ai un petit souci
je demande un numéro a l'utilisateur compris entre 1 et 1000
je test la valeur entrée et je lui répond
Seul problème lorsque l'on rentre une lettre au lieu d'un chiffre mon programme boucle et plante ... il ne redemande plus la valeur et boucle
une idée pour eviter cela ?
Pour complément je suis sous un cygwin et je compile avec gcc.

Alors tu devrais mieux configurer ton compilateur, car ton code est largement améliorable :

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:6: warning: return type defaults to `int'
main.c:6: warning: function declaration isn't a prototype
main.c: In function `verifentree':
main.c:6: warning: type of "nbuser" defaults to "int"
main.c:6: warning: type of "petit" defaults to "int"
main.c:6: warning: type of "grand" defaults to "int"
main.c:12: warning: left-hand operand of comma expression has no effect
main.c: At top level:
main.c:6: warning: unused parameter 'petit'
main.c:6: warning: unused parameter 'grand'
main.c:23: warning: return type defaults to `int'
main.c:23: warning: function declaration isn't a prototype
main.c: In function `main':
main.c:34: warning: control reaches end of non-void function
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 1 seconds)
0 errors, 11 warnings

Sinon, le problème soulevé est dû à une mauvaise utilisation de scanf(). Je recommande l'usage exclusif de fgets() suivit de fclean() (fonction 'communautaire' dont le code a été publié ici à de maintes reprises) et de la fonction de conversion adéquate (strtol() etc.).

Explications détaillées :

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers

Une fois les erreurs de base corrigées :

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
 
#include <stdio.h>
#include <signal.h>
int i;
int nbuser;
 
int verifentree (int nbuser, int petit, int grand)
{
 
   if (nbuser < 1 || nbuser > 1000)
   {
      printf (" votre nombre n'est pas dans la fourchette \n");
/* comptcoup(); */
      i = 0;
      return i, nbuser;
   }
 
   else
   {
      printf
         ("votre nb est bien dans la fourchette attendu\n"
         "il va etre envoyé au serveur \n");
/* comptcoup(); */
      i = 1;
      return i;
   }
}
 
int main (void)
{
/* int nbuser; */
   do
   {
      printf (" PERE >> Entrez votre nombre entre 1 et 1000 \n");
      scanf ("%d", &nbuser);
      printf (" vous avez saisie la valeur %d \n", nbuser);
      verifentree (nbuser);
/* printf(" coup = %i",coup); */
   }
   while (i != 1);
   printf ("PERE >> ok \n");
 
   return 0;
}

il reste encore ça, ce qui est encore bien confus...

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `verifentree':
main.c:14: warning: left-hand operand of comma expression has no effect
main.c: At top level:
main.c:6: warning: unused parameter 'petit'
main.c:6: warning: unused parameter 'grand'
main.c: In function `main':
main.c:36: error: too few arguments to function `verifentree'
Process terminated with status 1 (0 minutes, 0 seconds)
1 errors, 3 warnings

Sauf cas exceptionnels parfaitement justifiés et documentés, il ne faut pas utiliser les variables globales; Ca introduit de la confusion.

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
 
#include <stdio.h>
#include <signal.h>
 
int verifentree (int nbuser, int petit, int grand)
{
   int ok;
 
   if (nbuser < petit || nbuser > grand)
   {
      printf (" votre nombre n'est pas dans la fourchette \n");
      ok = 0;
   }
 
   else
   {
      printf
         ("votre nb est bien dans la fourchette attendu\n"
          "il va etre envoyé au serveur \n");
      ok = 1;
   }
   return ok;
}
 
int main (void)
{
   int nbuser;
   int ok;
   int min = 0;
   int max = 1000;
 
   do
   {
      printf (" PERE >> Entrez votre nombre entre %d et %d \n", min, max);
      scanf ("%d", &nbuser);
      printf (" vous avez saisie la valeur %d \n", nbuser);
      ok = verifentree (nbuser, 1, 1000);
   }
   while (!ok);
   printf ("PERE >> ok \n");
 
   return 0;
}

Reste encore à corriger la saisie. Pose des questions si tu ne comprends pas.

[dr4g0n]

merci pour ces réponses.
Je débute le c ( a cause de cours du soir au cnam ) et j'en chie un peu !

[Melem]

C'est un programme qui se rencontre asse souvent la réponse est dans la FAQ. Voici une discussion avec le même problème (et avec pas mal de réponses), ca peut t'aider : http://www.developpez.net/forums/sho...d.php?t=470699