Discussion :

[sedget]

Bonjour,
j'ai un soucis avec une portion de code et je n'arrive pas a en trouver l'origine.

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
 
int saisie;
int test=1;
 while (test)
{ 
printf("Saisissez un entier entre 1 et 3\n");
scanf("%d",&saisie);
 
 if (saisie<1 || saisie>3)
{
printf("votre saisie n'est pas conforme, recommencez");
test=1;
}
else
{
test=0;
} 
}

cette portion de code fonctionne normalement si la saisie utilisateur est bien un entier, mais si l'utilisateur saisie une lettre le programme boucle sans redonner la main a l'utilisateur pour saisir une nouvelle entrée. C'est comme si le programme s'auto-alimentait ??

j'ai bien essayer en rajoutant fflush(stdin); avant le scanf mais cela ne change rien.

je précise qu'a la compilation gcc -Wall je n'ai aucune alerte.
une ame charitable pourrait elle m'expliquer ce qui ce passe.

[dot-_-net]

salut
je pense qu’un Do while fera la faire

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
int saisie;
int test=1;
 while (test)
{ 
do{
printf("Saisissez un entier entre 1 et 3\n");
scanf("%d",&saisie);
 }
 while (saisie<1 || saisie>3);
 
 
test=0;
 
}

[hegros]

http://c.developpez.com/faq/?page=cl...EN_scanf_fgets

http://c.developpez.com/faq/?page=cl...N_fflush_stdin

Tu as également des tutoriaux sur le site à ce sujet

[Jack_serious]

C'est pour ca que l'utilisation de scanf() est deconseillee a moins d'en connaitre tres bien les mecanismes.

http://c.developpez.com/faq/?page=cl...EN_scanf_fgets

Jettes-y un oeil ainsi que les articles qui sont au dessus

[EDIT]

Grillaid... Snif...

[hegros]

salut
je pense qu’un Do while fera la faire

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
int saisie;
int test=1;
 while (test)
{ 
do{
printf("Saisissez un entier entre 1 et 3\n");
scanf("%d",&saisie);
 }
 while (saisie<1 || saisie>3);
 
 
test=0;
 
}

Cela n'a rien à voir. Tu pourrais mettre un for se serait identique. Le problème est la fonction scanf.

[sedget]

merci de vos réponses, je comprends bien ce que vous m'expliquez mais ca ne fait pas encore parti des cours que je suis pour le moment on a étudié getchar et scanf . Je suis un vrai bleu
Je vais donc je vais me rabattre vers getchar.

je voulais juste essayer de rajouter une gestion des erreurs à mon programme pour l'améliorer ca risque donc d'être plus compliqué que prévu.

[hegros]

Utilise fgets. Je pense que tu ne devrais pas avoir trop de mal à l'utiliser même avec une barbe bleu

[dot-_-net]

getchar c'est beug et deconseillé

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
char saisie;
int test=1;
 while (test)
{ 
do{
printf("Saisissez un entier entre 1 et 3\n");
saisie=getchar();
 }
 while (saisie<'1' || saisie>'3');
 
 
test=0;
 
}

[hegros]

getchar c'est beug et deconseillé

Je pourrais savoir c'est quoi le bug de cette fonction ?

[Melem]

Bonjour.
J'ai un soucis avec une portion de code et je n'arrive pas a en trouver l'origine.

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
int saisie;
int test = 1;
 
while (test)
{ 
    printf("Saisissez un entier entre 1 et 3\n");
    scanf("%d", &saisie);
 
    if (saisie < 1 || saisie > 3)
    {
        printf("votre saisie n'est pas conforme, recommencez");
        test=1; /* Inutile */
    }
    else
    {
        test=0;
    } 
}

Cette portion de code fonctionne normalement si la saisie utilisateur est bien un entier, mais si l'utilisateur saisie une lettre le programme boucle sans redonner la main a l'utilisateur pour saisir une nouvelle entrée. C'est comme si le programme s'auto-alimentait ??

Le format %d dans scanf attend qu'on lui fournisse un entier. Les détails : scanf va d'abord passer tous les caractères blancs (espaces, tabulations, fin de ligne, etc.) s'il y en a ensuite lire les caractères qui peuvent figurer dans l'expression d'un entier, la saisie s'arrête à la première rencontre d'un caractère invalide (uns espace, un point ou une lettre par exemple). Au prochain appel scanf (ou de n'importe quelle autre fonction de saisie), le dernier caractère non lu reste toujours dans le buffer du flux stdin (qui est attaché au clavier). Tant que tu demanderas de lire un nombre, tu n'avanceras pas.

La manière la plus robuste connue à ce jour pour saisir des données venant de l'utilisateur est de saisir une ligne avec fgets puis appeler ensuite fclean. On fait ensuite s'il y en a les conversions nécessaires.

je comprends bien ce que vous m'expliquez mais ca ne fait pas encore parti des cours que je suis pour le moment on a étudié getchar et scanf

fgets est beaucoup moins compliquée que scanf tu ne devras pas avoir de problème à l'utiliser. Sinon, tu peux l'implémenter pour stdin avec getchar :

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
char * get_line(char * buffer, int bufsize)
{
    int i = 0, c;
 
    c = getchar();
    while (i < bufsize - 1 && c != EOF && c != '\n')
    {
        buffer[i++] = (char)c;
        c = getchar();
    }
 
    buffer[i] = '\0';
 
    return buffer;
}

Attention : ici, contrairement à ce qui se passe avec fgets, le caractère de fin de ligne n'est pas lu. Tu peux la modifier pour avoir un comprtement strictement identique à celui de fgets(buffer, bufsize, stdin).

[Pouic]

http://xrenault.developpez.com/tutoriels/c/scanf/#L4.2