Discussion :

[ickis69]

Bonjour.

Je souhaite lister les fichiers d'un répertoire dans un tableau pour ensuite leur appliquer 1 à 1 un traitement particulier.

Mon idée d'organisation est la suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
// je liste les noms des fichiers à traiter dans un tableau 
char *tab_list_files[256];
 
//je fais une boucle qui va traiter tous les fichiers
While(tab_list_files[i]!=NULL)
{
    current_file = tab_list_files[i];
    traitement();
    i++
}

1 - Est ce que c'est une bonne organisation?


2 - J'ai un problème technique dans mon code à l'étape qui consiste à lister les fichiers à traiter et les stocker dans un tableau de char:

- je liste les fichiers d'un répertoire
- si le nom de fichier commence par "debut", je l'affiche et je le stock dans mon tableau
-> OK
- mais une fois le listing terminé, quand je veux réutiliser les noms de mes fichiers séléctionnés, ils ont tous pris le nom du dernier fichier lu dans le répertoire...

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
35
36
 
char *tab_list_files[256];
int i;
 
int main()
{
 
    struct dirent *lecture;
    DIR *rep;
    rep = opendir("." );
 
    i=0;
 
    while ((lecture = readdir(rep))) {                  // on lit tous les fichiers du répertoire local
 
        if(!strncmp(lecture->d_name,"debut",5))       // si les 5 premières lettres du nom de fichier = "début"
        {
            tab_list_files[i] = lecture->d_name;        // alors on enregistre ce nom de fichier dans le tableau tab_list_files
            printf("%s\n", tab_list_files[i]);          // T1 au passage on l'affiche
            i++;                                        
        }
 
    }
    closedir(rep);
 
    i=0;
    while(tab_list_files[i]!=NULL)                      // on va afficher toutes les noms de fichiers stoquées qui ne sont pas vides
    {
        printf("\n+1 %s\n",tab_list_files[i]);           // T2 on réaffiche les noms des fichiers commençant par "début" -> cela ne marche pas
        i++;
    }
 
 
    getch();
    return 1;
}

Ce code ne fonctionne pas car tab_list_files[i] = lecture->d_name; ne copie pas la valeur de lecture->d_name dans tab_list_files[i], il ne fait que pointer, du coup les tab_list_files[i] affichent tous la dernière valeur prise par lecture->d_name : le nom du dernier fichier du répertoire lu.

Donc j'ai pensé à utiliser strcpy(tab_list_files[i],lecture->d_name);

Compilation ok mais crash du programme à l’exécution.

Comment que je dois faire pour arranger ça?

Merci.


PS : pour la lecture des fichiers du répertoire j'ai utilisé un code que j'ai trouvé tout fait sur le net... si vous avez une toute autre prosposition, tant que c'est du C, je prends

[Bousk]

Bonsoir,

pour utiliser strcpy la chaîne de destination doit être assez grande pour contenir la chaîne source.
Dans ton cas la chaîne destination est... complètement indéfini puisque non allouée.

char* tab[254] défini un tableau de 254 char* il me semble, et non un pointeur sur des char[254]

Peut-être qu'une struct intermédiaire assurerait de créer un pointeur sur des char[254]
struct myDirName { char str[254]; };en créant un struct myDirName* myTabOfNames; pour avoir un tableau, toujours à initialiser.

Dans tous les cas, container non initialisé ou contenu non initialisé, ça manque d'initialisation.

[Kirilenko]

1 - ok, le tableau alloué statiquement n'est pas très esthétique, mai si tu essuie que tu n'as pas plus de 256 fichiers. :p

2 - En fait char*[256] declare 256 pointeurs sur char sur des chaînes statiques. Quand tu essayes de copier le nom de ton fichier dans une case du tableau, mais tu accèdes à un emplacement à lecture seule. D'où une erreur de segmentation.
Pour remédier à cela tu peux utiliser un tableau à deux dimensions char tab[256][MAXLEN] où MAXLEN est une taille représentative de la taille maximale d'un nom de fichier. (il me semble qu'il y a une variable posix qu'il l'indique, à voir).

[edgarjacobs]

Hello,

Ce que je ferais das ton cas, c'est conserver le tout tel quel, et pour mémoriser le nom du ficher il suffit de faire

Code :

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

1
2
tab_list_files[i]=malloc(strlen(lecture->d_name)+1);
strcpy(tab_list_file[i],lecture->d_name);

et le tour est joué.

Deux choses encore à faire: vérifier que malloc() ne renvoie pas NULL, et ne pas oublier de faire une boucle i* pour libérer tab_list_file[i] quand tu n'en as plus besoin...

Cordialement,

Edgar.

[anacharsis]

salut !
avec la taille du cache des disques durs actuels, il me faut surtout pas hésiter à accéder plusieurs fois à des données dont on est certain de la présence dans ce cache...

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
35
36
37
38
39
40
41
42
#include <stdio.h>
#include "dirent.h"
 
char ** dirtab(const char * s){
		int i, j;
		DIR * dir;
		struct dirent * e;
		i = 0;//compte les noms
		j = 0;//compte la somme des tailles des moms
		dir = opendir(s);
		while ((e = readdir(dir))){
			if ((strcmp(e -> d_name, ".") && strcmp(e -> d_name, ".."))){//on n'a pas besoin de . et .. 
				j += strlen(e -> d_name) + 1;// + 1 pour le 0 terminal
				i++;
			}
		}
 
		char ** p = (char **) malloc((i + 1) * sizeof(char *));//on alloue la place pour le tableau + 1 pour NULL
		char * r = (char *) malloc(j);//on alloue la place pour toutes les chaînes en tas
 
		rewinddir(dir);//on recommence la lecture
		i = 0;
		while ((e = readdir(dir))){
			if (strcmp(e -> d_name, ".") && strcmp(e -> d_name, "..")){
				p[i++] = strcpy(r, e -> d_name);//on copie les caractères du nom dans le tas, et son pointeur dans le tableau
				r += strlen(r) + 1;//mise à jour du pointeur de caractères du tas
			}
		}
		p[i] = NULL;
		closedir(dir);	
		return p;
}
 
void main (void)
{ 
		int i;
		char ** t = dirtab(".");
		for (i = 0 ; t[i] != NULL; i++)
			printf("%s\n", t[i]);
		free(*t[0]);//libère le tas de caractères
		free(t);//libère le tableau
}

A+

[ickis69]

Hello,

Ce que je ferais das ton cas, c'est conserver le tout tel quel, et pour mémoriser le nom du ficher il suffit de faire

Code :

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

1
2
tab_list_files[i]=malloc(strlen(lecture->d_name)+1);
strcpy(tab_list_file[i],lecture->d_name);

et le tour est joué.

Deux choses encore à faire: vérifier que malloc() ne renvoie pas NULL, et ne pas oublier de faire une boucle i* pour libérer tab_list_file[i] quand tu n'en as plus besoin...

Cordialement,

Edgar.

Bonjour.
C'est la solution la plus simple, donc la première que j'ai essayé.
Mais j'ai une erreur à la compilation :

53 invalid conversion from `void*' to `char'

J'ai toujours du mal à comprendre!

[Kirilenko]

Tu ne compilerais pas en C++ par hasard ?