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.

[gl]

Perso je préfère éviter de vider le buffer clavier (pour les raisons que je mentionne plus haut).

Pourquoi précisément ? (Je parle bien ici d'un nettoyage par lecture de tous les caractères présent, pas d'un quelconque appel à fflush()).

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.

Le gros problème de getline() c'est quelle n'est pas standard et pas forcément triviale à refaire proprement (même s'il doit être possible de trouver des implémentations correctes).

[Sve@r]

Pourquoi précisément ? (Je parle bien ici d'un nettoyage par lecture de tous les caractères présent, pas d'un quelconque appel à fflush()).

C'est vrai qu'en y réfléchissant, boucler jusqu'au '\n' fonctionnera aussi. Mais intrinsèquement je n'aime pas l'idée de dire "il a entré 90 caractères donc je décide qu'il y en a 10 de trop". C'est vrai que dire aussi "mon champ fait 80 caractères donc s'il en entre 90 je pourrai pas tous les stocker donc faudra que je purge" est aussi une voie possible mais bon, c'est perso quoi. Il y en a une que j'apprécie et pas l'autre. Comme Brice de Nice, "aujourd'hui je sens que le C ne veut pas que je programme ainsi"

Le gros problème de getline() c'est quelle n'est pas standard et pas forcément triviale à refaire proprement (même s'il doit être possible de trouver des implémentations correctes).

Ouaip. On peut pas tout avoir. Il y a des avantages et inconvénients à tout...

[gl]

C'est vrai qu'en y réfléchissant, boucler jusqu'au '\n' fonctionnera aussi. Mais intrinsèquement je n'aime pas l'idée de dire "il a entré 90 caractères donc je décide qu'il y en a 10 de trop". C'est vrai que dire aussi "mon champ fait 80 caractères donc s'il en entre 90 je pourrai pas tous les stocker donc faudra que je purge" est aussi une voie possible mais bon, c'est perso quoi.

Je suis d'accord que c'est une façon de gérer le problème, pas la seule et pas toujours la meilleure.
La purge des caractères surnuméraires est juste mon comportement par défaut (comprendre celui que j'applique en dehors de tout contexte ou sur des petits codes de test). Sinon, bien évidement, j'adapte la stratégie retenue en fonction du contexte.

PS : au passage, il ne suffit pas de boucler jusqu'au \n, il faut en fait s'arrêter effectivement sur un \n ou sur un "échec" de lecture (retour EOF de getchar()).

[Elaich]

Salut,
merci Sve@r pour tout, je crois que j'ai compris, on crée notre propre "buffer" qui est la chaîne de caractère "saisie" tout ce que je tape au clavier (stdin) va vers la chaîne, puis avec "sscanf" je lis vers mes variables, j'ai une question est ce que j'ai pas besoin de vider la chaîne après chaque "sscanf" ? Sinon je crois qu'il y a un problème après la ligne

Code :

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

l->sexe = getchar();

, parce que à l'execution il saute le fgets suivant cette ligne, donc c'est clair que le programme à lu quelque chose quelque part !
Ceci dit je vais essayer de faire une autre version en utilisant un fichier (flot) et je vais voir ce que ça va donner.

[Sve@r]

Salut,
merci Sve@r pour tout, je crois que j'ai compris, on crée notre propre "buffer" qui est la chaîne de caractère "saisie" tout ce que je tape au clavier (stdin) va vers la chaîne, puis avec "sscanf" je lis vers mes variables, j'ai une question est ce que j'ai pas besoin de vider la chaîne après chaque "sscanf" ?

Que signifie "vider la chaine" ? La remplir de blancs ? De zéros ? De "." ?

Imagine 2 saisies à suivre. La chaine contient d'abord la saisie n° 1 terminée par un '\0'. Quelle que soit la chaine saisie en saisie n° 2, il y aura écrasement des anciens caractères par les nouveaux et un '\0' viendra terminer la nouvelle saisie. Et toute fonction qui traitera cette chaine s'arrêtera au '\0' donc ne traitera que ce qui aura été saisi en n° 2 et tout ce qui persistera de "saisie n° 1" (donc entre le '\0' de la saisie n° 2 et le '\0' de la saisie n° 1) ne sera plus traité.

Sinon je crois qu'il y a un problème après la ligne

Code :

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

l->sexe = getchar();

, parce que à l'execution il saute le fgets suivant cette ligne, donc c'est clair que le programme à lu quelque chose quelque part !
Ceci dit je vais essayer de faire une autre version en utilisant un fichier (flot) et je vais voir ce que ça va donner.

Mouais. Remplace getchar() par fgets(chaine) puis sscanf(chaine, "%c", &l->sexe) pour être sûr que le problème vient de là...

[gl]

merci Sve@r pour tout, je crois que j'ai compris, on crée notre propre "buffer" qui est la chaîne de caractère "saisie" tout ce que je tape au clavier (stdin) va vers la chaîne, puis avec "sscanf" je lis vers mes variables, j'ai une question est ce que j'ai pas besoin de vider la chaîne après chaque "sscanf" ?

Non, la lecture suivante écrasera le contenu de la chaîne.

Sinon je crois qu'il y a un problème après la ligne

Code :

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

l->sexe = getchar();

, parce que à l'execution il saute le fgets suivant cette ligne, donc c'est clair que le programme à lu quelque chose quelque part !

Lorsque tu saisies le genre, tu rentres deux caractères : une lettre qui donne le genre et un caractère \n (l'appuie sur la touche ENTER), getchar() ne lit que le premier caractère, il reste donc un \n dans le flux qui est lu par fgets() (qui justement s'arrête sur un \n).
Il faut donc consommer ce caractère \n

[Elaich]

Salut,

Lorsque tu saisies le genre, tu rentres deux caractères : une lettre qui donne le genre et un caractère \n (l'appuie sur la touche ENTER), getchar() ne lit que le premier caractère, il reste donc un \n dans le flux qui est lu par fgets() (qui justement s'arrête sur un \n).
Il faut donc consommer ce caractère \n

Oui mais si je veux le supprimer du buffer, fflush ne suffit pas ?

[Médinoc]

Utiliser fflush() sur un flux entrant provoque un comportement indéfini: La norme C ne définit son utilisation que pour les flux sortants.

[Melem]

[FAQ] Pourquoi ne faut-il pas utiliser fflush(stdin) pour vider le buffer clavier ?

[Obsidian]

[FAQ] Pourquoi ne faut-il pas utiliser fflush(stdin) pour vider le buffer clavier ?

Il est peut-être intéressant d'ajouter, car la confusion provient généralement de là, que « vider un buffer » ne veut pas dire « effacer un buffer ». Ces tampons sont là pour retarder l'envoi de caractères vers leur destinataire, de façon à les transmettre en une seule fois, et fflush() force cet envoi.

Faire un fflush() sur un buffer d'entrée consiste donc à forcer l'envoi des caractère vers soi-même. C'est indéfini parce que l'opération elle-même n'a pas de sens mais, à mon goût, si on ne lit pas les caractères, l'opération la plus censée consiste à ne rien faire.