Discussion :

[YoshioToutou]

Bonjour, je travaille actuellement sur un projet de sauvegarde de fichier sous GNU/Linux, pour le moment, je souhaite crée un fichier qui sauvegarde le nom de tous les répertoire, sous répertoire et fichier ainsi que leur md5sum dans un fichier.
J'ai le code suivant

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
 
void dumpFile(string directory, ofstream& output){
   DIR *dir = NULL;
   struct dirent *file = NULL;
   if((dir = opendir(directory.c_str())) == NULL){
 
   }
   while((file = readdir(dir)) != NULL){
      if(strcmp(file->d_name, ".") && strcmp(file->d_name, "..")){
         if(file->d_type == DT_DIR){ // Folder
	   dumpFile(directory.append("/").append(file->d_name), output);
	 }else{
	    cout << directory << "/" << file->d_name << endl; // Just to see what is the problem
            output << file->d_name << endl << md5sum(directory, file->d_name) << endl; // Write the result on 2 lines (first the name, then de md5
	 }
      }
   }
 
   closedir(dir);
 
}

Donc, si le dossier ne possède que des fichier, ou que je supprime la ligne de récursivité (le if), cela fonctionne parfaitement, mais là, elle bloque et je ne comprend vraiment pas pourquoi.
Type d'erreur :
directory : /home/user/Bureau
Fichier : test.dump, test
Sous dossier : python
sous fichier : aucun

/home/user/Bureau/test.dump
/home/guillaume/Bureau/python/test
terminate called after throwing an instance of 'std::bad_alloc'
what(): std::bad_alloc
Abandon

Ce qui est normal, étant donnée que le fichier test n'existe pas dans le dossier python, mais est sur le bureau.

Merci pour votre aide.

[Ekleog]

Pourquoi ne pas utiliser boost.filesystem ?

[panthere noire]

Pourquoi ne pas utiliser boost.filesystem ?

parce que boost est pas une lib dans la stl et que donc réduis la portabilliter, c'est donc une autre sujet mai soumettre l'idée de la lib est toujours bonne

[JolyLoic]

parce que boost est pas une lib dans la stl et que donc réduis la portabilliter

Certes, boost n'est pas utilisable partout. Mais dirent ou readdir ne font pas partie non plus de la bibliothèque standard. Et donc en l'occurrence, utiliser boost.filesystem accroit probablement la portabilité par rapport à utiliser des fonctions spécifiques à un OS.

[liouan]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
   while((file = readdir(dir)) != NULL){
      if(strcmp(file->d_name, ".") && strcmp(file->d_name, "..")){
         if(file->d_type == DT_DIR){ // Folder
	   dumpFile(directory.append("/").append(file->d_name), output);
	 }else{
	    cout << directory << "/" << file->d_name << endl; // Just to see what is the problem
            output << file->d_name << endl << md5sum(directory, file->d_name) << endl; // Write the result on 2 lines (first the name, then de md5
	 }
      }
   }

Je pense que le scénario suivant est possible:

tu as un dossier D, dans lequel tu as:
- un dossier d1
- un fichier f

Au premier passage dans ta boucle, tu tombes sur d1, qui est un dossier donc tu rentres dans le if.
Dans le if, tu appends d1 à D. A mon avis, le souci se trouve là. Ca n'a pas juste concaténé les chaînes et renvoyé une copie du résultat. A mon avis, ça a modifié ta variable directory. Donc tu te retrouves avec directory qui vaut "D/d1"

du coup, après avoir traité le contenu de ton if, au deuxième passage dans ta boucle while, tu trouves le fichier f (normal) mais ta variable directory vaut alors D/d1.

Et donc tu cherches D/d1/f, qui n'existe pas.

Je te sugggère ce lien qui montre bien que string::append renvoie bien une ref et non une copie:

http://www.cplusplus.com/reference/s...string/append/

[panthere noire]

Bonjour, je travaille actuellement sur un projet de sauvegarde de fichier sous GNU/Linux, pour le moment, je souhaite crée un fichier qui sauvegarde le nom de tous les répertoire, sous répertoire et fichier ainsi que leur md5sum dans un fichier.
J'ai le code suivant

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
 
void dumpFile(string directory, ofstream& output){
   DIR *dir = NULL;
   struct dirent *file = NULL;
   if((dir = opendir(directory.c_str())) == NULL){
 
   }
   while((file = readdir(dir)) != NULL){
      if(strcmp(file->d_name, ".") && strcmp(file->d_name, "..")){
         if(file->d_type == DT_DIR){ // Folder
	   dumpFile(directory.append("/").append(file->d_name), output);
	 }else{
	    cout << directory << "/" << file->d_name << endl; // Just to see what is the problem
            output << file->d_name << endl << md5sum(directory, file->d_name) << endl; // Write the result on 2 lines (first the name, then de md5
	 }
      }
   }
 
   closedir(dir);
 
}

Donc, si le dossier ne possède que des fichier, ou que je supprime la ligne de récursivité (le if), cela fonctionne parfaitement, mais là, elle bloque et je ne comprend vraiment pas pourquoi.
Type d'erreur :
directory : /home/user/Bureau
Fichier : test.dump, test
Sous dossier : python
sous fichier : aucun

/home/user/Bureau/test.dump
/home/guillaume/Bureau/python/test
terminate called after throwing an instance of 'std::bad_alloc'
what(): std::bad_alloc
Abandon

Ce qui est normal, étant donnée que le fichier test n'existe pas dans le dossier python, mais est sur le bureau.

Merci pour votre aide.

5 problème vont venir
1.-les lien,(ton problème actuel)
2.- les montage (mount) ,
3.-les fichier qui on pas les bon droit
4.- les fichier de grande taille pour le md5
5.- la récursivité dans /proc


Pour ton problème il faut regarder du coter de lstat et stat si mes souvenir son bon

j'ai déjà fait ce type de code, je voudrai juste voir le tien pour comparer,car j'ai été et suis toujours avec des problème dans /proc (donc pas aux points quoi)

je te conseil de faire une fois ta fonction terminée, des essais sur / et des répertoires avec tout les droit possibles, tu verras c'est juste une question de patience.
une fois que t'arrive faire ton / de A a Z avec les md5, il faudrai voir la quantité de fichier , car certain disque monte a 2 To, et encore avec linux tu peux monter autant de disque que tu veux
tu peux vite te retrouver avec un fichier texte de 50 mg
vala

[YoshioToutou]

Merci pour vos réponse, en effet, append ne fait pas une copie, je suis donc passé par une variable intermédiaire, et cela fonctionne parfaitement bien.
J'ai eu des problèmes avec le symlink, que j'ai géré en rajoutant une condition dans le if (sale mais fonctionnel).
Pour une plus grande clarté et portabilitée du code, il faudrait en effet utiliser les stats, ce que je n'ai pas mis en place car je ne dois lancer cela uniquement dans dans le /var/www donc a priopri pas d'éléments trop fantaisistes.
Pour le code (si jamais ca interesse quelqu'un).

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 dumpFile(string directory, ofstream& output){
   string path="";
   DIR *dir = NULL;
   struct dirent *file = NULL;
   if((dir = opendir(directory.c_str())) == NULL){
     return 1;
   }
   while((file = readdir(dir)) != NULL){
      if(strcmp(file->d_name, ".") && strcmp(file->d_name, "..")){
         if(file->d_type == DT_DIR || file->d_type == DT_LNK){
	   path= "";
	   dumpFile(path.append(directory).append("/").append(file->d_name), output);
	 }else{
            output << file->d_name << endl << md5sum(directory, file->d_name) << endl; // Write the result on 2 lines (first the name, then de md5
	 }
      }
   }
 
   closedir(dir);
  return 0;
}