Discussion :

[padaka]

Bonsoir, j'ai un petit problème, lorsque l'on saisie un caractère a la place d'un nombre dans ce programme, celui-ci boucle.
comment empêcher ce problème ?
Merci de votre aide

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <stdio.h>
 
int main()
{
    int dec;
 
    do {
        printf("Entrer la valeur du decalage (1<n<25) : ");
        scanf("%d", &dec);
    } while(dec<1 || dec>25);
 
    printf("Suite...\n");
    return 0;
}

[diogene]

Si le scanf() échoue parce que l'entrée ne représente pas un nombre entier valide, alors dec est non modifié et conserve sa valeur antérieure. Dans ton code, dec n'est pas initialisé et contient n'importe quoi, alors le code peut faire dans ce cas n'importe quoi.

A noter qu'on peut vérifier que l'entrée est valide en testant la valeur de retour du scanf()

[Amnael]

Cela est dû je pense au fait que en C au final un caractère fait référence à une valeur de la table ASCII

C'est à dire si tu tapes le caractère A par exemple derrière le compilateur va remplacer A par 65 (sauf erreur de ma part)

Je connais pas la table ascii par cœur; mais il est possible que dans les 25 premiers caractères de a table il y ait celui que tu as entré.

Sous réserves que je ne dises pas de conneries.

Le mieux serait d'afficher un message d'erreur disant qu'il faut un chiffre et non un caractère; mais je connais pas la syntaxe en C.

[Sve@r]

Bonsoir, j'ai un petit problème, lorsque l'on saisie un caractère a la place d'un nombre dans ce programme, celui-ci boucle.
comment empêcher ce problème ?
Merci de votre aide

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <stdio.h>
 
int main()
{
    int dec;
 
    do {
        printf("Entrer la valeur du decalage (1<n<25) : ");
        scanf("%d", &dec);
    } while(dec<1 || dec>25);
 
    printf("Suite...\n");
    return 0;
}

Bonjour
Lorsque tu entres un caractère, celui-ci n'étant pas un nombre n'est donc pas récupéré par scanf() et reste dans le clavier. A l'itération suivante, le clavier contenant déjà une info, scanf() ne te laisse pas la main et tente de nouveau de récupérer l'info. Mais cette info est toujours le même caractère. Et voilà comment ton programme part en torche...

Cela est dû je pense au fait que en C au final un caractère fait référence à une valeur de la table ASCII

C'est à dire si tu tapes le caractère A par exemple derrière le compilateur va remplacer A par 65 (sauf erreur de ma part)

Je connais pas la table ascii par cœur; mais il est possible que dans les 25 premiers caractères de a table il y ait celui que tu as entré.

Sous réserves que je ne dises pas de conneries.

Tu fais des efforts padawan. Tu ne connais pas tout mais ce que tu dis est exact. C'est vrai que A vaut 0x41 ou 65.
Simplement scanf() ne peut pas récupérer ce 'A' car on lui dit "cherche un nombre" et on lui passe un caractère. Même si c'est un nombre en final, la barrière de contrôle intermédiaire posée par scanf() intercepte et refuse ce caractère.

Le mieux serait d'afficher un message d'erreur disant qu'il faut un chiffre et non un caractère; mais je connais pas la syntaxe en C.

En fait, il faut d'abord commencer par libérer immédiatement le clavier dès que quelque chose y est saisi (sans se préoccuper de la nature de la saisie). Pour ça, on peut utiliser fgets() qui attend une chaine.
Une fois que le clavier est redevenu propre, on se préoccupe d'extraire les infos demandées de la chaine saisie. Pour cela il y a sscanf(). Et cette fonction renvoie le nombre exact d'informations correctement récupérées.

Code c :

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
#include <stdio.h>
 
int saisie(char *prompt)
{
	char tampon[256];
	int val;
 
	while (1)
	{
		fputs(prompt, stdout);
		fflush(stdout);
		fgets(tampon, 256, stdin);
		if ((sscanf(tampon, "%d", &val)) == 1)
			break;
		fputs("Saisie incorrecte - Recommencez\n", stdout);
	}
 
	return val;
}
 
int main()
{
    int dec;
 
	do {
		dec=saisie("Entrer la valeur du decalage (1<n<25) : ");
	} while(dec<1 || dec>25);
 
    printf("Suite...\n");
    return 0;
}

On peut parfaitement effectuer la boucle de saisie dans le main() mais je trouve plus propre d'en faire un module réutilisable dans d'autres conditions...