Problème de scanf

Version imprimable

Problème de scanf

Salut tout le monde
Ces derniers jours je travaille sur le programme C sous le shell bash
Mon problème est le suivant :
Lors de l’exécution d’un programme du genre :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 #include <stdio.h> int main() { char num; printf("Donner votre num: "); scanf("%c",&num); printf("Votre num est: %c ",num); printf("Donner votre code: "); scanf("%c",&num); printf("Votre code est: %c ",num); return 0; }
Lorsque je donne la première entrée et je click sur entrée le programme s’exécute entièrement et il ne me demande plus de saisir l’autre valeur
Il affiche une sortie du genre :

Code:

Donner votre num:

(la je donne la valeur f par exemple)

Code:

Votre num est: f Donner votre code: Votre code est:

09/09/2007, 15h19
LLB

Je te conseille d'utiliser un int et le format %d. Après test, chez moi, ça marche.
10/09/2007, 08h22
Laughing Man

je pense que ton problème vient du tampon
avant chaque scanf essaie de mettre un fflush(stdin) pour voir.

Citation:

Envoyé par nick_08

Ces derniers jours je travaille sur le programme C sous le shell bash
Mon problème est le suivant :
Lors de l’exécution d’un programme du genre :
<...>
Lorsque je donne la première entrée et je click sur entrée le programme s’exécute entièrement et il ne me demande plus de saisir l’autre valeur
Il affiche une sortie du genre :

Code:

Donner votre num:

(la je donne la valeur f par exemple)

Code:

Votre num est: f Donner votre code: Votre code est:

Un cas typique de mauvais usage de scanf(). Il y a 2 solutions :

Apprendre à utiliser scanf() correctement
Se passer définitivement de scanf() et utiliser fgets() + les fonctions de conversions qui vont bien (strtol(), sscanf()) etc.)

Une solution avec scanf()
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 #include <stdio.h> int main (void) { char num; printf ("Donner votre num: "); scanf ("%c", &num); if (num != '\n') { int c; while ((c = getchar()) != '\n') { } } printf ("Votre num est: %c\n", num); printf ("Donner votre code: "); scanf ("%c", &num); if (num != '\n') { int c; while ((c = getchar()) != '\n') { } } printf ("Votre code est: %c\n", num); return 0; }
Code:

1 2 3 4 5 6 7 Donner votre num: 1 Votre num est: 1 Donner votre code: a Votre code est: a Press ENTER to continue.
Evidemment, vu la répétitivité du code, on crée une fonction de nettoyage : purge() :
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 #include <stdio.h> static void purge(int val) { if (val != '\n') { int c; while ((c = getchar()) != '\n') { } } } int main (void) { char num; printf ("Donner votre num: "); scanf ("%c", &num); purge(num); printf ("Votre num est: %c\n", num); printf ("Donner votre code: "); scanf ("%c", &num); purge(num); printf ("Votre code est: %c\n", num); return 0; }

10/09/2007, 08h58
Emmanuel Delahaye

Citation:

Envoyé par Laughing Man

je pense que ton problème vient du tampon
avant chaque scanf essaie de mettre un fflush(stdin) pour voir.

Le diagnostic est correct, mais le remède est erroné. fflush() n'est défini que pour les flux sortants.
10/09/2007, 08h59
Emmanuel Delahaye

Citation:

Envoyé par LLB

Je te conseille d'utiliser un int et le format %d. Après test, chez moi, ça marche.

Sauf qu'en cas de saisie erronée, on retombe sur le même problème. De plus, le résultat est différent (il y a conversion texte -> valeur numérique).

Citation:

Envoyé par Laughing Man

je pense que ton problème vient du tampon
avant chaque scanf essaie de mettre un fflush(stdin) pour voir.

Non! On le rappelle presque tous les jours sur ce forum: le comportement de fflush() n'est défini QUE pour un flux sortant. Cette fonction ne peut et ne doit par conséquent pas être utilisée pour purger le tampon du flux d'entrée standard (stdin).

scanf() est une fonction très complexe et réservée aux spécialistes. Personnellement, je ne l'utilise jamais. La solution suivante à base de fgets() est parfaitement adaptée:

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
60
61
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
/* -TC- Cette fonction est a utiliser en complement de fgets() pour detecter
   eliminer le caractere de fin de ligne saisi pat cette dernière et purger
   le flux d'entree standard si necessaire */
void fclean(char *tampoon, FILE *flux);
 
int main(void)
{
    /* -TC- Ce declare un tampon d'une longueur suffisante pour la saisie */
    char tampon[20] = "";
    char *pend = NULL;
    int num;
 
    do
    {
        printf("Donnez votre num: ");
        /* -TC- Cette fonction permet de purger le tampon du flux de sortie standard
           et de forcer l'affichage de "Donnez votre nom: " */
        fflush(stdout);
 
        fgets(tampon, sizeof tampon, stdin);
        fclean(tampon, stdin);
        num = strtol(tampon, &pend, 0);
    } while (*pend != 0);
 
    printf("Votre num est: %d\n", num);
 
    return 0;
}
 
void fclean(char *tampon, FILE *flux)
{
    /* On cherche si le caractere '\n' est present */
    char *pc = strchr(tampon, '\n');
 
    if (pc != NULL)
    {
        /* -TC- Il est present: la saisie est donc complete */
 
        /* OPTIONNEL: On remplece de caractere de fin de ligne par un caractere
           nul */
        *pc = 0;
    }
    else
    { /* -TC- '\n' ne peut etre trouve' -> la saisie a ete tronquee et il reste
        des caracteres sur le flux d'entree standard */
 
        int c;
 
        /* -TC- On affiche un message d'avertissement*/
        fprintf(stderr, "ATTENTION: La saisie a été tronquée!\n");
 
        /* -TC- On vide le tampon du flux d'entree standard */
        while ((c = fgetc(flux)) != '\n' && c != EOF)
        {
        }
    }
}

N'hésite pas à poser des questions si tu ne comprends pas.

Thierry

10/09/2007, 09h34
LLB

Citation:

Envoyé par Emmanuel Delahaye

Sauf qu'en cas de saisie erronée, on retombe sur le même problème. De plus, le résultat est différent (il y a conversion texte -> valeur numérique).

Ah oui, en effet. À la base (avant modération), mon message indiquait surtout que la question était au mauvais endroit et qu'elle n'avait pas de rapport avec le Shell. Je n'aurais pas osé faire une réponse si simple sur le forum C.
10/09/2007, 09h40
Laughing Man

ué c'est vrai j'ai répondu trop vite. en fouillant dans mes sources j'ai retrouvé du fgetc partout
du coup, la meilleure solution c'est de passer outre scanf pour entrer juste un caractère je pense
désolé de cette réponse à deux ronds:mouarf:
10/09/2007, 19h03
_SamSoft_

Ou utilise fgets(maVar, sizeof maVar, stdin); :)
10/09/2007, 21h07
Skyounet

Citation:

Envoyé par _SamSoft_

Ou utilise fgets(maVar, sizeof maVar, stdin); :)

Avec maVar de type char* ou char[].

Citation:

Envoyé par _SamSoft_

Ou utilise fgets(maVar, sizeof maVar, stdin); :)

Si maVar est de type pointeur sur char, il y aura un problème, car dans ce cas, sizeof maVar vaudra sizeof (char *), ce qui ne correspond pas à la taille du tampon pointé par maVar.
Code:

1 2 3 4 5 6 7 8 9 void saisir_ligne(char *tampon, size_t longueur) { if (tampon != NULL) { /* ATTENTION: fgets(tampon, sizeof tampon, stdin) n'est PAS correct */ fgets(tampon, longueur, stdin); fclean(tampon, stdin); } }
Thierry

11/09/2007, 16h18
_SamSoft_

J'avais oublié les pointeurs :aie: