Discussion :

[Guyome41]

Bonjour, je suis en train de faire une transparence sur une image via un code que j'ai trouvé :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
        Dim newBmp = New Bitmap(PictureBox1.Image.Width, PictureBox1.Image.Height)
        Dim graf = Graphics.FromImage(newBmp)
        Dim ColMatrix As New ColorMatrix
        Dim imgAttribute As ImageAttributes = New ImageAttributes()
        ColMatrix.Matrix33 = 0.3
 
        imgAttribute.SetColorMatrix(ColMatrix, ColorMatrixFlag.Default, ColorAdjustType.Bitmap)
        graf.DrawImage(PictureBox1.Image, New Rectangle(0, 0, newBmp.Width, newBmp.Height), 0, 0, PictureBox1.Image.Width, PictureBox1.Image.Height, GraphicsUnit.Pixel, imgAttribute)
 
        PictureBox2.Image = newBmp
        graf.Dispose()

Que j'insert dans un bouton, le code fonctionne bien.
Le problème et que à chaque fois que l'on clique sur le bouton le

Code :

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

Dim newBmp = New Bitmap(PictureBox1.Image.Width, PictureBox1.Image.Height)

Insert un bitmap dans la ram et donc à 500 clique on a plus de mémoire ram.

Pour vider la ram j'ai pensé à faire un newBmp.Dispose() après PictureBox2.Image = newBmp mais le picturebox2 n'a pas le temps d'inserer l'image à cause du dispose et sa me retourne donc une erreur.

Apparemment Vb.net n'attend pas que l'instruction PictureBox2.Image = newBmp se termine pour passer à newBmp.Dispose()

Une personne à une idée du problème ? Merci

[rv26t]

Bonjour,

En fait tu libère la ressource, ton objet bitmap. PictureBox2.Image et newBmp pointe sur le même objet.

Fait,

Code :

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

newBmp = Nothing

Cela ne concernera que newBmp.

A+, Hervé.

[Guyome41]

Salut, j'ai déjà tenté le nothing sur le newBmp et sa ne libère pas la Ram. Avec le code sur un timer à 100, même avec un nothing sur le newBmp au bout de 2 secondes plus de 400 Mo d'utilisés.

Pour ce qui est "qu'il pointe du même objet", ce qui est étonnant c'est que si je lance le dispose dans un nouveau thread qui patiente, là je n'est pas d'erreur, étonnant.

[ebastien]

Bonjour,

A mon avis : Tu définis newBmp en dehors de ta procedure et à chaque clique sur le bouton, tu testes :

Code :

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

1
2
3
4
5
6
If newBmp IsNot Nothing Then
  newBmp.dispose
  newBmp = Nothing
End If
newBmp = New Bitmap(PictureBox1.Image.Width, PictureBox1.Image.Height)
...

ça devrait résoudre le problème.
eb.

[Pol63]

Apparemment Vb.net n'attend pas que l'instruction PictureBox2.Image = newBmp se termine pour passer à newBmp.Dispose()

ce n'est pas ca qui se passe
les instructions sont exécutées les unes après les autres
c'est juste que .Dispose détruit un objet géré par l'OS (c'est le cas des images) et libère sa mémoire
donc si tu donnes l'image au picturebox et que tu la détruit il ne peut pas l'afficher (l'affichage n'est pas fait qu'à un l'instant T, l'interface est rafraichit assez souvent, et donc le picturebox a besoin de l'image)

pour éviter que la ram monte, il ne faut pas détruire le bitmap que tu viens de créer, mais celui que tu utilisais avant et que tu n'utilises plus, comme le montre ebastien, qui peut aussi se code de la manière suivante :

Code :

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

1
2
3
4
If PictureBox2.Image IsNot Nothing Then
  PictureBox2.Image.Dispose 
End If
PictureBox2.Image = newbmp

une variable est mise à nothing en sortie de portée (sur un dim dans une sub, la variable est mise à nothing à la sortie de la sub)
par contre le contenu est détruit à postériori et aléatoirement dans le temps
Dispose force le vidage de la ram des objets gérés par l'OS

[Guyome41]

Merci pour vos réponses, c'est vrai que Windows rafraîchi l'affichage et je n'y avais pas pensé malgré avoir lu quelque tutos sur le sujet.

Bref, sa fonctionne bien j'ai plus qu'a l'adapter sur mon code. Par contre il y a un truc que je ne comprend pas :

Quand tu fait un dispose() puis tu recharge l'image PictureBox2.Image = newbmp il y a bien un lapse de temps ou la picturebox n'a plus d'image alors pourquoi sa ne plante pas ?

[Pol63]

il y a plusieurs raisons pour qu'un controle se redessine, l'une d'entre elles est le changement de valeur d'une propriété (ou d'instance dans le cas d'image)
quand on fait .dispose sur l'image, la propriété ne change pas, le picturebox n'est pas averti
par contre en mettant newbmp là le picturebox sait qu'il doit se redessiner, mais sont image existe
pendant l'exécution de ta sub, il ne peut pas y avoit un event qui s'exécute entre temps, donc même si windows demande le rafraichissement du controle, ca ne sera fait qu'après la sortie de ta sub

[Guyome41]

Merci.

[rv26t]

Pour information,

Salut, j'ai déjà tenté le nothing sur le newBmp et sa ne libère pas la Ram. Avec le code sur un timer à 100, même avec un nothing sur le newBmp au bout de 2 secondes plus de 400 Mo d'utilisés.
...

Effectivement nothing ne libère pas la mémoire occupé par l'objet, chaque objet créé occupera donc une place supplémentaire, mais à un moment le système déclenchera le Garbage Collector qui nettoiera la mémoire des objets qui ne sont plus référencés.
(c'est pour cela que je te disais de mettre simplement ta variable newBmp sur nothing pour ne pas risquer de modifier PictureBox2.Image à travers newBmp et laisser le GC faire le ménage lorsque le besoin se fera sentir)

C'est ce que t'indique Pol63 dans le post 5

une variable est mise à nothing en sortie de portée (sur un dim dans une sub, la variable est mise à nothing à la sortie de la sub)
par contre le contenu est détruit à postériori et aléatoirement dans le temps

A+, Hervé.

[Pol63]

le gc c'est bien, mais dans certains cas dispose est indispensable (sous peine de OutOfMemoryException qui plante tout, même catchée)
et dans tous les cas il est recommandé

[Guyome41]

Oui en laissant tourné l'appli j'avais le droit à un "Mémoire insuffisante". Quand j'ai vue qu'a 3, 4 secondes l'appli bouffé toute ma ram je cherché une solution.

Enfin, maintenant une chose est sûr c'est que je vais faire plus attention à mes variables pour libérer au max la ram sur mes prochaines appli