Discussion :

[Elaich]

Bonjour,
j'essaye de faire une liste de chevaux de course en C, voici ma structure:

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
 
struct Liste_ch
 
{
 
       char Nom[20];
 
       int age;
 
       char sexe;
 
       int courses;
 
       int victoires;
 
       char driver[20];
 
       struct Liste_ch *next;
 
};

j'ai définis une fonction pour ajouter un élément à la liste que voice:

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
// fonction d'ajout d'un cheval à la liste de course.
 
 
 
liste_ch* ajouter_cheval(liste_ch *lcheval)
 
{
 
          liste_ch *l;
 
          if((l=(liste_ch*)malloc(sizeof(liste_ch)))==NULL)
 
          {printf("Erreur d'allocation !\n");
 
          exit(-1);
 
          }
 
 
 
          printf("Entrer le nom du cheval:\n");
 
          scanf("%s",l->Nom);
 
          printf("Entrer l'age:\n");
 
          scanf("%d",&l->age);
 
          printf("Entrer le sexe M/F/H: \n");
 
          l->sexe = getchar();
 
          printf("Entrer le nombre de courses du cheval:\n");
 
          scanf("%d",&l->courses);
 
          printf("Entrer le nombre de victoires du cheval:\n");
 
          scanf("%d",&l->victoires);
 
          printf("Entrez le nom du driver:\n");
 
	  scanf("%s",l->driver);
 
 
 
          l->next= lcheval;
 
          lcheval = l;
 
 
 
          return(l);
 
          }

le premier souci que j'ai eu c'est que je peux pas utiliser la fonction "gets" pour lire mes chaines de caractères, le compilateur envois ce message:

Code :

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

1
2
In function `ajouter_cheval':
liste.c:(.text+0xf9): warning: the `gets' function is dangerous and should not be used.

je veux bien utiliser "gets" à la place de " scanf("%s") ".
le deuxième souci c'est quand je veux lire le caractère sexe, à l'éxecution du programme ça saute toujours cette étape, et je peux rien entrer,
je sais pas ou est l'erreur, quelqu'un peut m'aider ??
merci.


Marouane.

[jeroman]

Salut

1) Concernant gets :

N'utilisez jamais gets(). Comme il est impossible de savoir à l'avance combien de caractères seront lus par gets(), et comme celui-ci écrira tous les caractères lus, même s'ils débordent du buffer, cette fonction est extrèmement dangereuse à utiliser. On a déjà utilisé ce dysfonctionnement pour créer des trous de sécurité. UTILISEZ TOUJOURS fgets() A LA PLACE DE gets().

( http://man.developpez.com/man3/fgets.3.php )

2) Il faut vider le buffer clavier avant d'appeler getchar.

[Elaich]

Salut,
merci jeroman pour ta réponse,
j'ai modifier ma fonction en ésperant que ça resoudera le problème, mais je crois que j'ai pas bien fait, voici la partie du code qui m'interesse:

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
 
          printf("Entrer l'age:\n");
 
          scanf("%d",&l->age);
 
          fflush(stdin);
 
	  printf("Entrer le sexe M/F/H: \n");
 
          l->sexe = getchar();
 
          printf("Entrer le nombre de courses du cheval:\n");
 
          scanf("%d",&l->courses);
 
          printf("Entrer le nombre de victoires du cheval:\n");
 
          scanf("%d",&l->victoires);
 
	  fflush(stdin);
 
          printf("Entrez le nom du driver:\n");
 
	  fgets(l->driver,20,stdin);

mais ça saute toujours les deux étapes de lecture, du "sexe" et du "driver",

[Sve@r]

2) Il faut vider le buffer clavier avant d'appeler getchar.

Non. Un buffer clavier ne peut pas être vidé (fflush(stdin) donne un comportement indéterminé) et de toute façon n'a pas besoin d'être vidé.
En effet, si toutes les saisies ont été bien faites avec fgets() suivi éventuellement de sscanf(), le clavier est toujours vide quand le programme arrive à la saisie suivante.

Salut,
merci jeroman pour ta réponse,
j'ai modifier ma fonction en ésperant que ça resoudera le problème, mais je crois que j'ai pas bien fait, voici la partie du code qui m'interesse:

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
 
          printf("Entrer l'age:\n");
 
          scanf("%d",&l->age);
 
          fflush(stdin);
 
	  printf("Entrer le sexe M/F/H: \n");
 
          l->sexe = getchar();
 
          printf("Entrer le nombre de courses du cheval:\n");
 
          scanf("%d",&l->courses);
 
          printf("Entrer le nombre de victoires du cheval:\n");
 
          scanf("%d",&l->victoires);
 
	  fflush(stdin);
 
          printf("Entrez le nom du driver:\n");
 
	  fgets(l->driver,20,stdin);

mais ça saute toujours les deux étapes de lecture, du "sexe" et du "driver",

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
 
          char saisie[80];
          printf("Entrer l'age:\n");
          fgets(saisie, 80, stdin);
          sscanf(saisie, "%d",&l->age);
 
	  printf("Entrer le sexe M/F/H: \n");
          l->sexe = getchar();
 
          printf("Entrer le nombre de courses du cheval:\n");
          fgets(saisie, 80, stdin);
          sscanf(saisie, "%d",&l->courses);
 
          printf("Entrer le nombre de victoires du cheval:\n");
          fgets(saisie, 80, stdin);
          sscanf("%d",&l->victoires);
 
          printf("Entrez le nom du driver:\n");
	  fgets(l->driver,20,stdin);

[gl]

Non. Un buffer clavier ne peut pas être vidé (fflush(stdin) donne un comportement indéterminé) et de toute façon n'a pas besoin d'être vidé.
En effet, si toutes les saisies ont été bien faites avec fgets() suivi éventuellement de sscanf(), le clavier est toujours vide quand le programme arrive à la saisie suivante.

Petit bémol, le buffer n'est pas toujours vide après un fgets(), notamment lorsque la taille de la ligne est plus importante que celle du buffer (par exemple si je saisie 90 caractères dans ton exemple).

Dans ce cas, il faut bel et bien vider le buffer clavier. Mais effectivement, il ne faut pas utiliser fflush() pour réaliser cette opération mais lire tout les caractères supplémentaires (chercher fclean() dans le forum pour plus d'info à ce sujet).

[Elaich]

Salut,
merci Sve@r, ta solution marche mais pas tout a fait, d'abord je comprend pas pourquoi je dois écrire dans une chaîne de caractère puis lire depuis cette chaîne, pourquoi pas lire depuis le clavier directement, plus pratique , non ? avec fgets(...,...,stdin), je dois seulement vider le tampon après chaque lecture, est ce qu'il y a une solution simple à ce problème ?
Et merci.


Marouane

[Sve@r]

Salut,
merci Sve@r, ta solution marche mais pas tout a fait, d'abord je comprend pas pourquoi je dois écrire dans une chaîne de caractère puis lire depuis cette chaîne, pourquoi pas lire depuis le clavier directement, plus pratique , non ? avec fgets(...,...,stdin)
Marouane

Le gros problème d'un scanf("%d", ...) c'est que ça attend une entrée formatée. Ici il lui faut du nombre et ne traitera que du nombre.
Mais celui qui répond à l'ordinateur peut entrer ce qu'il veut. Et s'il entre "toto" au lieu du nombre attendu, ou "12toto", le scanf récupère la valeur "12" mais laissera la chaine "toto" dans le buffer stdin.
Ensuite, si par hasard tu fais un scanf("%s", nom) ou même un fgets(), la fonction récupèrera la chaine "toto" qui y est restée depuis la saisie précédente. D'où le conseil malheureux du vidage du buffer stdin que t'as traduit par un fflush(stdin) inutile.

Et même si tu entres bien le nombre "12" comme il faut, tu termines tout de même ta saisie par un "<return>". Ben ce <return> n'étant pas du nombre, scanf le laisse dans le stdin.

Dans la solution que je te présente, on commence par saisir une string. Quoi que soit ce qui est entré, il va intégralement dans la variable "saisie" et disparait donc du clavier => plus de problème de fflush(stdin).
Ensuite avec sscanf(), tu extraits de la chaine le nombre attendu. Tu peux même tester la réussite de l'opération car sscanf() te renvoie le nombre réel de variables qui ont pu être remplies donc si sscanf() ne donne pas 1, c'est qu'il n'a pas fonctionné => permet de reboucler sur la saisie si tu le désires

, je dois seulement vider le tampon après chaque lecture, est ce qu'il y a une solution simple à ce problème ?

Oui, celle-là car le tampon est automatiquement vidé par fgets(). C'est une des meilleures solutions préconisées (et elle n'est pas de moi mais je crois de Emmanuel Delahaye, le cador du C de ce forum).

Avec cette méthode, tu peux même créer un fichier contenant les réponses attendues et donner le fichier à manger à ton programme via un pipe. Ton programme déroulera et ingurgitera sans problème les réponses prévues dans le fichier (chose qui ne fonctionne pas quand on vide le buffer après la saisie car on court le risque de vider une info utile)

Petit bémol, le buffer n'est pas toujours vide après un fgets(), notamment lorsque la taille de la ligne est plus importante que celle du buffer (par exemple si je saisie 90 caractères dans ton exemple).

Exact. Je pense que 80 est une bonne valeur mais peut ne pas suffire face à ce qu'on nomme communément "test du crétin".

Dans ce cas, il faut bel et bien vider le buffer clavier. Mais effectivement, il ne faut pas utiliser fflush() pour réaliser cette opération mais lire tout les caractères supplémentaires (chercher fclean() dans le forum pour plus d'info à ce sujet).

Perso je préfère éviter de vider le buffer clavier (pour les raisons que je mentionne plus haut). Si vraiment on doit faire une saisie "cretin proof", je conseillerais alors de passer par un getline() qui adapte automatiquement la zone à remplir en fonction du nombre de caractères saisis.