Problème realloc char**

**orj30754** · 03/11/2009, 14h02

Bonjour,

je suis entrain de faire un petit programme permettant de lire dans un fichier et de récupérer les caractères mot à mot.

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
)
int main()
{
	FILE* monFichier = NULL;
	char *temp;
	char** tab;
	int i=0;
	int j = 0;
	int taille = 0;
 
	tab = (char**)malloc(sizeof(char*));
	tab[0] = (char*)malloc(sizeof(char));
 
	monFichier = fopen("C:\\Users\\Julien\\Desktop\\test.txt","r");
 
	if (monFichier != NULL)
    {
		while(!feof(monFichier))
		{
			temp = LireMot(monFichier);
			taille = strlen(temp);
			tab[i] = (char*)realloc(tab, taille*sizeof(char));
			//tab[i] = temp;
 
			i++;
			tab = (char**)realloc(tab, i*sizeof(char*));
		}
        // On peut lire et écrire dans le fichier
    }
    else
		printf("Impossible d'ouvrir le fichier");
 
}

J'essai de faire ca de facon dynamique, et lorsque je récupère un nouveau mot dans la variable temp et que je souhaite réallouer mon tableau, je perds ce qui a dans temp

J'attends votre aide avec impatience. Merci

**Beniou** · 03/11/2009, 14h34

Bonjour,

Premièrement, on ne sait pas comment ta fonction LireMot fonctionne, ce qu'elle retourne etc.. Tu pourrais allouer ton temp directement en faisant plutôt lireMot(temp) ou lireMot(&temp) et allouer l'espace pour temp dans ta fonction.

Ensuite pour copier deux chaînes il vaut mieux utiliser la fonction strcpy en allouant l'espace nécessaire avant comme tu fais (et pas besoin de realloc juste un malloc ou un calloc suffit sur ton tab[i]). Avec le +1 à la fin pour le caractère '\0'
ex:

Code :

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

1
2
3
4
 
taille = strlen(temp);
tab[i] = (char*)realloc(tab, (taille*sizeof(char)) + 1);
strcpy(tab[i],temp);

Enfin je ferai une reallocation de tab, une incrémentation de i avant de faire la copie de tab[i] : c'est plus propre et tu ne perds pas d'espace.

Bon ceci est basé sur mes vieux souvenirs de C. Je me trompe peut être ou bien je n'ai pas saisi ton problème (et je n'ai pas testé non plus

)mais j'espère que cela t'aideras.

**orj30754** · 03/11/2009, 14h36

Ok merci pour tes réponses, je vais travailler dans ton sens.

Pour la fonction LireMot, je ne peux pas la changer elle doit rester comme elle est déclarée.

Voici son code.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
char *LireMot(FILE *_monFichier)
{
	char* _buffer;
	char _temp[50];
 
	fscanf(_monFichier, "%s",_temp);
	_buffer = (char*)malloc(strlen(_temp) * sizeof(char));
	_buffer = _temp;
 
	return _buffer;
}

**Pouet_forever** · 03/11/2009, 14h48

Les cast sur les malloc sont inutiles ...
Je ne comprend pas pourquoi tu crées ça :

Code :

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

1
2
tab = (char**)malloc(sizeof(char*));
tab[0] = (char*)malloc(sizeof(char));

Là tu crées un 'tableau 2D' de 1 pointeur sur 1 char

En gros c'est comme si tu faisais : char tab[1][1].
Et donc tu ne peux pas récupérer ce que retourne ta fonction correctement

A ce niveau là c'est incorrect :

Code :

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

_buffer = _temp;

Tu alloues de la mémoire dans _buffer et quand tu fais la ligne du dessus, tu fais pointer _buffer sur un autre endroit ...
De plus comme la variable _temp est une variable locale quand tu retournera _buffer hé bien ce ne sera pas correct

Il y a sûrement d'autres erreurs ...

**orj30754** · 03/11/2009, 14h54

Comme il y a plusieurs mots de plusieurs caractères dans mon fichier, il me faut bien un tableau 2D.

Ensuite pour les mallocs, si je ne les cast pas de la sorte, j'ai une erreur lorsque je compile.

Pour ce qui est du retour de buffer, je suis d'accord avec toi, a mon avis il doit y avoir la perte d'info ici.

J'ai fait les corrections soumises plus hautes

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
int main()
{
	FILE* monFichier = NULL;
	char *temp;
	char** tab;
	int i=0;
	int j = 0;
	int taille = 0;
 
	tab = (char**)malloc(sizeof(char*));
 
	monFichier = fopen("C:\\Users\\Julien\\Desktop\\test.txt","r");
 
	if (monFichier != NULL)
    {
		while(!feof(monFichier))
		{
			temp = LireMot(monFichier);
			taille = strlen(temp);
			tab[i] = (char*)malloc(taille*sizeof(char));
			strcpy(tab[i],temp);
 
			i++;
			tab = (char**)realloc(tab, i*sizeof(char*));
		}
    }
    else
		printf("Impossible d'ouvrir le fichier");
 
}