Recherche de voyelles dans un fichier

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Recherche de voyelles dans un fichier

lassault1

Bonjour a tous

Pourquoi mon code ne trouve pas tout les voyelles qui se trouve dans mon texte ?

Code :

#include <stdlib.h>
#include <stdio.h>
#define TAILLE_MAX 1000
int main(int argc, char *argv[])
{
char voy[6] = {'a','e','i','o','u','y'};
int compte[6] = {0}, i;
FILE* fichier = NULL;
char chaine[TAILLE_MAX] = "";
if ((fichier = fopen("test.txt", "r" )) != NULL)
{
while (fgets(chaine, TAILLE_MAX, fichier) != NULL) // On lit le fichier tant qu'on ne reçoit pas d'erreur (NULL)
{
for(i = 0; i < 6; i++)
if(chaine[i] == voy[i])
compte[i] = compte[i] + 1; // On affiche la chaîne qu'on vient de lire
}
printf("Votre texte comporte :\n" );
for(i = 0; i < 6; i++)
printf("%d fois la lettre %c\n", compte[i], voy[i]);
fclose(fichier);
}
return 0;
}

Publicité

Un Programmeur

- tu ne regardes que les 6 premiers caractères de chaque ligne lue
- tu ne compares chacun de ces caractères qu'avec une seule voyelle

En prime,
- les constantes magiques comme 6, c'est mal
- l'indentation aléatoire

---------------
The truth is rarely pure and never simple (Oscar Wilde)

lassault1

Je dois faire quoi pour l'arranger?

Un Programmeur

- regarder chaque caractère de tes lignes
- les comparer avec toutes les voyelles
- définir un nom symbolique et l'utiliser à la place de 6
- indenter correctement

---------------
The truth is rarely pure and never simple (Oscar Wilde)

gilou

Modérateur
It's the only NEET thing to do

Quand tu fais
for(i = 0; i < 6; i++)
if(chaine[i] == voy[i])
tu testes seulement si le i-eme caractère de la chaine est la i-eme voyelle au lieu de tester si chaque caractère de la chaine est la i-me voyelle.

En modifiant un peu ton code ceci marche:

Code :

#include <stdio.h>
#include <string.h>
#define TAILLE_MAX 1000
int main(int argc, char *argv[])
{
char voy[6] = {'a','e','i','o','u','y'};
int compte[6] = {0}, i, j;
FILE* fichier = NULL;
char chaine[TAILLE_MAX] = "";
if ((fichier = fopen("test.txt", "r" )) != NULL)
{
while (fgets(chaine, TAILLE_MAX, fichier) != NULL) // On lit le fichier tant qu'on ne reçoit pas d'erreur (NULL)
{
for (i = 0; i <strlen(chaine); i++) {
for(j = 0; j < 6; j++) {
if(chaine[i] == voy[j])
compte[j] = compte[j] + 1; // On affiche la chaîne qu'on vient de lire
}
}
}
printf("Votre texte comporte :\n" );
for(i = 0; i < 6; i++)
printf("%d fois la lettre %c\n", compte[i], voy[i]);
fclose(fichier);
}
return 0;
}

Une version un peu plus évoluée, qui teste aussi les majuscules:

Code :

#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#define TAILLE_MAX 1000
int main(int argc, char *argv[])
{
char voy[12] = {'a', 'e', 'i', 'o', 'u', 'y',
'A', 'E', 'I', 'O', 'U', 'Y'};
int compte[6] = {0};
char buffer[TAILLE_MAX];
FILE* fichier = fopen("test.txt", "r" );
int i, j, lgchaine;
if (!fichier) {
printf("Cannot open text.txt\n" );
return EXIT_FAILURE;
}
while (fgets(buffer, TAILLE_MAX, fichier)) {
lgchaine = strlen(buffer);
for (i = 0; i <lgchaine; ++i) { // Pour chaque caractère de la chaine
for(j = 0; j < 6; ++j) { // Pour chaque voyelle
if(buffer[i] == voy[j] || buffer[i] == voy[j+6]) { // On teste la minuscule et la majuscule
++compte[j];
break; // Si on a trouvé une voyelle, inutile de tester les autres
}
}
}
}
fclose(fichier);
printf("Votre texte comporte :\n" );
for(i = 0; i < 6; ++i) {
printf("%d fois la lettre %c\n", compte[i], voy[i]);
}
return EXIT_SUCCESS;
}

La même chose avec un switch:

Code :

#include <stdlib.h>
#include <stdio.h>
#define TAILLE_MAX 1000
int main(int argc, char *argv[])
{
char voy[6] = {'a', 'e', 'i', 'o', 'u', 'y'};
int compte[6] = {0};
char buffer[TAILLE_MAX];
FILE* fichier = fopen("test.txt", "r" );
int i;
if (!fichier) {
printf("Cannot open text.txt\n" );
return EXIT_FAILURE;
}
while (fgets(buffer, TAILLE_MAX, fichier)) {
i = 0;
while (buffer[i]) {
switch (buffer[i]) {
case 'a': case 'A': ++compte[0]; break;
case 'e': case 'E': ++compte[1]; break;
case 'i': case 'I': ++compte[2]; break;
case 'o': case 'O': ++compte[3]; break;
case 'u': case 'U': ++compte[4]; break;
case 'y': case 'Y': ++compte[5]; break;
default: break;
}
++i;
}
}
fclose(fichier);
printf("Votre texte comporte :\n" );
for(i = 0; i < 6; ++i) {
printf("%d fois la lettre %c\n", compte[i], voy[i]);
}
return EXIT_SUCCESS;
}

A+,

Message cité 1 fois
Message édité par gilou le 17-04-2010 à 16:06:12

---------------
I think you guys should really consider virtualizing the whole process and moving it to the cloud.

Sve@r

gilou a écrit :

Une version un peu plus évoluée, qui teste aussi les majuscules:

Code :

#include <stdlib.h>
#include <stdio.h>
#include <string.h>
#define TAILLE_MAX 1000
int main(int argc, char *argv[])
{
char voy[12] = {'a', 'e', 'i', 'o', 'u', 'y',
'A', 'E', 'I', 'O', 'U', 'Y'};
...
if(buffer[i] == voy[j] || buffer[i] == voy[j+6]) { // On teste la minuscule et la majuscule
}

Mouais. tolower() aurait été ici assez approprié...

Code :

if(tolower(buffer[i]) == voy[j]) { // On teste la minuscule et la majuscule
...

Là où on s'amuserait, ce serait pour tester les é, è, ê, ë, î, ï, ...

Message édité par Sve@r le 17-04-2010 à 18:14:54

gilou

Modérateur
It's the only NEET thing to do

En effet. ça serait surtout approprié dans le switch (buffer[i])
un switch (tolower(buffer[i])) permet de virer la moitié des cases du switch.
De toute façon, si j'avais vraiment codé cela en dehors d'une amélioration du code donné, j'aurais introduit une structure de donnée struct avec un champ lettre et un champ compte, car c'est ce que suggère le problème.
A+,

Message cité 1 fois

---------------
I think you guys should really consider virtualizing the whole process and moving it to the cloud.

Sve@r

gilou a écrit :

De toute façon, si j'avais vraiment codé cela en dehors d'une amélioration du code donné, j'aurais introduit une structure de donnée struct avec un champ lettre et un champ compte, car c'est ce que suggère le problème.

Voire même un champ "tableau de lettres" associé au champ compte ce qui permet ainsi de compter les é, è, ê comme la même lettre...

gilou

Modérateur
It's the only NEET thing to do

Je ferais plutôt cela au niveau d'une fonction dédiée mytolower, car ça risque de compliquer l'algorithmique d'un exo a la base simple, mais c'est une question de gout.

Et on peut vite arriver aux questions métaphysiques: si on compte aussi les consonnes, un ß compte t'il pour deux s? etc
A+,

Message édité par gilou le 17-04-2010 à 19:20:21

---------------
I think you guys should really consider virtualizing the whole process and moving it to the cloud.

lassault1

Merci gilou c'est ok

Si j'ai compris mon 1er code test que les 6 premiers caractères sur 1000 c'est ça ou pas?

Code :

while (fgets(chaine, TAILLE_MAX, fichier) != NULL)
{
for(i = 0; i < 6; i++)
if(chaine[i] == voy[i])
compte[i] = compte[i] + 1;
}

PS: Tu utilise for (i = 0; i <strlen(chaine); i++) mais je peut mettre aussi for (i = 0; i <TAILLE_MAX; i++) ?

Message cité 1 fois

gilou

Modérateur
It's the only NEET thing to do

lassault1 a écrit :

Merci gilou c'est ok

Si j'ai compris mon 1er code test que les 6 premiers caractères sur 1000 c'est ça ou pas?

Code :

while (fgets(chaine, TAILLE_MAX, fichier) != NULL)
{
for(i = 0; i < 6; i++)
if(chaine[i] == voy[i])
compte[i] = compte[i] + 1;
}

Même pas: ton code teste si le premier caractère est 'a', si le 2e est 'e', si le 3e est 'i'... et si le 6e est 'y'.

lassault1 a écrit :

PS: Tu utilise for (i = 0; i <strlen(chaine); i++) mais je peut mettre aussi for (i = 0; i <TAILLE_MAX; i++) ?

Surtout pas: si la ligne lue par fgets et copiée dans chaine fait moins que TAILLE_MAX caractères tu vas continuer a tester pour des caractères qui trainent dans le reste du tableau chaine, mais qui n'ont pas été mis la par fgets.
C'est a ça que sert la limitation i <strlen(chaine): a ne tester que les caractères effectivement copiés par fgets.
fgets met un '\0' a la fin de ce qu'il copie, et c'est ce que détecte strlen pour calculer la bonne longueur.
A+,

Message édité par gilou le 17-04-2010 à 19:56:12

---------------
I think you guys should really consider virtualizing the whole process and moving it to the cloud.

lassault1

Citation :

Même pas: ton code teste si le premier caractère est 'a', si le 2e est 'e', si le 3e est 'i'... et si le 6e est 'y'.

C'est pas ce que j'ai dis :??: ?

Citation :

Ok ! dans ce cas strlen(chaine) sert a optimisé la boucle ?

Message cité 1 fois

Un Programmeur

Non, à la rendre correcte. Si par hasard (ou parce que la ligne précédente était plus longue) il y a une voyelle après la fin de la ligne, tu la compterais.

Pour optimiser, il vaut mieux ne calculer strlen qu'une fois en stockant le résultat dans une variable, ce qu'avait fait Gilou dans une de ses versions.

---------------
The truth is rarely pure and never simple (Oscar Wilde)

gilou

Modérateur
It's the only NEET thing to do

lassault1 a écrit :

Citation :

Même pas: ton code teste si le premier caractère est 'a', si le 2e est 'e', si le 3e est 'i'... et si le 6e est 'y'.

C'est pas ce que j'ai dis :??: ?

Non, ce que tu as dit, que tu testes seulement les 6 premier caracteres, ca signifierait que ton code teste si le premier caractère est 'a',ou 'e', ou 'i'... ou 'y', puis qu'il teste si le 2e caractère est 'a',ou 'e', ou 'i'... ou 'y', ... puis qu'il teste si le 6e caractère est 'a',ou 'e', ou 'i'... ou 'y'.
A+,

---------------
I think you guys should really consider virtualizing the whole process and moving it to the cloud.

lassault1

Code :

# if ((fichier = fopen("test.txt", "r" )) != NULL)
# {
# while (fgets(chaine, TAILLE_MAX, fichier) != NULL) // On lit le fichier tant qu'on ne reçoit pas d'erreur (NULL)
# {
# for (i = 0; i <strlen(chaine); i++) {
# for(j = 0; j < 6; j++) {
# if(chaine[i] == voy[j])
# compte[j] = compte[j] + 1; // On affiche la chaîne qu'on vient de lire
# }
# }
# }

Pourquoi pas avoir mis tout simplement for(i = 0; i < sizeof(chaine); i++) au lieu de :

Code :

for (i = 0; i <strlen(chaine); i++)

gilou

Modérateur
It's the only NEET thing to do

Parce que sur un tableau, sizeof d'un tableau, ca renvoie le nombre total de cases, ca se fout complètement de la position du \0 dans le tableau.
Exemple:
char chaine[8] = "Toto"; ca mt les caractères T, o, t, o et \0 en position 0, 1, 2, 3 et 4 du tableau.
sizeof(chaine) vaut 8, mais strlen(chaine) vaut 4 (4 caractères avant le \0)
A+,

---------------
I think you guys should really consider virtualizing the whole process and moving it to the cloud.

lassault1

Merci gilou..

Publicité

FORUM HardWare.fr

Programmation

Recherche de voyelles dans un fichier

Sujets relatifs
Recherche d un enregistrement en utilisant Hibernate et Struts	[SDL] Acceder a une surface depuis n'importe quel fichier ?
[BATCH] creation de repertoires par parcours de fichier texte	Télécharger un fichier depuis batch
Récupération des données d'un fichier .hex	[PHP/MYSQL] formulaire et modification à distance de fichier
Recherche un lecteur flash progressive download complet svp	Création d'un outil de visualisation et de recherche PDF
Création d'un outil de visualisation et de recherche PDF	Transformation d'un tableau (fichier xml vers html)
Plus de sujets relatifs à : Recherche de voyelles dans un fichier

Page générée en 0.497 secondes