Discussion :

[lassault1]

Bonjour a tous

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

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
   1. #include  <stdlib.h>
   2. #include <stdio.h>
   3. #define TAILLE_MAX 1000
   4.
   5.
   6. int main(int argc, char *argv[])
   7. {
   8.
   9. char voy[6] = {'a','e','i','o','u','y'};
  10. int compte[6] = {0}, i;
  11. FILE* fichier = NULL;
  12. char chaine[TAILLE_MAX] = "";
  13.
  14.    if ((fichier = fopen("test.txt", "r" )) != NULL)
  15. {
  16.          while (fgets(chaine, TAILLE_MAX, fichier) != NULL) // On lit le fichier tant qu'on ne reçoit pas d'erreur (NULL)
  17.           {
  18.              for(i = 0; i < 6; i++)
  19.              if(chaine[i] == voy[i])
  20.              compte[i] = compte[i] + 1; // On affiche la chaîne qu'on vient de lire
  21.           }
  22.
  23.     printf("Votre texte comporte :\n" );
  24.     for(i = 0; i < 6; i++)
  25.     printf("%d fois la lettre %c\n", compte[i], voy[i]);
  26.
  27.     fclose(fichier);
  28. }
  29.
  30.   return 0;
  31. }

[Neitsa]

Bonjour,

Le problème vient du fait que tu utilise le mauvais indice pour parcourir la chaine et/ou le tableau de voyelles.

Code C :

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

1
2
3
4
 
  18.              for(i = 0; i < 6; i++)
  19.              if(chaine[i] == voy[i])
  20.              compte[i] = compte[i] + 1;

ici tu fais:

0) lecture ligne
1) chaine[0] == voy[0] ?
2) chaine[1] == voy[1] ?
...
6) chaine[5] == voy[5] ?
7) lecture d'une autre ligne

Alors qu'une solution serait de faire:

Pour chaque caractère de la chaîne on test toutes les voyelles.

Code C :

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
 
while (fgets(chaine, TAILLE_MAX, fichier) != NULL)
{
	//calcule la longueur de la  chaine
	size_t longueur_chaine = strlen(chaine);
 
	//pour chaque caractère de la chaine
	for(i = 0; i < longueur_chaine; i++)
	{
		//pour chacune des voyelles
		for(j = 0; j < sizeof(voy); j++)
		{
			//test si le caractère courant vaut une des voyelles
			if(chaine[i] == voy[j])
				compte[j]++;
		}
	}
}

Il faut donc deux indices:

i: indice du caractère courant de la chaine
j: indice dans le tableau de voyelles

Il existe surement des solutions plus efficaces (ou tout du moins plus courte) avec des fonctions C spécifiques. Je pense à strtok(), strpbrk() ou encore strcspn(), etc.

P.S: fait attention à l'indentation et au bloc de code:

Code :

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

1
2
3
4
 
  18.              for(i = 0; i < 6; i++)
  19.              if(chaine[i] == voy[i])
  20.              compte[i] = compte[i] + 1;

Code :

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

1
2
3
4
5
6
 
               for(i = 0; i < 6; i++)
               {
                   if(chaine[i] == voy[i])
                       compte[i] = compte[i] + 1;
               }

[Melem]

Parce qu'il faut tester chaque caractère du fichier, là tu parcours le fichier ligne par ligne (si effectivement une ligne ne dépasse jamais TAILLE_MAX caractères), et tu ne testes pas chaque caractère de chaque ligne. Tu as deux options :

- Parcourir le fichier avec fgetc et tester chaque caractère (la solution que je recommande)
- Parcourir le fichier avec fgets et tester chaque caractère de chaque chaîne retournée. C'est-à-dire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
while (fgets(chaine, TAILLE_MAX, fichier) != NULL)
{
    for(i = 0; chaine[i] != '\0'; i++) // Pour chaque caractère de chaine ...
    {
        int j;
        for(j = 0; j < 6; j++) // ... tester si c'est une voyelle
        {
            if (chaine[i] == voy[j])
                compte[j] = compte[j] + 1;
        }
    }
}

[lassault1]

Ok merci a vous

1 / Au lieu de mettre longueur_chaine dans for(i = 0; i < longueur_chaine; i++) je peut mettre TAILLE_MAX ?

2/ Juste pour comprendre.. dans mon 1er code ( là ou se trouve l'erreur) j'ai :

lecture de la 1er ligne

chaine[0] == voy[0]
chaine[1] == voy[1]
...
chaine[5] == voy[5]

mais à la lecture de la 2eme ligne la valeur de i revient à 0 ? (car a la fin de la lecture de la 1ere ligne i =5)

[Melem]

1 / Au lieu de mettre longueur_chaine dans for(i = 0; i < longueur_chaine; i++) je peut mettre TAILLE_MAX ?

Non. D'une part, parce que fgets stoppe la lecture dès qu'une ligne complète a été lue, pas lorsque le buffer (ici chaine) est plein (sauf si la ligne est trop longue pour être lue d'un seul coup), or la longueur d'une ligne est variable dans un fichier. Il calculer la longueur de chaque ligne avec strlen. De plus, si la taille du buffer que tu utilises dans fgets est TAILLE_MAX, la longueur max d'une ligne, si on veut à chaque fgets lire une ligne complète, doit donc être TAILLE_MAX - 1 et non TAILLE_MAX. En effet, il faut toujours avoir de la place pour le '\0' final.

mais à la lecture de la 2eme ligne la valeur de i revient à 0 ?

Non. i revient à 0 après le i = 0 dans for(i = 0; ...). En conclusion, en effet, i revient à 0 à chaque itération.

[lassault1]

Ok merci..

Il y a un truc bête que je comprend pas..

Code :

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

1
2
3
4
 for(i = 0; i < 6; i++) 
        {
           // instructions
        }

Dans cette boucle, a la fin de l'instruction de la 1ere boucle i = 1 car il a été incrémenter, mais lorsqu'il fait la 2eme boucle il va lire i = 0. Alors pourquoi i = 1 ??

PS: Neitsa a mit sizeof(voy) dans for(j = 0; j < sizeof(voy); j++) car voy est de type char.. si ça serait un tableau de type int ça serait faux de faire comme il a fait si j'ai bien compris ?

[Melem]

Dans cette boucle, a la fin de l'instruction de la 1ere boucle i = 1 car il a été incrémenter, mais lorsqu'il fait la 2eme boucle il va lire i = 0. Alors pourquoi i = 1 ??

Tu n'as toujours pas compris le fonctionnement de for ? Dans ce cas je ne vois qu'une seule solution : [Tutoriel] L'instruction for.

Neitsa a mit sizeof(voy) dans for(j = 0; j < sizeof(voy); j++) car voy est de type char.. si ça serait un tableau de type int ça serait faux de faire comme il a fait si j'ai bien compris ?

voy n'est pas de type char. C'est, génériquement, un tableau de char et, plus précisément, un tableau de 6 char. Puisque sizeof(char), par définition même, vaut 1, il est inutile d'écrire sizeof(voy) / sizeof(char). Si voy était un tableau de int, il fallait effectivement écrire sizeof(voy) / sizeof(int). Une autre manière de faire et souvent recommandée est tout simplement d'écrire sizeof(voy) / sizeof(voy[0]).

[lassault1]

Merci a vous.. résolu

[lassault1]

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
while (fgets(chaine, TAILLE_MAX, fichier) != NULL)
{
	//calcule la longueur de la  chaine
	size_t longueur_chaine = strlen(chaine);
 
	//pour chaque caractère de la chaine
	for(i = 0; i < longueur_chaine; i++)
	{
		//pour chacune des voyelles
		for(j = 0; j < sizeof(voy); j++)
		{
			//test si le caractère courant vaut une des voyelles
			if(chaine[i] == voy[j])
				compte[j]++;
		}
	}
}

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

Code :

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

1
2
3
size_t longueur_chaine = strlen(chaine);
 
for(i = 0; i < longueur_chaine; i++)

[jeroman]

Code :

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

sizeof(chaine)

donne la taille du tableau et non de la chaine.

[lassault1]

Merci jeroman