Discussion :

[r0d]

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?

[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.

[r0d]

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

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

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.

[theMonz31]

salut

le plus simple,

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

Code :

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

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

[theMonz31]

genre :

Code :

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

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();

[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.

[r0d]

Okay c'est parfait
Merci pour votre aide

[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

[Gurdil le nain]

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. ^^

[realpasso]

Commence par la fin

Code :

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

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

[kheironn]

genre :

Code :

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

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