Discussion :

[Redgard]

bonjour,

J'aimerais dénombrer les mots d'un file, tout en étant évitant certaines erreurs qui viendrait fausser le résultat.
En cherchant sur internet, je suis souvent tombé sur différente variante de la fonction suivante, mais ce qui me dérange c'est que dans le cas de figure où tu as une ligne vide ou la dernière ligne sans '\n', ton résultat est faussé.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
int WordCounterBIS(FILE *pFile)
{
	int numbWord = 0;
	char readedCharacter = '0';
 
	while ((readedCharacter = fgetc(pFile)) != EOF)
	{
		if (readedCharacter == '\n')
			numbWord++;
	}
 
	return numbWord;
}

En cherchant de mon côté, J'ai tenté ma chance avec fscanf et j'obtiens finalement la fonction suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
int WordCounter(FILE *pFile)
{
	int numbWord = 0, attribution = 0;
	char readedWord[100];
 
	while ((attribution = fscanf(pFile,"%s", readedWord)) != EOF)
	{
		if (attribution == 1)
			numbWord++;
	}
 
	return numbWord;
}

Y a-t-il d'autres approches?

Cordialement,
Red'

[Bousk]

Salut,

pourquoi ne pas faire l'inverse ?
Quand tu vois un caractère, tu commences à lire un mot et incrémentes le compteur de mot, quand tu vois un délimiteur, t'es plus dans un mot.
En partant de ton premier code

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
int WordCounterBIS(FILE *pFile)
{
	int numbWord = 0;
	int isReadingWord = 0;
	char readedCharacter = '0';
 
	while ((readedCharacter = fgetc(pFile)) != EOF)
	{
		if (readedCharacter != '\n' && isReadingWord == 0)
		{
			++numbWord;
			isReadingWord = 1;
		}
		else if (readedCharacter == '\n')
			isReadingWord = 0;
	}
 
	return numbWord;
}

Bon bien sûr là c'est super simpliste, parce que le seul délimiteur c'est le retour à la ligne, ce qui revient à compter les lignes d'un fichier contenant quelquechose et non les mots.

[transgohan]

Je ne comprends pas ce que vous cherchez à faire.
Compter le nombre de ligne ? (ce que fait la première fonction)
Ou bien compter le nombre de mots dans un fichier texte ?

Pour les mots il faut se faire un dictionnaire des séparateurs (espace, virgule, point, ect).
Puis chercher tout séparateur et compter.

Edit : ah bah voilà ce que c'est de vouloir corriger ses fautes d'orthographe, on se retrouve second.

[Matt_Houston]

Extraire les mots d'un bloc de texte est un exercice qui revient très souvent sur le forum. Nous avons déjà donné pas mal de fonctions voire programmes en exemple, tu devrais en retrouver assez facilement.

En règle générale, ça se résume à quelque chose du style de tant que non EOF faire :

1. boucler pour lire le mot..
2. faire un truc quelconque avec le mot
3. boucler pour avancer jusqu'au prochain mot..

Et on utilise soit strcspn / strspn, soit les outils de ctype.h pour être un peu moins « locale dependent »..

Je te conseille de séparer le traitement de son contexte en écrivant une fonction qui opère soit sur une chaîne terminée par NUL (const char *), soit sur un buffer générique (const void *, size_t) sans t'occuper de la source des données. Là tu risques de te mélanger les pinceaux entre les bugs d'entrée-sortie et la logique du traitement que tu essaies de mettre en œuvre.

[Redgard]

Merci pour vos interventions, c'est intéressant de voir les différentes approches possibles.

Pour ta remarque Matt, je vais pas te cacher que je ne suis pas rendu aussi loin donc je n''ai pas du tout compris ta remarque sur le traitement. J'ai encore du pain sur la planche

[Sve@r]

Bonjour

Pour ta remarque Matt, je vais pas te cacher que je ne suis pas rendu aussi loin donc je n''ai pas du tout compris ta remarque sur le traitement.

Ce qu'il explique, c'est qu'en séparant les choses tu deviens plus modulaire. Par exemple au lieu de compter les mots d'un FILE*, tu devrais juste faire une fonction qui compte les mots d'un char*.
Ensuite, tu peux appeler cette fonction au choix avec le char* chargé à partir de la lecture d'un FILE*, ou d'autre chose (ex un pipe, un socket, etc etc etc)...

Pour les mots il faut se faire un dictionnaire des séparateurs (espace, virgule, point, ect).

Edit : ah bah voilà ce que c'est de vouloir corriger ses fautes d'orthographe, on se retrouve second.

Hé oui, pas de bol. Et en plus tout n'a pas été corrigé...

[Redgard]

bah en fait, j'ai un peu peur de trop fractionner mes fonctions.

Sinon si j'ai bien compris:
- Vous me conseigner d'utiliser une fonction comptant les mots à partir d'un pointer de caractères (?)
- et De copier le contenu du FILE* dans un tableau de caractères(?)

L'intérêt étant d'avoir une fonction plus flexible, moins spécialisé, utilisable dans d'autres circonstances.

[transgohan]

C'est tout à fait cela.
Cela permet d'être plus modulaire, plus lisible, plus compréhensible.
De plus tu pourras plus facilement réutiliser cette fonction sur un autre projet sans besoin de modifications.

C'est un peu comme si on écrivait des pavés de texte tout alignés à la suite, c'est imbuvable et peu productif.
Par contre si on espace, qu'on fait des retours à la ligne et qu'on insère des chapitres alors c'est tout autre chose.