Discussion :

[Amnael]

Bonjour; bonsoir,

je viens vous voir pour quelque chose de somme toute assez basique mais que j'aimerais voir noir sur blanc. =)

Le but; lire un fichier ( à priori avec une boucle tant que !EOF)

EN commençant ligne par ligne, puis mot par mot pour finir éventuellement caractère par caractère.


Je pense qu'une première méthode serait d'employer fgets et de compter en sommes les espaces ou ce genre de choses; mais ça ne me parait pas très fiable.
J'ai pas trouvé la page parlant de cela sur le site. Autrement pour les mots on peut boucler sur \0 ?

Donc au final un fichier sera structuré un peut comme ça ?

mot1\0 mot2\0 \n
mot3\0 \n
EOF

Comment procéderiez vous pour faire cela ?

Merci à vous !

[Amnael]

Par exemple;

Imaginons un programme qui lise un fichier (pour simplifier, j'ai considéré que je n'avais qu'un seul mot par ligne)


Voilà ce que j'ai fait; qu'en pensez vous ?

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
void frequenceMot(FILE *f)
{
	f=fopen("rep.txt","r");
	char chaine[TAILLE_MAX]="";
	char* tabMot[TAILLE_MAX]={""};
	if(f != NULL)
	{
		int k=0;
		while (fgets(chaine, TAILLE_MAX, f) != NULL) // On lit le fichier tant qu'on ne reçoit pas d'erreur (NULL)
		{
			tabMot[k]=calloc(TAILLE_MAX,sizeof(char));
			if (tabMot[k]==NULL)
			{
				fprintf(stderr, "Erreur calloc\n");
				exit(1);
			}
			strcpy(tabMot[k],chaine);
			k++;
			printf("%s\n",chaine);
			printf("%s\n",tabMot[k]); //segfault
		}
	}
}

Ici, je pense que si je me suis pas planté je devrais lire les lignes du fichiers en entier.
L'affichage me montre que chaine prend bien la valeur de ma première ligne du fichier.

Je veux alors stocké cette chaîne dans un tableau de chaines que j'aurais alloué au préalable. Mais je prend quand même une erreur de segmentation sur l'affichage de tabMot[k]; sous entendu, la copie avec strcpy n'a pas été effectué correctement ! Je n'arrive pas à régler ce problème.

[ymerej5312]

Ton probleme de segmentation vient surement de ta chaine oú tu copie qui doit etre trop courte
Pour lire ton fichier ligne par ligne , lit le jusqu'à un \n grace a un fgetc et une boucle par exemple (fait attention a la longueur de ta chaine)

mot par mot grace au meme principe et caractere par caractere grace a fgetc
le principe maintenant :

Code :

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

1
2
3
4
5
6
7
8
int nb_caractere_lu = 0, taille_chaine = 100, caractere = 0 ;/* c'est un exemple, la taille peut etr plus grande */
char chaine[taille_chaine] = {0};
while( nb_caractere_lu < 100 && caractere != EOF)
{
caractere = fgetc(ton_fichier);
chaine[nb_caractere_lu] = caractere;
nb_caractere_lu++;
}

Voila le principe de base, j'espere qu'il t'aidera
attention ne copie colle pas ça betement j'ai surement du faire plein de faute de typo (sur un mini telephone c'est dur !)

[Amnael]

Pourtant je vois mal comment le soucis peut venir de là.

fgets prend la chaine en lui donnant une taille maximum.
Et la chaîne de copie à la même taille max en paramètre (j'ai tenté d'augmenté cette taille de 1 pour être sur tout de même) mais l'erreur de segmentation reste présente.

EDIT; bon alors après vérification il s'agit bien de la taille; j'ai affiché la taille de chaine qui est de 8 "Bonjour\0"; mais la taille de mon tableau à l'indice k est de 0.
Je ne comprends pas pourquoi, je fais bien un calloc au préalable et il ne plante pas puisque je ne rentre pas dans le if !!!
J'avoue ne pas comprendre !

La ligne tabMot[k]=calloc(TAILLE_MAX+1,sizeof(char)) devrait me retourner une taille positive hors la taille est de 0


Le comportement est en fait assez étrange.

Après le calloc/malloc la longueur du tableau est de 0 (Alors que ça devrait être 51 sachant que TAILLE_MAX est un define sur 50).
Après le strcpy la longueur du tableau est bel et bien la même que celle de chaîne; sous entendu la copie c'est bien passé puisque même longueur; pourtant l'erreur de segmentation persiste !

EDIT 3

J'ai résolu si je puis dire l'erreur dans le sens ou je vois ce qui l'a provoque; mais je n'ai aucune idée de la raison de ce "bug"

Mon affectation est bonne en réalité et l'affichage se passe bien. SI et seulement si je ne fais pas d'affichage avant.

C'est à dire

Code :

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

1
2
3
4
5
strcpy(tabMot[k],chaine);
			k++;
			printf("%s\n",chaine);
			printf("test\n");
			printf("%s\n",tabMot[k]);

n'affichera que chaine puis prend une core dumped sur tabMot

Mais si je prend la même structure en sautant les affichages précédent; plus rien et le prog s'exécute normalement !

Code :

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

1
2
3
strcpy(tabMot[k],chaine);
			k++;
			printf("%s\n",tabMot[k]);

Donc voilà; ceci m'affiche bien les chaines copiés.


Mais si quelqun est en mesure de m'expliquer pourquoi le fait de faire un affichage au préalable me provoque une erreur ça serait cool =)

[Bktero]

Code :

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

1
2
3
4
5
6
7
8
9
10
            tabMot[k]=calloc(TAILLE_MAX,sizeof(char));
            if (tabMot[k]==NULL)
            {
                fprintf(stderr, "Erreur calloc\n");
                exit(1);
            }
            strcpy(tabMot[k],chaine);
            k++; // !!!!!!!
            printf("%s\n",chaine);
            printf("%s\n",tabMot[k]); //segfault

Tu alloues à l'indice k, tu copies, tu incrémentes k et tu essayes d'accéder à un pointeur non alloué…

D'ailleurs chez moi, voilà ce qui se passe quand j'appelle ta fonction :

$ gccw test.c && ./a.out test.c: In function 'main':
test.c:35:14: warning: 'fichier' is used uninitialized in this function [-Wuninitialized]
  frequenceMot(fichier);
              ^
bonjour


(null)
developpez


(null)
.com


(null)

$ cat rep.txt bonjour
developpez
.com

Je ne sais pas ce que ca affiche chez toi, mais tu devais voir quelque chose de louche, non ?
Quand tu veux afficher des chaines comme ça, il ne faut pas hésiter à mettre des marques plus clairs. Exemple :

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
void frequenceMot(FILE *f){
	f=fopen("rep.txt","r");
	char chaine[TAILLE_MAX]="";
	char* tabMot[TAILLE_MAX]={""};
	if(f != NULL)
	{
		int k=0;
		while (fgets(chaine, TAILLE_MAX, f) != NULL) // On lit le fichier tant qu'on ne reçoit pas d'erreur (NULL)
		{
			tabMot[k]=calloc(TAILLE_MAX,sizeof(char));
			if (tabMot[k]==NULL)
			{
				fprintf(stderr, "Erreur calloc\n");
				exit(1);
			}
			strcpy(tabMot[k],chaine);
			printf("Lu {%s}\nSauve [%s]\n\n",chaine, tabMot[k]); // nouvelle façon
			k++; // correction
		}
	}
}

Ce qui donne :

gccw test.c && ./a.out test.c: In function 'main':
test.c:34:14: warning: 'fichier' is used uninitialized in this function [-Wuninitialized]
  frequenceMot(fichier);
              ^
Lu {bonjour
}
Sauve [bonjour
]


Lu {developpez
}
Sauve [developpez
]


Lu {.com
}
Sauve [.com
]