probleme avec fgets et un tableau à 2 dimensions

**l1086** · 12/12/2007, 21h48

Bonsoir,

J'ai un tableau alloué dynamiquement, le problème ce pose lorsque je veux le remplir.Je souhaite le remplir à partir d'un fichier.
La ligne incriminé est la suivante :

Code :

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

while((fgets(tab[i++],50,fp)) != NULL);

tab est le tableau à 2 dimensions allouer précédemment
fp le pointeur de lecture de fichier

et voici le log d'erreur que j'ai par valgrind:

==12058== Invalid write of size 1
==12058== at 0x40243EA: memcpy (mc_replace_strmem.c:406)
==12058== by 0x409506C: _IO_getline_info (in /lib/tls/i686/cmov/libc-2.6.1.so)
==12058== by 0x4094F50: _IO_getline (in /lib/tls/i686/cmov/libc-2.6.1.so)
==12058== by 0x4093EC9: fgets (in /lib/tls/i686/cmov/libc-2.6.1.so)
==12058== by 0x8048C28: main (lecture.c:280)
==12058== Address 0x0 is not stack'd, malloc'd or (recently) free'd
==12058==
==12058== Process terminating with default action of signal 11 (SIGSEGV)
==12058== Access not within mapped region at address 0x0
==12058== at 0x40243EA: memcpy (mc_replace_strmem.c:406)
==12058== by 0x409506C: _IO_getline_info (in /lib/tls/i686/cmov/libc-2.6.1.so)
==12058== by 0x4094F50: _IO_getline (in /lib/tls/i686/cmov/libc-2.6.1.so)
==12058== by 0x4093EC9: fgets (in /lib/tls/i686/cmov/libc-2.6.1.so)
==12058== by 0x8048C28: main (lecture.c:280)

Merci de votre aide

**vinzzzz** · 12/12/2007, 21h58

Je pense qu'il faut que tu poste plus de code...

Comment tu as alloué tes variables? est-ce que ton i corresponds à la dimension de ton tableau? Est-ce que la taille de chaque chaine utilisée pour lire les lignes du fichier est suffisante?

Bref poste ta fonction

**l1086** · 12/12/2007, 22h13

Voici donc la fonction incréminée

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
int main()
{
	FILE *lire=NULL;
	lire=fopen("texte.txt","r");
	if(lire == NULL)
	{
		printf("fichier inexistant\n");
		return 1;
	}
 
	char **tab;
	int ligne=57;
	int colonne=167570;
	tab=createtab(ligne,colonne);
 
	int i=0;
	/*insertion*/
	while(fgets(tab[i++],50,lire) !=NULL);
	/*fermer le dico*/
	fclose(lire);
        return 0;
}

**Sve@r** · 12/12/2007, 22h45

Envoyé par l1086

Voici donc la fonction incréminée

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
int main()
{
	FILE *lire=NULL;
	lire=fopen("texte.txt","r");
	if(lire == NULL)
	{
		printf("fichier inexistant\n");
		return 1;
	}
 
	char **tab;
	int ligne=57;
	int colonne=167570;
	tab=createtab(ligne,colonne);
 
	int i=0;
	/*insertion*/
	while(fgets(tab[i++],50,lire) !=NULL);
	/*fermer le dico*/
	fclose(lire);
        return 0;
}

Hum... et le contenu de "createtab" ???

Il manque la libération de "tab". par ailleurs tu ne contrôles absolument pas "i" par rapport à l'allocation de "tab". Maintenant si chaque ligne ne contient que 50 caractères, tu aurais tout avantage à déclarer "char (*tab)[50 + 1]" comme ça tu gagnes une dimension au malloc et au free...

**l1086** · 12/12/2007, 22h55

au bah la fonction createtab c'est ceci
Désolé de l'avoir oublié

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
char **createtab(int sizeI,int sizeJ)
{
        int i;
	char **tab; // tab est un pointeur de pointeur
	tab = (char**)malloc(sizeof(char*)*sizeI);
	if(tab == NULL)
		return NULL;
 
	for(i=0;i<sizeI;i++)
	{
		tab[i] = (char*)malloc(sizeof(char)* sizeJ);
		if(tab[i] == NULL)
			return NULL;
	}
	return(tab);
}

**souviron34** · 13/12/2007, 03h33

ben lis le diagnostic du compilateur...

Il te dit tout...

(enfin pas tout, mais si tu règles ça tu verras mieux)

**Sve@r** · 13/12/2007, 07h55

Envoyé par souviron34

ben lis le diagnostic du compilateur...

Il te dit tout...

(enfin pas tout, mais si tu règles ça tu verras mieux)

Zut !!! Je dois vraiment avoir les yeux remplis de pisse parce que je vois pas ce qui choque. Il va me faloir tester ce code pour comprendre mais ça m'énerve de ne pas le voir directement.

A la limite, dans un soucis de rigueur, je rajouterais ceci à createtab

Code :

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

1
2
3
4
5
6
7
if(tab[i] == NULL)
{
     for (int j=0; j < i; j++)
          free(tab[j]);
     free(tab);
     return NULL;
}

Mais c'est pas ça qui fait que ça plante au fgets()...

**corentin59** · 13/12/2007, 10h07

Envoyé par l1086

Voici donc la fonction incréminée

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
int main()
{
	FILE *lire=NULL;
	lire=fopen("texte.txt","r");
	if(lire == NULL)
	{
		printf("fichier inexistant\n");
		return 1;
	}
 
	char **tab;
	int ligne=57;
	int colonne=167570;
	tab=createtab(ligne,colonne);
 
	int i=0;
	/*insertion*/
	while(fgets(tab[i++],50,lire) !=NULL);
	/*fermer le dico*/
	fclose(lire);
        return 0;
}

Ce n'est certainement pas ça le pb mais pourquoi faire des lignes de 167570 éléments (ce qui fait que tab pointe vers un espace mémoire de 9Mo) alors qu'on ne s'autorise qu'a mettre 50 éléments par ligne ?

Il y a peut-être une inversion entre "ligne" et "colonne" ?

**Melem** · 13/12/2007, 12h25

Envoyé par Sve@r

Zut !!! Je dois vraiment avoir les yeux remplis de pisse parce que je vois pas ce qui choque. Il va me faloir tester ce code pour comprendre mais ça m'énerve de ne pas le voir directement.

A la limite, dans un soucis de rigueur, je rajouterais ceci à createtab

Code :

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

1
2
3
4
5
6
7
if(tab[i] == NULL)
{
     for (int j=0; j < i; j++)
          free(tab[j]);
     free(tab);
     return NULL;
}

Mais c'est pas ça qui fait que ça plante au fgets()...

Et après

Code :

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

while(fgets(tab[i++],50,lire) !=NULL);

aussi.

Envoyé par l1086

J'ai un tableau alloué dynamiquement, le problème ce pose lorsque je veux le remplir.Je souhaite le remplir à partir d'un fichier.

Es-tu sûr que chaque ligne de ton fichier fait 48 caractères tout au plus pour justifier le 50 dans fgets ?

**l1086** · 13/12/2007, 12h57

Alors melem en fait c'est un dico qui contient 167569 mot

Puis pour le 50 je me suis rabattu sur le faite que le mot le plus long de la langue française c'est anticonstitutionnellement

**Melem** · 13/12/2007, 13h27

Alors melem en fait c'est un dico qui contient 167569 mot

Dans ce cas tu dois avoir un tableau de 167569 mots, sachant qu'un mot = 1 char *, comme te l'a dit corentin59

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
32
33
34
 
char ** CreateTab(size_t NbMots, size_t MaxMot)
{
    char ** pTab;
    int success = 1; /* 1 : TRUE */
    size_t i;
 
    pTab = malloc(NbMots * sizeof(char *));
    if (pTab != NULL)
    {
        for(i = 0; i < NbMots; i++)
        {
            pTab[i] = malloc(MaxMot);
            if (pTab[i] == NULL)
            {
                size_t j;
 
                for(j = 0; j < i; j++)
                    free(pTab[j]);
 
                success = 0; /* 0 : FALSE */
                break;
            }
        }
    }
 
    if (success)
        return pTab;
    else
    {
        free(pTab);
        return NULL;
    }
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
int main()
{
    char ** pTab;
 
    ...
 
    pTab = CreateTab(167569, 50);
 
   ...
}

**l1086** · 13/12/2007, 14h09

Merci beaucoup à tous cela marche quasiment parfaitement

A part une erreur de lecture au début de mon insertion

Me reste qu'à trouver d'où vient l'erreur et de la corriger

**Melem** · 13/12/2007, 14h40

Et si tu peux, poste ton code lorsque tu auras terminé s'il te plait

.

**l1086** · 13/12/2007, 17h37

en fait y'a juste une ptite astuce dans le while du 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
24
 
	FILE *lire;
 
	lire=fopen("dico.txt","r");
	if(lire == NULL)
	{
		printf("fichier inexistant\n");
		return 1;
	}
 
	char **tab;
	tab=dico(167569, 29);
 
	int i=0;
 
	while((fgets(tab[i++],29,lire) != NULL) && i<167569);
	fclose(lire);
	printf("dico prêt \n");
 
	//print(tab,167569);
 
	desalloctab(tab);     
	return 0;
}

**Melem** · 14/12/2007, 07h42

Code :

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

desalloctab(tab);

Ah, c'est surtout ce que je voulais voir

**l1086** · 14/12/2007, 19h14

Bonsoir,

voici la fonction desalloc alors

Code :

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

1
2
3
4
5
6
7
8
void desalloctab(char **tab)
{
	int temp=0;
 
	for(temp=0; temp < 167569; temp++)
          		free(tab[temp]);
     	free(tab);
}

Sinon je reviens vers vous pour un autre problème que je ne comprend pas.
Je veux enlever les \n que m'attrape fgets, voici le code :

Code :

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

1
2
3
4
5
6
7
8
9
10
while((fgets(tab[i++],29,lire) != NULL) && i<167569)
	{
 
         /*on recherche le caractere de fin de ligne dans le tableau */
        	char *pc = strchr(*tab, '\n');
    		if (pc != NULL)
       		{	
       		 	*pc=0;
        	}
        }

Mais a ma grande surprise aucun \n n'est enlevé.
Pourtant j'ai teste cette fonction il rentre bien dans le if

**diogene** · 14/12/2007, 19h43

Code :

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

1
2
3
4
5
6
7
8
while( i<167569 &&(fgets(tab[i],29,lire) != NULL) )
{
 
   /*on recherche le caractere de fin de ligne dans le tableau */
   char *pc = strchr(tab[I], '\n');
   if (pc != NULL)*pc=0;
   i++;
}

probleme avec fgets et un tableau à 2 dimensions

Bibliothèque standard C

Discussions similaires

Partager

Partager