Discussion :

[NiamorH]

Bonjour,

j'ai un problème, le destructeur d'une de mes classe n'est pas appellé lors d'un delete.

Ici je passe bien par le delete, l'objet Image recupere du tampon est bien valide avec des attributs memebres cohérents

Code :

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

1
2
3
4
5
6
7
8
9
10
11
      if (m_tampon_image->attendreFileImagesDecodeesPlusVide())
      {
        Image* im = m_tampon_image->defilerImageDecodee();
        
        if (im)
        {
          dessiner();
          delete im;
          m_compteur->tic();
        }
      }

Mais son destructeur n'est pas appelé :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
Image::~Image(void)
{
  if (m_donnees_fic)
    delete [] m_donnees_fic;
 
  if (m_pixels_decode)
    delete [] m_pixels_decode;
 
 
#if defined DEBUG_TAMPON_IMAGE
  fprintf(stderr,"Destructeur Image\n");
#endif
}

J'instancie mes images comme ça :

Code :

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

1
2
3
4
5
6
  // creation d'une image
  im = new Image(header->width,header->height);
  // passage des donnees utiles
  im->setDonnees(donneespixels, donnees, entete, code_format_tga);
 
  return im;

Ma classe Image n'est pas heritee ni n'herite de quoi que ce soit, le destructeur n'est pas declaré virtuel...

Cela vous est il deja arrivé ?

[Laurent Gomila]

Si tu fais une exécution pas à pas, il passe par où ?

PS : si tu utilisais std::vector pour tes tableaux dynamiques, tu n'aurais pas besoin de destructeur.

[NiamorH]

Si tu fais une exécution pas à pas, il passe par où ?

Si je n'ai qu'une image, il fait bien le new Image, ensuite j'utilise mon image (je la decode), puis il passe par le delete en gras, mais il ne passe jamais par le destructeur.

PS : si tu utilisais std::vector pour tes tableaux dynamiques, tu n'aurais pas besoin de destructeur.

Je ne comprends pas comment je pourrais l'utiliser ici. Si tu fais référence à :

Code :

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

m_tampon_image->defilerImageDecodee();

sache que derriere il y a :

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
/**
 * @brief Enleve une image decodee de la file d'image decodee FIFO/PEPS
 * @return L'image decodee defilee
 */
Image*
TamponImage::defilerImageDecodee()
{
  m_sec_crit_file_decodee.entrer();
  Image* im = m_deque_images_decodees.front();
  m_deque_images_decodees.pop_front();
  m_sec_crit_file_decodee.sortir();
 
  if (m_autoriser_remplissage)
    verifierEvenementsFileDecodee();
 
  return im;
}

[Patriarch24]

Code :

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

1
2
std::vector <???> m_donnees_fic;
std::vector <???> m_pixels_decode;

[NiamorH]

non ce n'est pas une bonne idee car ce sont des donnees brutes que je dois pouvoir passer en argument de fonctions systeme du style CreateDIBSection ou meme ReadFile.
Mais a la limite le probleme n'est pas la, je pourrais les deleter à la main de l'exterieur.

J'essaye de comprendre les raisons pour lesquelles un destructeurs n'est pas appelé ??

[Médinoc]

Si m_donnees_fic est un tableau de pointeurs, le destructeurs des objets qu'ils pointent ne seront pas appelés automatiquement : Détruire un pointeur ne fait pas de delete dessus, il y a les types "pointeurs intelligents" pour ça.
Exemples : le std::auto_ptr<> est un pointeur intelligent minimal à sémantique de transfert, les pointeurs intelligents de boost sont plus complexes (compteurs de références, etc.)