destructeur pas appelé ?

Version imprimable

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:

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:

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:

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é ?

26/04/2007, 11h16
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.

Citation:

Envoyé par Laurent Gomila

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.

Citation:

Envoyé par Laurent Gomila

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:

m_tampon_image->defilerImageDecodee();

sache que derriere il y a :
Code:

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

Code:

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

26/04/2007, 11h45
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é ??
26/04/2007, 11h56
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.)

ces deux tableaux sont des tableaux d'uchar donc pas de type complexe

mais le soucis est ailleurs car le point d'arret suivant n'est JAMAIS atteint
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 Image::~Image(void) { // point d'arret 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 }

26/04/2007, 12h59
Laurent Gomila

Je n'ai aucune idée de pourquoi ton destructeur n'est jamais appelé, mais pour ce qui est de ça :

Citation:

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.

C'est une mauvaise excuse. Les std::vector ne sont qu'une encapsulation des tableaux dynamiques classiques, ils ne leurs enlèvent aucune fonctionnalité.
Y a même une entrée dans la FAQ pour l'utilisation avec des fonctions C :
http://c.developpez.com/faq/cpp/?pag...teneurs_avec_c

Peux tu me montrer ce que donne ce code avec un vector, car je ne vois pas bien ?

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
  uchar* donnees = 0;
  unsigned int taille;

  HANDLE fp = 0;
  
  char fichier[256] = "C:\\image.tga";
  
  fp = CreateFile( fichier, FILE_READ_DATA, FILE_SHARE_READ, 0, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL | FILE_FLAG_NO_BUFFERING, 0);

  if (!fp)
      {
        MessageBox (0, "erreur: fichier non trouve ou bloque ", fichier,0);
        return 0;
      }

    // recuperation de la taille du fichier
    taille = GetFileSize(fp,0);
    
    // allocation des donnees pour charger l'image
    donnees = new uchar[ taille ];
  	
    DWORD read;
    // lecture du fichier image d'une traite
    ReadFile(fp, donnees, sizeof(uchar)*taille, &read, 0);

    // fermeture du fichier
     CloseHandle  (fp);
    
    delete [] donnees;

Code:

1
2
3
4
5
6
// allocation des donnees pour charger l'image
donnees.resize(taille);
 
DWORD read;
// lecture du fichier image d'une traite
ReadFile(fp, &donnees[0], sizeof(uchar)*taille, &read, 0);

26/04/2007, 14h14
NiamorH

Ok j'y penserai pour la prochaine fois car pour le moment la manipulation de uchar* ne me gene nul part.

pour revenir a mon probleme je viens de voir un warning que j'avais zappé

Citation:

Envoyé par bill gates

warning C4150: deletion of pointer to incomplete type 'Image'; no destructor called

En gros j'avais oublié l'include "Image.h"
La compilation passait car je n'utilisais aucune methode membre de l'Image recuperee et je devais faire un class Image; quelque part dans les autres includes...

bref résolu.
26/04/2007, 15h32
poukill

Citation:

Envoyé par NiamorH

Ok j'y penserai pour la prochaine fois car pour le moment la manipulation de uchar* ne me gene nul part.

Ca risque de te géner plus vite que prévu ! ;)
26/04/2007, 16h02
NiamorH

C'est vrai lorsqu'on manipule des chaines de caracteres et que l'on veut recupere des sous chaines, detecter certains caracteres clefs faire de la concatenation, j'ai un passage comme ça dans mon appli où j'ai vraiment galéré et à l'avenir j'essayerai d'utiliser des strings, mais dans ce cas ce sont des donnees pixels RGB dont la taille de varie pas de la creation a la deletion.

Non non, la partie decodage de mon appli sur ces tableaux est bien rodee et je retrouve en sortie mes images tels quelles. Aucun probleme de corruption de heap non plus.
26/04/2007, 18h44
diogene

Citation:

pour revenir a mon probleme je viens de voir un warning que j'avais zappé

Citation:

bill: gates a écrit :
warning C4150: deletion of pointer to incomplete type 'Image'; no destructor called

Les warning ne sont donc pas seulement des éléments décoratifs ? 8O
26/04/2007, 19h02
NiamorH

celui ci etait caché parmis les : < attention ! printf est une fonction deprecated >

Je n'en ai pas d'autres à part ceux ci : http://www.developpez.net/forums/sho...d.php?t=324737
27/04/2007, 09h21
Médinoc

Citation:

Envoyé par NiamorH

celui ci etait caché parmis les : < attention ! printf est une fonction deprecated >

Il me semble que printf() elle-même n'est pas deprecated.
Mais bon, un petit #define _CRT_SECURE_NO_DEPRECATE suffit à faire taire ses warnings qui encouragent la non-portabilité (surtout parfois pour des trucs absolument débiles, comme fopen_s()...)
27/04/2007, 10h18
NiamorH

Effectivement, j'ai uniquement du sprintf, strcpy, strncpy et du strtok.

Pas mal ton astuce je vais tester.

EDIT : J'ai toujours 'strdup' deprecated :mouarf:
27/04/2007, 10h31
Médinoc
- sprintf : Pour moi, le mieux est d'utiliser un truc qui reproduit le comportement standard du snprintf C99 : http://www.developpez.net/forums/sho...8&postcount=14
- strcpy, strncpy : C'est vrai qu'elles sont dangereuses, toutes les deux. Une bonne utilisation de strncat() peut être préférable (même s'il l'ont dépréciée aussi :mur: )
- strtok() n'est en effet pas réentrante. La dernière version propose un équivalent direct à la version réentrante POSIX strtok_r(), sous le nom strtok_s().
27/04/2007, 15h21
poukill
Citation:

Envoyé par Laurent Gomila

Je n'ai aucune idée de pourquoi ton destructeur n'est jamais appelé, mais pour ce qui est de ça :

C'est une mauvaise excuse. Les std::vector ne sont qu'une encapsulation des tableaux dynamiques classiques, ils ne leurs enlèvent aucune fonctionnalité.
Y a même une entrée dans la FAQ pour l'utilisation avec des fonctions C :
http://c.developpez.com/faq/cpp/?pag...teneurs_avec_c

Hum Laurent, je vais de tester tes dires sur un bout de mon projet.

Voici la fonction "C" à invoquer
Code:

1 2 extern "C" int IRimages_pas( int num_choc, int camera, long image_deb, long image_fin, long pas, unsigned short ** Images );
Voici comment je l'appelle:
Code:

1 2 3 4 5 std::vector<unsigned short>* image; image->reserve(77120); int cr = IRimages(getChocNum(), numero_camera, num_image, num_image, &image[0]);
Et j'ai l'erreur suivante :

Citation:

error C2664: 'IRimages' : impossible de convertir le paramètre 5 de 'std::vector<_Ty> *__w64 ' en 'unsigned short **'

?? :aie: ??

Je comprend pas !
27/04/2007, 15h24
Médinoc

Tu passes un vecteur de short à un unsigned short **
Cela fait une étoile de trop pour que le vecteur soit utilisable.
27/04/2007, 15h25
Laurent Gomila

Sous hypothèse que la paramètre à passer à ta fonction est un pointeur vers un tableau d'unsigned short* à remplir, alors tu t'en sortirais avec un std::vector<unsigned short*>.

Mais je doute que ce soit le cas, je pense plutôt que la fonction alloue le tableau et l'affecte au pointeur passé en paramètre. Non ?
27/04/2007, 15h30
poukill
Euh... Ce n'est pas moi qui ai écrit la fonction C (sinon elle serait en C++ :mrgreen: )...
Je suis responsable de l'allocation du vecteur, que j'envoie en paramètre...

Mon code (qui marche) précédent est :
Code:

1 2 3 unsigned short *image= new unsigned short[77120]; int cr = IRimages(getChocNum(), numero_camera, num_image, num_image, &image);
27/04/2007, 15h37
Laurent Gomila

Hmm j'ai un doute. Pourquoi la fonction demanderait-elle un pointeur sur pointeur, si le tableau n'a qu'une dimension et est déjà alloué ? Tu es sûr que ton pointeur n'est pas réaffecté dans la fonction (compare la valeur de image avant et après l'appel).
27/04/2007, 15h50
poukill

Hum...
Pour info, cette fonction est dans une DLL (plus pratique).

A l'origine, la fonction IRimage effectuait l'allocation mais ne désallouait pas... C'était à moi (client) de désallouer le tableau! 8O
Mon collègue m'a expliqué qu' en /MD le tas était en commum, donc ça marchait!8O J'ai trouvé ça particulièrement foireux, et je lui ai donc dis que j'allais m'occuper de l'allocation.
J'ai donc râler auprès de mon collègue qui a changé sa fonction, mais du coup pas le prototype.

J'avais pas réfléchi, mais effectivement, la fonction serait à revoir, non?:D
27/04/2007, 16h03
Laurent Gomila
Citation:

J'avais pas réfléchi, mais effectivement, la fonction serait à revoir, non?

Clairement...

Mais en attendant puisque la deuxième étoile n'a plus aucune utilité, tu peux faire ça :
Code:

1 2 3 4 std::vector<unsigned short> image(77120); unsigned short* image_ptr = &image[0]; int cr = IRimages(getChocNum(), numero_camera, num_image, num_image, &image_ptr);
27/04/2007, 16h07
poukill

:aie: Merci Laurent ! :king:
J'y avais pas pensé!
Du coup, je peux laisser ça comme ça, la perte de performance n'est pas énorme? (à la "propreté près)
27/04/2007, 16h10
Laurent Gomila

Il n'y a aucune perte de performances.
27/04/2007, 16h12
poukill

Ok merci Laurent ! :king:
(c'était pas moi l'auteur du post mais j'en ai profité un peu ! ;) )
27/04/2007, 16h20
NiamorH

y a pas de soucis :)
z'etes bienvenue dans mes threads

Fuseau horaire GMT +2. Il est actuellement 15h28.