passer un char** en argument

Version imprimable

25/03/2006, 16h39
ben_popcorn

passer un char** en argument
bonjour,

La fonction suivante a pour but de rechercher certains fichiers dans un répertoire et les rendre dans un tableau à 2 dimensions

Code:

long ExploreRep(char *, char **);

Pour l'appel de la fonction je procéde ainsi:
Code:

1 2 TmpNb = ExploreRep(l_Repertoire_Principal,l_ListeFichiers);
où TmpNB reçoit le nombre de fichier trouvés dans le répertoire.

Mais l_ListeFichiers reste vide en sortie de ma fonction alors qu'il est bien remplie dans la fonction ....
je sens que je ne suis pas loin.....mais j'y arrive pu ..... help ?

merci les gens
25/03/2006, 17h45
Biosox

sans le code de ExploreRep, difficile a répondre.

A vue de nez, le problème pourrait etre le suivant:

Dans ta fonction tu crée une variable vers laquelle pointe l_ListeFichiers. Ensuite dans ta fonction toujours tu remplis l_ListeFichiers. Mais à la fin de ta fonction, la variable crée est détruite, et donc l_ListeFichiers pointe vers n'importe quoi.

c'est une erreur fréquente et:

Citation:

Mais l_ListeFichiers reste vide en sortie de ma fonction alors qu'il est bien remplie dans la fonction ....

me fait penser que ça pourrait être à ça.

Mais sans ton code, difficile d'être sur...

oups, voici le code de la fonction:

Code:

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
 
long ExploreRep(char * p_Chemin,
                char ** p_ListeFichiers)
{
	long NbFiles, cr;
	long l_indiceFichier;
	NbFiles = 0;
	p_ListeFichiers = malloc(sizeof(char*)*CST_NB_Fichiers_A_Lire25);
 
	DIR* rep;
	struct dirent *read;
 
	//rep = opendir(p_Chemin);
	if ((rep = opendir(p_Chemin)) == NULL)
    {
            perror("opendir");
            return 0;
    }
 
	//on passe les repertoire . et ..
	read = readdir(rep);
	read = readdir(rep);
	while(read = readdir(rep))
	{
		NbFiles++;
		l_indiceFichier=NbFiles-1;
		p_ListeFichiers[l_indiceFichier]=malloc(sizeof(char*)*255);
		strcpy(p_ListeFichiers[l_indiceFichier],read->d_name);
		printf("listefic[%d]=%s\n",l_indiceFichier, p_ListeFichiers[l_indiceFichier]);
	}
 
   return NbFiles;
}

p_ListeFichiers est une variable locale à la fonction et le paramètre d'appel correspondant ne peut être modifié. On devrait avoir
Code:

1 2 3 4 5 6 7 8 9 10 11 long ExploreRep(char * p_Chemin, char *** p_ListeFichiers) { long NbFiles, cr; long l_indiceFichier; NbFiles = 0; *p_ListeFichiers = malloc(sizeof(char*)*CST_NB_Fichiers_A_Lire25); ..... (*p_ListeFichiers)[l_indiceFichier]=malloc(sizeof(char)*255); .....
A noter que le dernier malloc n'est pas sizeof(char*)*255), mais 255

la suite des festivités:

Code:

1
2
3
4
5
6
7
8
9
10
 
char* l_Repertoire_Principal;
char** l_ListeFichiers;
 
...
 
TmpNb = ExploreRep(l_Repertoire_Principal,&l_ListeFichiers);
   //boucle pour controle en DEBUG
   for(i=0;i<TmpNb;i++)
	printf("ficname sortie=%s\n",&l_ListeFichiers[i]);

Code:

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
 
long ExploreRep(char * p_Chemin,
                char *** p_ListeFichiers)
{
	long NbFiles, cr;
	long l_indiceFichier;
	NbFiles = 0;
	*p_ListeFichiers = malloc(sizeof(char*)*CST_NB_Fichiers_A_Lire25);
	int i;
	DIR* rep;
	struct dirent *read;
 
	if ((rep = opendir(p_Chemin)) == NULL)
    {
            perror("opendir");
            return 0;
    }
 
...
	while(read = readdir(rep))
	{
		NbFiles++;
		l_indiceFichier=NbFiles-1;
		p_ListeFichiers[l_indiceFichier]=malloc(255);
		strcpy(p_ListeFichiers[l_indiceFichier],read->d_name);
	}
 
//boucle pour controle en DEBUG
for(i=0;i<NbFiles;i++)
		printf("ficname=%s\n",p_ListeFichiers[i]);
 
   return NbFiles;
}

en sortie:

avant de quitter la fonction:

Code:

1
2
3
4
ficname=Copie (2) de toto.txt
ficname=Copie (3) de toto.txt
ficname=Copie de toto.txt
ficname=toto.txt

après la fonction:

Code:

1
2
3
4
ficname sortie=Copie (2) de toto.txt
ficname sortie=e (2) de toto.txt
ficname sortie=) de toto.txt
ficname sortie= toto.txt

crénom ........ any idea ?

25/03/2006, 20h26
loufoque

T'es au courant qu'il faut la libérer la mémoire après j'espère ?
25/03/2006, 20h35
ben_popcorn
Citation:

Envoyé par loufoque

T'es au courant qu'il faut la libérer la mémoire après j'espère ?
Code:

1 2 3 4 5 6 for(i=0;i<NbFiles;i++) free(p_ListeFichiers[i]); free(p_ListeFichiers); free(p_Chemin);
j'ai bon ? Mais bon ça intervient bien après l'appel et le retour de ExploreRep vu que j'ai besoin de l_ListeFichiers et donc de son homologue local dans ExploreRep p_ListeFichiers
26/03/2006, 14h58
ben_popcorn

pas d'idées ......... :cry:
26/03/2006, 16h46
Pragmateek

Citation:

Code:

p_ListeFichiers = malloc(sizeof(char*)*CST_NB_Fichiers_A_Lire25);

Je suis pas sur mais cela crée un objet local à la fonction qui est forcément détruit à sa sortie.

En fait cet objet court-circuite p_ListeFichiers passé en parametre.

Le malloc doit se faire avant l'appel de la fonction.
26/03/2006, 17h03
Trap D
Attention :
Code:

1 2 3 4 5 char *foo(void) { char *tmp = malloc(10); return tmp; }
tmp est une variable locale à la fonction foo.
Sauf que l'on retourne sa valeur, qui est une adresse valide sur le tas. Donc tmp est bien détruite au sortir de la fonction mais sa valeur reste elle valide.
Par contre :
Code:

1 2 3 4 5 6 char *bad_fonc(void) { char tmp[256]; .......... return tmp; }
Ici la valeur retournée n'est plus valide au sortir de la fonction, puisque tmp est une variable locale à cette fonction (son adresse est une adresse de la pile d'exécution) , donc le contenu peut-être détruit (ou non d'ailleurs) dans la suite de l'exécution du programme.

Code:

1
2
3
4
TmpNb = ExploreRep(l_Repertoire_Principal,&l_ListeFichiers); 
   //boucle pour controle en DEBUG 
   for(i=0;i<TmpNb;i++) 
   printf("ficname sortie=%s\n",&l_ListeFichiers[i]);

non :

Citation:

printf("ficname sortie=%s\n"l_ListeFichiers[i]);

Code:

1
2
3
4
5
6
7
 while(read = readdir(rep)) 
   { 
      NbFiles++; 
      l_indiceFichier=NbFiles-1; 
      p_ListeFichiers[l_indiceFichier]=malloc(255); 
      strcpy(p_ListeFichiers[l_indiceFichier],read->d_name); 
   }

Non :

Code:

1
2
3
4
5
6
7
while(read = readdir(rep)) 
   { 
      NbFiles++; 
      l_indiceFichier=NbFiles-1; 
      (*p_ListeFichiers)[l_indiceFichier]=malloc(255); 
      strcpy((*p_ListeFichiers)[l_indiceFichier],read->d_name); 
   }

26/03/2006, 21h57
Luther13

J'ai pas trop lu les conversations precedentes, mais ton problème reside effectivement sur le malloc que tu effectue à l'interieur.
Essaye le prototype suivant:

Code:

char ** ExploreRep(char *, char ** );

Tu pourras recuperer ainsi la bonne adresse