Problème de mémoire avec Bitmap (c++/cli)

Bonjour à tous ,
Voilà, je suis débutant et je travaille sur un projet avec les winforms en c++/cli ,
Je cherche à afficher une capture d'écran dans un(e) pictureBox, pour cela , j'utilise le code trouvé sur la faq de developpez.com :

Code:

---

1 2 3

System::Windows::Forms::SendKeys::SendWait("+{PRTSC}"); pictureBox1->Image = safe_cast<Bitmap^>(Clipboard::GetDataObject()->GetData("Bitmap"));

---

Le hic , c'est que lorsque je souhaite par exemple mettre ce code dans une boucle :

Code:

---

1 2 3 4 5 6 7 8 9

public : Bitmap^ screen; ..... for(int i=0;i<100;i++){ System::Windows::Forms::SendKeys::SendWait("+{PRTSC}"); screen=safe_cast<Bitmap^>(Clipboard::GetDataObject()->GetData("Bitmap")); pictureBox1->Image=screen; }

---

Là je vois la mémoire qu'utilise le programme augmente de manière très inquiétante, ca ne le fait pas planter mais bon c'est génant ,

Ma question est la suivante: comment cela se fait que ayant déclarer une seule fois le Bitmap qui va "recevoir" la capture d'écran (ici screen) la mémoire utilisé continue à augmenter ?
Sinon, ne dois je pas faire une libération de ressource quelque part ?

Merci à tous ,

A chaque passage dans la boucle, tu fais une nouvelle copie d'écran, et tu la récupères dans un Bitmap... donc c'est un peu normal que la mémoire utilisée augmente. Le fait de ne déclarer qu'une fois le Bitmap ne change rien à l'affaire : la variable screen n'est qu'une référence vers une instance de Bitmap, et à chaque passage dans la boucle c'est une nouvelle instance qui est utilisée.

Vu que tu ne conserves pas de référence vers chaque instance créée, le garbage collector passera au bout d'un moment pour "nettoyer" en récupérant la mémoire des instances inutilisées. Si tu veux forcer le passage du GC, appelle la méthode GC::Collect(). Ou mieux, tu peux appeler Dispose sur le Bitmap (dans ce cas, mets d'abord pictureBox1->Image à null)

ok, donc dans le premier cas , je peux mettre :

Code:

---

1 2 3 4 5 6 7 8 9 10

public : Bitmap^ screen; ..... for(int i=0;i<100;i++){ System::Windows::Forms::SendKeys::SendWait("+{PRTSC}"); screen=safe_cast<Bitmap^>(Clipboard::GetDataObject()->GetData("Bitmap")); pictureBox1->Image=screen; GC::Collect(); }

---

en effet, ca fonctionne bien ,
mais avec le Dispose sur le Bitmap et le null sur l'image du pictureBox, la je bloque parce que le

Code:

---

1 2 3 4 5 6

pictureBox1->Image=NULL   for(...){ ... }

---

Il aime pas du tout

Code:

---

1 2	error C2664: 'System::Windows::Forms::PictureBox::Image::set' : cannot convert parameter 1 from 'int' to 'System::Drawing::Image ^'

---

donc je ne vois pas trop avec ton second conseil , tu peux m'aider ?
Merci

NULL (=0) est un entier, tu ne peux pas le convertir implicitement en Bitmap^, il faut faire un cast

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

public : Bitmap^ screen; ...   for(int i=0;i<100;i++) { System::Windows::Forms::SendKeys::SendWait("+{PRTSC}"); if (screen != null) { pictureBox1->Image = (Bitmap^) NULL; screen->Dispose(); } screen=safe_cast<Bitmap^>(Clipboard::GetDataObject()->GetData("Bitmap")); pictureBox1->Image=screen; }

---

Il vaut mieux faire un Dispose sur l'image que faire un GC::Collect (qui est assez lourd)

Alors , voilà j'ai fait comme tu m'a dit :

Code:

---

1 2 3 4 5 6 7 8 9 10

for(int i=0;i<100;i++){ System::Windows::Forms::SendKeys::SendWait("+{PRTSC}"); if (screen != NULL){ pictureBox1->Image = (Bitmap^) NULL; screen->Dispose(); } screen=safe_cast<Bitmap^>(Clipboard::GetDataObject()->GetData("Bitmap")); pictureBox1->Image=screen; }

---

J'ai quelques erreurs :

Code:

---

1 2 3 4 5 6 7 8 9

if (screen != NULL){.... >>>> error C2446: '!=' : no conversion from 'int' to 'System::Drawing::Bitmap ^'   pictureBox_Image->Image = (Bitmap^) NULL; >>>> error C2440: 'type cast' : cannot convert from 'int' to 'System::Drawing::Bitmap ^'   screen->Dispose(); >>>> error C2039: 'Dispose' : is not a member of 'System::Drawing::Bitmap'

---

Et là je ne sais vraiment pas quoi faire , avec GC::collect(), ca fonctionne nickel mais si tu dis que c'est lourd , c'est pas la peine ,
que dois- je faire ?

Merci bien ,

Ah oui, je me rappelais plus comment on spécifiait une référence nulle en C++/CLI... en fait c'est nullptr. Donc :

Code:

---

1 2 3 4 5 6 7 8 9

for(int i=0;i<100;i++){ System::Windows::Forms::SendKeys::SendWait("+{PRTSC}"); if (screen != nullptr){ pictureBox1->Image = nullptr; screen->Dispose(); } screen=safe_cast<Bitmap^>(Clipboard::GetDataObject()->GetData("Bitmap")); pictureBox1->Image=screen; }

---

Par contre, pour cette erreur :

Code:

---

'Dispose' : is not a member of 'System::Drawing::Bitmap'

---

8O Il a fumé quoi ton compilateur ??! Je suis sûr à 200% que cette méthode existe... en fait elle n'est pas définie dans Bitmap mais dans la classe de base, Image.

Bon , pour le nullptr, c'est parfait , par contre j'aimerai bien qu'il fume un peu moins mon compilateur , voilà ce que j'ai avec :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

for(int i=0;i<10;i++){ System::Windows::Forms::SendKeys::SendWait("+{PRTSC}"); if (screen != nullptr){ pictureBox1->Image = nullptr; } screen=safe_cast<Bitmap^>(Clipboard::GetDataObject()->GetData("Bitmap")); pictureBox1->Image=screen; screen->Dispose(); }   1> error C2039: 'Dispose' : is not a member of 'System::Drawing::Bitmap' 1> c:\windows\microsoft.net\framework\v2.0.50727\system.drawing.dll : see declaration of 'System::Drawing::Bitmap' 1> You should invoke the destructor, '~Bitmap' instead

---

que faire ? :mur:

ok, je crois que j'ai l'explication...
http://msdn.microsoft.com/en-us/library/ms177197.aspx

Citation:

---

Envoyé par MSDN

Because the Visual C++ compiler provides support for deterministic release of resources, Visual C++ programmers should not attempt to implement the Dispose or Finalize methods.

---

Je crois que la solution passe par un bon vieux delete :

Code:

---

1 2 3 4 5 6 7 8 9

for(int i=0;i<100;i++){ System::Windows::Forms::SendKeys::SendWait("+{PRTSC}"); if (screen != nullptr){ pictureBox1->Image = nullptr; delete screen; } screen=safe_cast<Bitmap^>(Clipboard::GetDataObject()->GetData("Bitmap")); pictureBox1->Image=screen; }

---

Sinon je sais pas trop...

Impec', ca fonctionne nickel , merci de ton aide et de ta patience :king:

ben j'ai aussi appris 2 ou 3 trucs au passage... j'utilise rarement C++/CLI donc j'ai quelques lacunes ;)