Tableau de chaînes de caractères

[EDIT] fork de cette discussion[/EDIT]

Bonjour.

Je préfère continuer ici que de créer mon propre topic, les sujets étant similaires.
Je souhaite moi aussi lister les fichiers d'un répertoire, puis leur appliquer un traitement particulier.

Mon idée d'organisation est la suivante:

Code:

---

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; traitement(); i++ }

---

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


2 - J'ai un problème technique dans mon code qui est le suivant:

- 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... HELP !!!

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 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; }

---

Je tourne en rond, je ne comprend pas pourquoi mes variables qui ont à l'intstant T1 le bon nom de fichier prennent en T2 le nom du dernier fichier du répertoire lu...

Toutes tes lignes de tableau sont des pointeurs sur chaîne qui prennent la valeur de lecture->d_name. Je soupçonne que le struct est simplement réutilisé et mis à jour quand tu pointes sur le fichier suivant...ce qui fait que toutes tes lignes pointent sur la même chaine de caractère...qui ne peut pas avoir plusieurs valeurs à la fois. C'est pour ça que tu te retrouves avec n fois le même nom de fichier.

Pourquoi n'utilises-tu pas un vector de std::string? Il est possible de recopier une chaine "C-style" dans une string, là c'est une copie et tu n'auras pas de prise de tête.

Ce que je ne comprend pas c'est que quand je fais

Code:

---

tab_list_files[i] = lecture->d_name;

---

tab_list_files[i] prend la valeur de lecture->d_name à l'instant T.

Même si lecture->d_name change par la suite, je ne comprend pas à quel moment tab_list_files[i] est impacté.


Je préférerais réussir à résoudre mon problème sans passer par un vector std::string, j'ai énormément de mal avec le C++... en fait j'ai posté ici avant tout parce que le sujet était le même que celui que j'allais ouvrir dans la partie C.

Citation:

---

tab_list_files[i] prend la valeur de lecture->d_name à l'instant T.

Même si lecture->d_name change par la suite, je ne comprend pas à quel moment tab_list_files[i] est impacté.

---

Parce qu'il s'agit d'un pointeur. Le contenu n'est pas dans tab_list_files[i] mais dans lecture->d_name, que le tableau se contente de pointer. Donc toutes les chaines ont la même valeur, celle de lecture->d_name. D'ailleurs le tableau peut donner lieu à des bugs mémoire (segfault, changements incompréhensibles) s'il est utilisé après libération du struct lecture.

Bonjour,
En C++, les chaînes de caractères, c'est std::string, les tableaux, c'est std::vector et les E/S c'est les flux std::cout.

Code:

---

1 2 3 4 5 6 7 8 9

std::vector<std::string> tab_list_files;   //je fais une boucle qui va traiter tous les fichiers for(std::vector<std::string>::iterator it = tab_list_files.begin();it!=tab_list_files.end();++it) { std::string current_file = *it; traitement(); }

---

etc...

Ok, la solution en C++ est de repartir avec une autre nature de variable.

Merci.