Discussion :

[MaelstroeM]

Hello,

Si l'on regarde dans MSDN pour cette méthode, on trouve cette note :

Important :
L'appel à la méthode Clear ne supprime pas de handle de contrôle de la mémoire. Vous devez appeler explicitement la méthode Dispose pour éviter des fuites de mémoire.

On se demande à quoi peut bien servir le Clear alors...

Bref, il s'avère que je doit utiliser un FlawLayoutPanel pour y ajouter et supprimer des PicturesBox qui comme tout bon contrôle sont "Disposable".

Ma question est donc, que faut il vraiment faire pour bien purger le FlawLayoutPanel ?

ça ?

Code :

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

1
2
3
4
        While MyFlawLayoutPanel.Controls.Count > 0
            MyFlawLayoutPanel.Controls(0).Dispose()
        End While
        MyFlawLayoutPanel.Controls.Clear()

Ou ça ?

Code :

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

1
2
3
4
5
6
7
        While MyFlawLayoutPanel.Controls.Count > 0
            Dim Ctrl As Control
            Ctrl = MyFlawLayoutPanel.Controls(0)
            MyFlawLayoutPanel.Controls.Remove(Ctrl)
            Ctrl.Dispose()
        End While
        MyFlawLayoutPanel.Controls.Clear()

Ou je ne sais quoi d'autre...

Rq : le For Each est à bannir quand on ne peut pas faire de supression dans un For Each, de plus le Dispose semble supprimer le controle de la collection.

Bon c'est barbar je le reconnais, mais je suis un peu dans le flou sur ce coup là. Si qq c'est déjà retrouvé confronter à ce pb a un trouvé une solution efficasse je suis preneur !!!

Merci

[Pol63]

à priori les controls t'as pas besoin de les disposer, sauf si tu en as plus besoin et que tu fermes pas la feuille
quand une form est disposé, elle appelle dispose sur tous ses controles, qui appellent dispose sur tous leurs controles etc...

[MaelstroeM]

Je n'ai plus besoin des controles contenu dans le Flow mais j'ai encore besoin du Flow... Le Dispose sur les PictureBox me semble innévitable quand même.

[Pol63]

dans ce cas là t'as le droit de les disposer
par contre moi j'aurais fait un for each

du genre

Code :

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

1
2
3
4
for each ctr in panel
  panel.controls.remove(ctr)
  ctr.dispose
next

[MaelstroeM]

Oui le dispose me semble inévitable par contre tu ne peux pas utiliser un For Each sur une collection si dans la boucle tu fais des suppression, le For Each perd les pédale, ça décale ses index et il sort de la boucle avant d'avoir parcourru tous les controles.

[Pol63]

ah
étrange ...
y me semblait que je l'avais dejà fait ...