supprimer des éléments selon leur type

Version imprimable

21/06/2011, 13h43
r0d

supprimer des éléments selon leur type

Bonjour,

j'ai une collection (de type ControlCollection : ArrangedElementCollection, IList, ICollection, IEnumerable, ICloneable), qui contient tout types d'objets. Je voudrais pouvoir manipuler (pour l'instant supprimer) des éléments de cette collection en fonction de leur type.

Un exemple concret.
En fait, cette collection est la collection de controles que possède toute winform. Je voudrais, par exemple, supprimer tous ceux qui sont de type Button.
Comment faire?
21/06/2011, 13h49
Gurdil le nain

Tu parcours ta collection et tu switch sur le type de l'objet.

Par contre, il me semble que tu ne pourras pas directement les supprimer dans ta boucle, il faudra stocker leur index et les supprimer après.
21/06/2011, 13h51
r0d

Citation:

Envoyé par Gurdil le nain

Tu parcours ta collection et tu switch sur le type de l'objet.

D'accord, mais comment fait-on ça concrètement?

Citation:

Envoyé par Gurdil le nain

Par contre, il me semble que tu ne pourras pas directement les supprimer dans ta boucle, il faudra stocker leur index et les supprimer après.

C'est ce que je pensais aussi, et j'étais parti sur cette méthode oui.
21/06/2011, 13h51
theMonz31
salut

le plus simple,

Faire une boucle qui regarde tous les éléments présents, et tester si
Code:

1 2 if(myItem is MonType)
Et si c'est le cas, ajouter dans une liste de suppression, et ensuite, enlever
les éléments qui sont présents dans cette liste dans ta liste de controle.

en effet, dans une boucle, il est plus que déconseillé (voir impossible avec les foreach) de supprimer des éléments pendant leur parcours :)

genre :

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
 
//Options 1 - un peu crade
			for(int i = 0; i < Controls.Count;i++)
			{
				if(Controls[i] is Button)
				{
					Controls.RemoveAt(i);
					i--;
				}
			}
 
			// Options 2
			List<Control> listc = new List<Control>();			
			foreach(Control c in this.Controls)
			{
				if(c is Button)
				{
					listc.Add(c);
				}
			}
 
			foreach(Control c in listc)
			{
				Controls.Remove(c);
			}
			listc.Clear();

21/06/2011, 14h01
Gurdil le nain

Je sais pas si la première solution ne va pas générer une IndexOutOfRangeException. Je sais plus s'il recalcule le Count à chaque tour.:aie:
21/06/2011, 14h10
r0d

Okay c'est parfait :D
Merci pour votre aide :ccool:
21/06/2011, 14h11
theMonz31

J'ai testé les 2 solutions ==> Résultat identique !!!

Le Count est recalculé à chaque fois... donc, si tu supprimes un éléments et que
tu décrementes de 1, tu reviens à l'élément précédent...

Mais, personnellement, meme si le code est plus "simple", je n'aime pas du tout cette façon de faire...mais elle fonctionne quand même :)
21/06/2011, 14h27
Gurdil le nain

Citation:

Envoyé par theMonz31

J'ai testé les 2 solutions ==> Résultat identique !!!

Le Count est recalculé à chaque fois... donc, si tu supprimes un éléments et que
tu décrementes de 1, tu reviens à l'élément précédent...

Mais, personnellement, meme si le code est plus "simple", je n'aime pas du tout cette façon de faire...mais elle fonctionne quand même :)

Ouais normal, j'étais juste plus sur qu'il le recalculait...j'utilise tellement peut souvent les For sur les collections génériques. ^^

Commence par la fin

Code:

1
2
3
4
5
6
7
 
int idx = Controls.Count;
while (idx-- > 0)
{
    if (Controls[idx] is CheckBox)
        Controls.RemoveAt(idx);
}

Citation:

Envoyé par theMonz31

genre :

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
 
//Options 1 - un peu crade
			for(int i = 0; i < Controls.Count;i++)
			{
				if(Controls[i] is Button)
				{
					Controls.RemoveAt(i);
					i--;
				}
			}
 
			// Options 2
			List<Control> listc = new List<Control>();			
			foreach(Control c in this.Controls)
			{
				if(c is Button)
				{
					listc.Add(c);
				}
			}
 
			foreach(Control c in listc)
			{
				Controls.Remove(c);
			}
			listc.Clear();

j'aurai plutôt fait une boucle décroissante... ce qui en IL donne une boucle while comme ci-dessus :mrgreen: