Forum |  HardWare.fr | News | Articles | PC | S'identifier | S'inscrire | Shop Recherche
1606 connectés 

  FORUM HardWare.fr
  Programmation
  C++

  [C++] Erreur lors de lecture/écriture d'un certain nb de fichiers

 
 


 Mot :   Pseudo :  
 
Bas de page
Auteur Sujet :

[C++] Erreur lors de lecture/écriture d'un certain nb de fichiers

n°989823
benj63
Posté le 22-02-2005 à 14:26:10  profilanswer
 

Bonjour,
 
J'ai créé une classe StringList qui me permet de stocker dans un vecteur de string, des chaines de caractères.
 
Voici ma fonction de lecture :

Code :
  1. void StringList::LoadFromFile(string nom_fichier)
  2. {
  3.         this->Clear();
  5.         char str[10000];
  6.         ifstream fichier(nom_fichier.c_str());
  7.         if(fichier)
  8.         {
  9.              while(fichier.getline(str,10000))
  10.                 this->Add(StrReplace(ToString(str), "\r", "" ));
  11.              fichier.close();
  12.         }
  13. }


 
Et voici ma fonction d'écriture :  

Code :
  1. void StringList::SaveToFile(string nom_fichier)
  2. {
  3. ofstream fe (nom_fichier.c_str(), ios::trunc);
  4. int i;
  5. if(fe)
  6. {
  7.         i=0;
  8.         while(i<this->Count)
  9.         {
  10.                 fe << this->Strings[i].c_str() << "\r\n";
  11.                 i++;
  12.         }
  13.         fe.close();
  14. }
  15. else
  16.      cout << "Erreur d'ouverture en ecriture du fichier " << nom_fichier << endl;
  17. }


 
Au bout d'un certain nombre de lectures/écritures (nombre aléatoire !!), mon programme plante et ne parvient plus à ouvrir de fichiers en écriture.
 
Je ne comprends pas du tout pourquoi !  :sweat: Auriez-vous des pistes pour m'éclairer ?? Merci !


Message édité par benj63 le 22-02-2005 à 14:26:28
mood
Publicité
Posté le 22-02-2005 à 14:26:10  profilanswer
 

n°989830
benj63
Posté le 22-02-2005 à 14:28:30  profilanswer
 

Petite précision : je suis sous RedHat 9 et mon compilateur est GCC 3.2.2-5

n°989831
Taz
bisounours-codeur
Posté le 22-02-2005 à 14:30:06  profilanswer
 

hile(fichier.getline(str,10000))
 
 
c'est pas fini ces conneries ?
 
 
std::string line;
 
while(std::getline(fin, line)) { ... }

n°989833
benj63
Posté le 22-02-2005 à 14:32:22  profilanswer
 

c'est ce que j'avais mis avant... mais ça ne fonctionnait pas mieux...  :sweat:

n°989837
benj63
Posté le 22-02-2005 à 14:36:33  profilanswer
 

Voici ma fonction de lecture, corrigée :
 

Code :
  1. void StringList::LoadFromFile(string nom_fichier)
  2. {
  3.         this->Clear();
  5.         string str;
  6.         ifstream fichier(nom_fichier.c_str());
  7.         if(fichier)
  8.         {
  9.              while(getline(fichier,str))
  10.                 this->Add(StrReplace(str, "\r", "" ));
  11.              fichier.close();
  12.         }
  13. }

n°989838
Taz
bisounours-codeur
Posté le 22-02-2005 à 14:36:53  profilanswer
 

déjà passe en TOUT std::string, ensuite on verra

n°989840
Taz
bisounours-codeur
Posté le 22-02-2005 à 14:37:28  profilanswer
 

c'est quoi ta fonction StrReplace ?

n°989848
benj63
Posté le 22-02-2005 à 14:47:36  profilanswer
 

J'ai tout passé en std::string... même erreur.
 
Ma fonction StrReplace permet de remplacer dans une chaine de caractères, un bout de chaine par une autre :

Code :
  1. string StrReplace(std::string chaine, std::string avant, std::string apres)
  2. {
  3. int pos;
  4. pos=StrPos(avant, chaine);
  5. while(pos!=-1)
  6. {
  7.         chaine.replace(pos-1, avant.length(), apres);
  8.         pos=StrPos(avant, chaine);
  9. }
  10. return chaine;
  11. }


 
avec StrPos pour trouver la position d'un caractère dans une chaîne :

Code :
  1. int StrPos(std::string caractere, std::string chaine)
  2. {
  3.         int pos;
  4.         pos=chaine.find(caractere);
  5.         if(pos!=-1)
  6.                 pos=pos+1;
  7.         return pos;
  8. }

n°989868
Taz
bisounours-codeur
Posté le 22-02-2005 à 15:03:26  profilanswer
 

bon et alors ça plante ou ?

n°989882
benj63
Posté le 22-02-2005 à 15:17:16  profilanswer
 

En fait j'ouvre une dizaine de fichiers, et j'en écris au total plus d'un milliers. C'est généralement aux alentours de 1000 fichiers écrits qu'il "plante" dans le sens où il ne parvient plus à ouvrir les fichiers en écriture dans SaveToFile :
 

Code :
  1. ofstream fe (nom_fichier.c_str(), ios::trunc);


 
renvoie NULL (alors que le fichier n'existe pas encore, et qu'il n'y aucune limitation d'accès à ce répertoire...)

mood
Publicité
Posté le 22-02-2005 à 15:17:16  profilanswer
 

n°989885
Taz
bisounours-codeur
Posté le 22-02-2005 à 15:18:38  profilanswer
 

comment ça renvoie NULL ?

n°989891
benj63
Posté le 22-02-2005 à 15:22:28  profilanswer
 

Ma fonction SaveToFile renvoie :
 
Erreur d'ouverture en ecriture du fichier nom_fichier1098.txt
Erreur d'ouverture en ecriture du fichier nom_fichier1099.txt
Erreur d'ouverture en ecriture du fichier nom_fichier1100.txt
Erreur d'ouverture en ecriture du fichier nom_fichier1101.txt
Erreur d'ouverture en ecriture du fichier nom_fichier1102.txt
Erreur d'ouverture en ecriture du fichier nom_fichier1103.txt
Erreur d'ouverture en ecriture du fichier nom_fichier1104.txt
...
 
donc je me dis que fe est NULL non ?

n°989894
Taz
bisounours-codeur
Posté le 22-02-2005 à 15:23:18  profilanswer
 

c'est un pointeur fe ? d'où il sort ce message d'erreur ?

n°989908
benj63
Posté le 22-02-2005 à 15:29:40  profilanswer
 

Dans SaveToFile :
 

Code :
  1. ofstream fe (nom_fichier.c_str(), ios::trunc);
  2. if(!fe)
  3.    cout << "Erreur d'ouverture en ecriture du fichier " << nom_fichier << endl;


 
mais ce n'est peut-être pas ainsi qu'on vérifie si un fichier est bien ouvert ?!

n°989912
Taz
bisounours-codeur
Posté le 22-02-2005 à 15:32:49  profilanswer
 

ulimit -n ?
 
ce que je comprends pas c'est pourquoi les descripteurs ne sont pas fermés ... tu ferais pas de la rétention de stream quelque part ?

n°989920
benj63
Posté le 22-02-2005 à 15:37:39  profilanswer
 

[quote=989912,0,15,72553]ulimit -n ?
 
ce que je comprends pas c'est pourquoi les descripteurs ne sont pas fermés ... tu ferais pas de la rétention de stream quelque part ?[/quote]
 
ulimit -n me retourne 1024... donc 1024 fichiers avant bug ou je mélange tout ?!
 
Il est possible que je fasse de la rétention de stream, comment m'en assurer ??
 
Merci  :jap:

n°989928
Taz
bisounours-codeur
Posté le 22-02-2005 à 15:41:29  profilanswer
 

genre tu maintiendrais pas une collection de stream ? ou tu ferais pas des new ofstream() ?

n°989932
benj63
Posté le 22-02-2005 à 15:42:16  profilanswer
 

ah peut-être... enfin pas de new() ostream mais maintenir une collection?... Que veux-tu dire par là ?

n°989943
Taz
bisounours-codeur
Posté le 22-02-2005 à 15:45:58  profilanswer
 

ben une liste un vecteur :o

n°989959
benj63
Posté le 22-02-2005 à 15:51:05  profilanswer
 

non je ne fais pas de liste de vecteurs de fstream, ifstream ou ofstream... Par contre y'a-t'il une commande comme ulimit -n pour vérifier lors de l'exécution de mon programme s'il empile des streams sans les libérer au fûr et à mesure ?

n°989965
Taz
bisounours-codeur
Posté le 22-02-2005 à 15:53:26  profilanswer
 

essaie de strace'r ton programme (man strace)

n°989973
HelloWorld
Salut tout le monde!
Posté le 22-02-2005 à 15:55:31  profilanswer
 

Est-que tu as une fonction récursive quelque part ?


---------------
FAQ fclc++ - FAQ C++ - C++ FAQ Lite
n°989979
benj63
Posté le 22-02-2005 à 15:57:04  profilanswer
 

[quote=989973,0,22,21301]Est-que tu as une fonction récursive quelque part ?[/quote]
 
Oui, StrReplace (voir ci-dessus)

n°990019
Taz
bisounours-codeur
Posté le 22-02-2005 à 16:08:30  profilanswer
 

Code :
  1. #include <iostream>
  2. #include <sstream>
  3. #include <fstream>
  4. #include <string>
  5. #include <stdexcept>
  6. #include <cstring>
  7. #include <cstdlib>
  9. namespace My
  10. {
  11.   using std::string;
  13.   string get_first_line(const string &filename)
  14.   {
  15.     std::ifstream f(filename.c_str());
  16.     string line;
  18.     if(!f) {
  19.       std::ostringstream os;
  20.       os << "Cannot open "
  21.  << '\'' << filename.c_str() << '\''
  22.  << '\t'
  23.  << std::strerror(errno);
  25.       throw std::runtime_error(os.str());
  26.     }
  28.     std::getline(f, line);
  30.     return line;
  31.   }
  32. }
  35. int main()
  36. {
  37.   std::string filename;
  38.   unsigned i = 0;
  40.   while(std::getline(std::cin, filename))
  41.     {
  42.       i++;
  44.       try
  45. {
  46.   My::get_first_line(filename);
  47. }
  48.       catch(std::exception &ex)
  49. {
  50.   std::cerr << ex.what() << std::endl;
  51.   // break;
  52. }
  54.       if(i % 1000 == 0)
  55. std::cout << i << '\n';
  56.     }
  58.   std::cout << "Opened files : " << i << '\n';
  59. }


 
ça donne quoi ce genre de chose chez toi ?
 
find /etc -type f 2>/dev/null | ./a.out
 
par exemple

n°990029
HelloWorld
Salut tout le monde!
Posté le 22-02-2005 à 16:12:06  profilanswer
 

Il faut faire des close() avant d'entrer en récursion...


---------------
FAQ fclc++ - FAQ C++ - C++ FAQ Lite
n°990031
Taz
bisounours-codeur
Posté le 22-02-2005 à 16:12:43  profilanswer
 

[quote=990029,0,25,21301]Il faut faire des close() avant d'entrer en récursion...[/quote]rien à voir, merci de lire le code.

n°990032
benj63
Posté le 22-02-2005 à 16:12:57  profilanswer
 

:heink: Est-ce que cette fonction ne poserait pas problème par hasard ?
(en ne fermant pas le opendir)  
 

Code :
  1. bool DirectoryExists(string repertoire)
  2. {
  3. return(opendir(repertoire.c_str()));
  4. }

n°990036
Taz
bisounours-codeur
Posté le 22-02-2005 à 16:13:57  profilanswer
 

man opendir
man 2 stat <- remplace par ça

n°990037
Taz
bisounours-codeur
Posté le 22-02-2005 à 16:14:15  profilanswer
 

et les const &, c'est quand tu veux

n°990038
HelloWorld
Salut tout le monde!
Posté le 22-02-2005 à 16:14:23  profilanswer
 

[quote=990031,0,26,72553]rien à voir, merci de lire le code.[/quote]
J'te parle pas à toi.


---------------
FAQ fclc++ - FAQ C++ - C++ FAQ Lite
n°990040
Taz
bisounours-codeur
Posté le 22-02-2005 à 16:15:10  profilanswer
 

il n'empêche


Message édité par Taz le 22-02-2005 à 16:15:17
n°990047
Taz
bisounours-codeur
Posté le 22-02-2005 à 16:19:55  profilanswer
 

rapidement
 

Code :
  1. bool isDirectory(const std::string &filename)
  2. struct stat buf;
  3. if(stat(filename.c_str(), &buf) == -1) return false;
  4. return S_ISDIR(buf.st_mode);
  5. }


Message édité par Taz le 22-02-2005 à 16:20:13
n°990049
benj63
Posté le 22-02-2005 à 16:21:09  profilanswer
 

[quote=990019,0,24,72553]
 
(...)
 
ça donne quoi ce genre de chose chez toi ?
 
find /etc -type f 2>/dev/null | ./a.out
 
par exemple[/quote]
 
me donne :  
 
1000
Opened files : 1264

n°990051
Taz
bisounours-codeur
Posté le 22-02-2005 à 16:22:05  profilanswer
 

ouais, je pense vraiment que c'est ta fuite de ressources au niveau d'opendir. Tu as peut être d'autres fuites, mais en corrigeant ça tu devrais déjà aller plus loin

n°990056
benj63
Posté le 22-02-2005 à 16:24:53  profilanswer
 

Faut-il inclure une unité pour utiliser stat comme tu le fais dans ton code de isDirectory ?

n°990059
Taz
bisounours-codeur
Posté le 22-02-2005 à 16:26:03  profilanswer
 

STAT(2)                   Manuel du programmeur Linux                  STAT(2)
 
NOM
       stat, fstat, lstat - Obtenir le statut d’un fichier (file status).
 
SYNOPSIS
       #include <sys/types.h>
       #include <sys/stat.h>
       #include <unistd.h>
 
 
 
c'était trop te demander d'ouvrir le manuel de stat ?

n°990070
benj63
Posté le 22-02-2005 à 16:31:29  profilanswer
 

Désolé, je n'ai pas encore pris l'habitude de faire des man lorsque je programme sous Unix... Je développe la plupart de mes applications sous C++ Builder sous Windows, et là depuis peu en me plongeant dans du codage + standard et + propre sous Unix, j'ai un peu du mal pour chercher de l'aide etc...  :sweat:  
 
Enfin bref, il est vrai que j'aurais pu trouver cette réponse tout seul.
 
 :jap:  :jap: Merci pour ton aide, c'était effectivement la fonction DirectoryExists() qui ouvrait des tas de stream sans les fermer...

n°990074
Taz
bisounours-codeur
Posté le 22-02-2005 à 16:33:04  profilanswer
 

perdu. ta fonction DirectoryExists ouvre des descripteurs de fichiers avec la fonction C POSIX opendir, et ne les referme pas.

n°990079
benj63
Posté le 22-02-2005 à 16:39:38  profilanswer
 

oui enfin je m'étais compris à mon niveau...
 
merci pour ta correction.

mood
Publicité
Posté le   profilanswer
 


Aller à :
Ajouter une réponse
  FORUM HardWare.fr
  Programmation
  C++

  [C++] Erreur lors de lecture/écriture d'un certain nb de fichiers

 

Sujets relatifs
Erreur de segmentationQuestion sur lecture de fichier !!
Formulaire Combobox + Value = erreur de récuparation ???Erreur de déploiement d'EAR sous Weblogic 8.1
message d'erreur bien etrangeVBSCRIPT, tri/suppression fichiers
[PHP] Coppermine et les extensions de fichierschemin de fichier pour une lecture.
[C/C++] Lecture de structures en C++ ? Nouvelles questions >erreur SQL sans référence
Plus de sujets relatifs à : [C++] Erreur lors de lecture/écriture d'un certain nb de fichiers

Forum MesDiscussions.Net, Version 2010.2
(c) 2000-2011 Doctissimo
Page générée en 0.174 secondes

Copyright © 1997-2022 Hardware.fr SARL (Signaler un contenu illicite / Données personnelles) / Groupe LDLC / Shop HFR