Est-ce une bonne utilisation de fgets ?

**clampin** · 04/07/2007, 09h23

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 :

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

**Médinoc** · 04/07/2007, 09h31

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 :

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

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 :

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

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

**Melem** · 04/07/2007, 09h39

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

Code :

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

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

**Médinoc** · 04/07/2007, 09h43

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

Code :

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

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