Est-ce une bonne utilisation de fgets ?

Salut,

Après quelques semaine de cours de langue intensif, je reprends mon apprentissage en auto-didacte du C.

Et j'ai donc pondu un petit programme de recherche de caractère basé sur un exemple de mon bouquin.

Je me demande si j'ai bien compris l'utilisation de fgets

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

#include <stdio.h> #include <string.h>   int main(void) { char *loc, buf[80]; int ch;   printf("tapez la chaîne de caractère : "); fgets(buf,80,stdin); printf("tapez le caractère a trouver : "); ch = getchar();   loc = strchr(buf, ch);   if (loc == NULL) printf("On n'a pas trouvé le caractère %c", ch); else printf("Le caractère %c a été trouvé en position %d\n", ch, loc-buf);   return 0; }

---

Bon visiblement le programme compile bien sur mon Mac, mais est-ce j'ai bien utilisé fgets... ou bien aurais-je du utilisé une autre fonction de saisie ?

C'est bien fgets() qu'il faut utiliser, mais ton utilisation ne prend pas tout en compte:

• Notamment, si la chaîne tapée fait moins de 80 caractères, le \n sera inclus dedans.
• Sinon, les caractères non-lus seront toujours dans le buffer d'entrée du terminal (le le getchar() sera alors non-bloquant).

Pour résoudre ces problèmes, on se fait généralement une fonction qui vide la ligne courante:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

#include <stdio.h>   int PurgeStream(FILE *pfIn) { int c; do { c = fgetc(pfIn); } while(c != '\n' && c != EOF); if(c==EOF) return -1; return 0; }

---

Et ensuite, on se fait souvent une fonction à utiliser après chaque fgets(). Là, ce qui arrive au 'reste' de la ligne dépend de ce que veut le programmeur:

• Soit on l'ignore (le plus simple)
• soit on joue avec l'allocation dynamique pour retourner toute la ligne (avancé, et il faut connaitre l'allocation dynamique).

Dans le cas simple, une fonction comme celle-ci suffit:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

#include <string.h> #include <stdio.h>   int PurgeStream(FILE*);   int PurgeLineAndStream(char *buf, FILE *pfIn) { int ret = 0; char *pLF = strchr(buf, '\n'); /* Cherche le \n */ if(pLF != NULL) { /* Trouvé: Toute la ligne a été lue, on tronque la chaîne */ *pLF = '\0'; } else { /* Non-trouvé: Il reste des caractères dans le buffer */ ret = PurgeStream(pfIn); } return ret; }

---

Et ça s'utilise ainsi:

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

#include <stdio.h> #include <string.h>   int PurgeStream(FILE*); int PurgeLineAndStream(char *, FILE *);   int main(void) { char *loc, buf[80]; int ch;   printf("tapez la chaîne de caractère : "); fgets(buf,80,stdin); PurgeLineAndStream(buf, stdin);   printf("tapez le caractère a trouver : "); ch = getchar(); PurgeStream(stdin);   loc = strchr(buf, ch);   if (loc == NULL) printf("On n'a pas trouvé le caractère %c\n", ch); else printf("Le caractère %c a été trouvé en position %d\n", ch, loc-buf);   return 0; }

---

Et je conseille également, du moins juste par souci de maintenabilité du code, de toujours faire :

Code:

---

fgets(buffer, sizeof(buffer), stdin);

---

+1 Melem.
Sauf que pour des raisons de passage éventuel à unicode, je conseillerais plutôt l'utilisation d'une macro:

Code:

---

1 2 3 4 5 6

#ifndef ARRAYSIZE #define ARRAYSIZE(x) (sizeof(x)/sizeof(x[0])) #endif   ... fgets(buffer, ARRAYSIZE(buffer), stdin);

---

Note: Cela ne marque que sur les tableaux. Sur un pointeur, c'est la taille du pointeur qui sera prise en compte à la place.
En C++ il y a des ruses pour corriger ce problème, mais pas en C...

Ouais. Et pour loc - buf aussi.

Code:

---

(loc - buf)/sizeof(buf[0])

---

Citation:

---

Envoyé par Melem

Ouais. Et pour loc - buf aussi.

Code:

---

(loc - buf)/sizeof(buf[0])

---

---

NON : Lors d'une soustraction de pointeurs, la division est comprise.

Citation:

---

Envoyé par Médinoc

NON : Lors d'une soustraction de pointeurs, la division est comprise.

---

Donc (p + 1) - p vaut 1 et non sizeof(*p)?

Merci de vos réponses.... J'ai encore appris grâce a vous....

Citation:

---

Envoyé par Melem

Donc (p + 1) - p vaut 1 et non sizeof(*p)?

---

Oui (confirmé par tests).