Discussion :

[agparchitecture]

Bonsoir à tous,

J'ai dans une application en cours de développement un menu contextuel que je souhaite mettre à jour en fonction d'une List. Le code marche relativement bien à deux exceptions près.

Le problème d'affichage est :
1. Il m'affiche parfois deux fois le même element malgré le fait qu'il n'y en a qu'un
2. il semblerait que le nettoyage ne se fait pas entièrement.

le code utilisé est le suivant :

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
27
28
29
30
31
32
33
34
 
		private void mnuRecherche_TextChanged(object sender, EventArgs e)
		{
			CleanMenu();
 
			Repertoire rep = App.AppIconed.Repertoire;
 
			System.Collections.Generic.List<Personne> lst = rep.Search(mnuRecherche.Text);
 
			ToolStripMenuItem mnua = new ToolStripMenuItem();
 
			mnu.SuspendLayout();
 
			for (int i = 0; i < lst.Count; i++)
			{
				mnua = new ToolStripMenuItem();
 
				mnua.Text = lst[i].ShowForTel();
				//mnua.Click += mnu_Click;
 
				this.mnu.Items.Insert(i, mnua);
			}
 
			mnu.ResumeLayout();
 
		}
 
		private void CleanMenu()
		{
			for (int i = 0; i < this.mnu.Items.Count - 1; i++)
			{
				this.mnu.Items.RemoveAt(i);
			}
		}

Auriez-vous une idée de comment résoudre ce problème lors de l'affiche et surtout lors de la recherche de l'élément?

Merci

[François DORIN]

Bonjour,

Code :

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

1
2
3
4
5
6
7
8
 
		private void CleanMenu()
		{
			for (int i = 0; i < this.mnu.Items.Count - 1; i++)
			{
				this.mnu.Items.RemoveAt(i);
			}
		}

Ici tu as effectivement un problème. Il faut bien comprendre que Count ne te renvoi pas la même valeur à chaque itération, car tu supprimes un élément à chaque fois. De plus, à chaque suppression, l'ordre des éléments est modifié. Si tu supprimes l'élément à la position 2, le 3 devient le 2, le 4 let 3, etc...

Bref, ici, il faut que :

• tu copies la valeur de Count dans une variable avant la boucle, et que tu utilises cette copie pour itérer ;
• tu supprimes les éléments, en commençant par le dernier (ainsi, tu ne modifies pas l'ordre des éléments restant). Variante : tu supprimes toujours le premier !

Exemple :

Code :

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

1
2
3
4
5
6
7
8
9
 
		private void CleanMenu()
		{
                        var count = this.mnu.Items.Count;
			for (int i = 0; i < count - 1; i++)
			{
				this.mnu.Items.RemoveAt(0);
			}
		}

Le premier problème que tu rencontres n'est certainement qu'une conséquence du second.

[agparchitecture]

Oh l'erreur de débutant.... En plus c'est pas la première fois.... Ah programmer devant un bon film à la télé....

Merci François ça marche parfaitement maintenant.

[Noxen]

C'est bien un objet ToolStripItemCollection qui contient des MenuItem, non ? Elle possède une méthode Clear() pour supprimer tous les éléments, pourquoi ne pas l'utiliser ?