pb avec gestion memoire

pb avec gestion memoire - C++ - Programmation

Marsh Posté le 03-08-2005 à 12:22:10    

Salut,
 
Alors que je pensais faire un prog tout simple qui lit dans un fichier, j'ai eu ce probleme.
le prog lit dans un fichier les lignes qu'il trouve. Un jeu de donnees se compose d'un titre commençant par '!' et des donnees (ici du texte, mais peut-etre des donnees bianires).
 
A l'aide du debugger ddd, je vois que la chaine _s n'est pas effacee.
 

Code :
  1. #include <iostream>
  2. #include <fstream> //ifstream = lecture fichier
  3. using namespace std;
  6. class B{
  7. public:
  8. char* _s;
  9. string _str;
  11. B(char* chaine="" ){
  12.  _str = string (chaine);
  13. //   _s = (char*) malloc(2560*sizeof(char));
  14.  _s = new char[2560];
  15.  cout << "qllocation de _s\n";
  16. }
  17. ~B(){
  18.  cout << "dstr A\n";
  19. /*  if(s)
  20.   free(_s);*/
  21.  delete[] _s; //n'efface rien en memoire
  22.  _s = NULL; //sans ca, _s toujours actif
  23. }
  24. };
  26. B* Bload(ifstream &fe)
  27. {
  28. char line[256];
  29. char x;
  30. char* ptr = &x;
  31. B* b = NULL;
  33. while(fe.get(*ptr))
  34. {
  35.  if(*ptr=='!')
  36.   if(b == NULL)
  37.   {
  38.    fe.getline(line, 256);
  39.    b = new B(line);
  40.    ptr = b->_s;
  41.   }
  42.   else
  43.   {
  44.    fe.unget();
  45.    break;
  46.   }
  47.  else if(*ptr != '\n')
  48.  {
  49.   if(b == NULL)
  50.    b = new B();
  51.   ptr++;
  52.  }
  53. }
  54. cout << "b->_str = " << b->_str
  55.  << "\tb->_s = " << b->_s << endl;
  56. return b;
  57. }
  59. int main(void)
  60. {
  61. int i;
  62. ifstream fe("dat.txt" );
  63. B* b = NULL;
  65. for(i=0; i<3; ++i)
  66. {
  67.  b = Bload(fe);
  68.  delete b; //marche pas correctement
  69. }
  70. return 0;
  71. }


et le fichier dat.txt

!1e serie de donnees1
ici les premieres donnees rencontrees
sur plusieurs lignes
!donnees 2.
la se trouvent les donnees de la 2e serie
!derniere serie
enfin je termine ici


Merci de m'eclairer sur ce pb de memoire.
 
KooK


Message édité par KooK le 03-08-2005 à 12:25:29
Reply

Marsh Posté le 03-08-2005 à 12:22:10   

Reply

Marsh Posté le 03-08-2005 à 12:55:39    

#         delete[] _s;    //n'efface rien en memoire
#         _s = NULL;    //sans ca, _s toujours actif
 
si si ça efface.
 
# char x;
#     char* ptr = &x;
 
... c'est compliqué tout ça ...
 
 
ça veut dire quoi marche pas ? ton programme est assez dégueux, avec une indentation pas clair du tout.
 
 
je conclue : n'utilise qu'std::string

Reply

Marsh Posté le 03-08-2005 à 13:29:51    

std::string pas question, pour stocker des donnees binaires c'est pas terrible.
l'indentation est tres bonne.

Code :
  1. char x;
  2. char *ptr = &x;


c'est pour stocker le caractere lu tant que je ne sais pas ce que je lis.
 
Et si le delete est correct, alors pourquoi j'ai la sortie suivante :
"qllocation de _s
b->_str = 1e serie de donnees1  b->_s = ici les premieres donnees rencontreessur plusieurs lignes!
dstr A
qllocation de _s
b->_str = donnees 2.    b->_s = la se trouvent les donnees de la 2e serie!
dstr A
qllocation de _s
b->_str = derniere serie        b->_s = enfin je termine icionnees de la 2e serie!
dstr A "
 ?


Message édité par KooK le 03-08-2005 à 13:33:01
Reply

Marsh Posté le 03-08-2005 à 13:47:23    

bien sur ! std::string est toute indiquée pour stocker des données binaires ! tu peux aussi utilise un std::vector.
 
mais si t'es si malin que ça, t'as qu'à correctement terminée tes chaines par un '\0' (comprendre que tes données ne sont donc pas binaire, et qu'ici tu dois te cogner ce problème à la main)
 
 
 
    * char x;
    * char *ptr = &x;
 
ptr te sert à rien, c'est l'assombrissement de code.
 
ptr++;
 
j'espère qu'ici t'es bien sur de plus pointer sur x ... ou de ne pas dépasser les borne de ptr = b->_s;
 
cherche pas, t'es dans les décor.
 
utilise std::string

Reply

Marsh Posté le 03-08-2005 à 14:01:10    

le delete libère juste la mémoire utilisée ... si tu utilises _s après avoir fait le delete, tu n'as aucune garantie quant au contenu de la zone mémoire pointée par _s.

Reply

Marsh Posté le 03-08-2005 à 15:41:13    

Je crois que je viens de comprendre, en fait tout se passe normalement et si j'ai ce bout de phrase collé à la nouvelle, c'est justement parce que la mémoire a été réutilisée et comme je n'ai pas de symbole de fin fin de chaine, il affiche tous jusqu'à en rencontrer un.
 
Une erreur de débutant quoi !
 
->Tazounet, c'est aussi un peu grâce à toi que j'ai compris, mais un peu de douceur, de finesse, de diplomatie que diable. =>"un char* dans un flux affichera tous les caractères jusqu'au premier '\0' rencontré" eu été une réponse tellement plus courtoise et pédagogue.
 
Problème résolu.
KooK

Reply

Marsh Posté le 03-08-2005 à 15:45:03    

bah c'est toi qui sort une sentence sur le string ! alors si apres il faut repartir au "un char* est délimité par un '\0'", c'est difficile de savoir par où commencer.
 
La mémoire est pas réutilisée, c'est exactement là même. Donc si tu t'amuses à change certains caractères au début, la fin reste la même.
 
 
Allez vire TOUS tes char*

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed