Discussion :

[nick_08]

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 :

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

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 :

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

Donner votre num:

(la je donne la valeur f par exemple)

Code :

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

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

[LLB]

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

[Laughing Man]

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

[Emmanuel Delahaye]

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 :

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

Donner votre num:

(la je donne la valeur f par exemple)

Code :

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

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 :

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

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

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 :

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

[Emmanuel Delahaye]

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.

[Emmanuel Delahaye]

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

[Thierry Chappuis]

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 :

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

[LLB]

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.

[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

[_SamSoft_]

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

[Skyounet]

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

Avec maVar de type char* ou char[].

[Thierry Chappuis]

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 :

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

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

[_SamSoft_]

J'avais oublié les pointeurs