Discussion :

[Luffy Duck]

Bonjour à tous

Voilà mon problème : J'ai dans mon appli une fenêtre avec des onglets, sur lesquels je charge des images. les données sont stockés dans des char *, dont je transmets les valeurs à la fenêtre pour qu'elle les affiche.

Evidemment, il faut que je désalloue la mémoire, mais quand je vais sur un autre onglet, Windows (enfin les MFC) a déjà désallouer la mémoire pour les photos qui se trouvent sur cet onglet.

Je teste en faisant

Code :

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

1
2
3
 
if(photo.data)
     delete []photo.data;

Mais ça ne fonctionne pas. Comment je peux tester ce pointeur pour savoir si l'adresse est toujours valide ou pas ?

Merci

[Blowih]

salut

Je teste en faisant

Code :

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

1
2
3
 
if(photo.data)
     delete []photo.data;

Mais ça ne fonctionne pas. Comment je peux tester ce pointeur pour savoir si l'adresse est toujours valide ou pas ?

peux tu preciser, parce que pour moi ton test est correct

[Luffy Duck]

Ben il me met en debug
"<Ptr Incorrect>"

mais il rentre quand même dans le test et essaie le delete.

Et là il me fait une exception, comme quoi il est pas très content, il a essayé d'accéder à une adresse où il n'a pas le droit d'aller...

Je viens de tester en Release, là il ferme le programme : "Opération non conforme, blabla..."

C'est bizarre tout ça quand même !

[Blowih]

Evidemment, il faut que je désalloue la mémoire, mais quand je vais sur un autre onglet, Windows (enfin les MFC) a déjà désallouer la mémoire pour les photos qui se trouvent sur cet onglet.

C'est à dire?


(PS: tu devrai peut etre voir sur le forum visual C++ si c 'est un probleme avec les MFC)

[Luffy Duck]

Ben en fait, j'ai une structure qui contient toutes les photos dont j'ai besoin pour mon appli.

Imaginons que sur l'onglet 1, j'affiche l'image 1, et sur l'onglet 2 la 2.

Je quitte l'onglet 1 pour aller sur le 2è, j'appelle ma fonction qui désalloue la mémoire (enfin qui est censée le faire) car il faut que je recharge les images (c'est pour une application industrielle, donc en "live", il faut afficher les produits en cours de production).

Dans ma fonction Release(), il y a le code que j'ai posté tout à l'heure. Or il n'ya que l'image 1 qui a un pointeur incorrect. J'imagine donc qu'une fonction a libérée la mémoire quand je quitte l'onglet pour aller sur l'autre.

donc soit je ne libère que les images qu'il n'y avait pas sur mon onglet (ce qui ne m'arrange pas du tout), soit je fais un autre test sur le pointeur pour savoir s'il est valide.

[Jean-Marc.Bourguet]

Si c'est un probleme de C++, peut-etre qu'il y a un mismatch entre la fonction utilisee pour allouer (new, new[], malloc) et celle utilisee pour liberer (il faudrait delete, delete[], free) et qu'un tel mismatch est detecte par le mode debug.

Sinon, je ne connais pas assez Windows pour commenter.

[Luffy Duck]

Pour allouer la mémoire, je fais :

Code :

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

photo.data = new char[photo.size];

il faut donc que je fasse

Code :

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

delete []photo.data;

si j'ai bien compris ce que tu m'as dit ?

Et pourtant c'est bien ce que je fais. en fait c'est comme si les données était parti, mais le pointeur non réinitialisé.

Il n'ya donc pas de fonctions qui puisse m'assurer que je peux faire un delete sur le pointeur ?

[Luther13]

y at-il une relation quelconque avec: l'accès d'une zone de mémoire des données privées d'une classe ?

[Luffy Duck]

non, je libère la mémoire dans la classe où se trouve ma structure, par la fonction CFicheProduit::Release();

j'ai essayé de passer la variable en public, mais même résultat.

[Jean-Marc.Bourguet]

Pour allouer la mémoire, je fais :

Code :

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

photo.data = new char[photo.size];

il faut donc que je fasse

Code :

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

delete []photo.data;

si j'ai bien compris ce que tu m'as dit ?

Oui.

Il n'ya donc pas de fonctions qui puisse m'assurer que je peux faire un delete sur le pointeur ?

Rien de standard. Il te faut donc comprendre ce qui se passe.

[Blowih]

je pense pas que les MFC desalloue de la memoire comme ça, ( ça fait un bon moment que j'y ai plus touché je dis peut etre une betise)
de memoire, la selection d'un onblet (avec les MFC) declanche un evemement, sur cet evenement tu dois redessiner la zone graphique avec l'image correspondante.
il me semble que pour dessiner une image, tu passe l'adresse d'un buffersi tu ne modifi pas ce buffer de façon expliqite, je pense pas que les MFC le fasse.

es tu sur de ne pas modifier ou alterer ton tableau de char, de le realloué, ou de modifier le pointeur?

[Invité]

Plutot qu'un pointeur sur un char*, essaie d'inserer un objet ayant un constructeur et un destructeur, et affiche un message sur stdout dans chacun d'eux. De cette facon tu verra precisement si un delete se fait sans que tu le sache.

Je teste en faisant

Code :

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

1
2
3
 
if(photo.data)
     delete []photo.data;

Juste une petite note: Il est inutile (mais surtout redondant) de tester pour un pointeur null avant de faire un delete dessus. Le standard C++ dit bien que delete fait deja cette verification a l'interne.

[Luffy Duck]

pffffffou, sujet résolu

En effet, c'était moi

En fait, pour afficher l'image, je passe le char * data, que je cast en BYTE *.
Evidemment ensuite, je libérais le BYTE *, qui pointais sur data.

J'ai donc fait à la place un memcpy(). Est-ce que c'est propre de faire appel à cette fonction ?

En tout cas, désormais, les delete sur data marche nickel.

Merci à tous

[Fiquet]

J'ai donc fait à la place un memcpy(). Est-ce que c'est propre de faire appel à cette fonction ?

Disons que si tu veux employer les équivalent C++ il y a les std::fill() et std::fill_n().