[C#]Analyse de code -> warning CA2000

Bonjour,

Je viens d'installer VS2010 version trial, pour me faire une idée et surtout parce que des fonctionnalités de C#4 m'intéressent.

Je migre un code en cours de développement qui fonctionne.

Par curiosité, je lance l'analyse de code, et j'obtiens pour plusieurs blocs de code un warning CA2000.

Par exemple, pour le bloc qui suit, j'obtiens :

Citation:

---

Avertissement 2 CA2000 : Microsoft.Reliability : Dans la méthode 'MdiMain.MdiMain()', l'objet 'item' n'a pas été supprimé dans tous les chemins d'accès d'exception. Appelez System.IDisposable.Dispose sur l'objet 'item' avant que toutes les références s'y rapportant ne soient hors de portée. D:\Visual_Studio\C#\Domogest\Domogest\MdiMain.cs 153 Domogest

---

Je comprends parfaitement ce que ça veut dire, mais l'ennui est que le bloc concerné est ceci :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

// Pour chaque valeur de langue listée dans l'Enum Langages foreach (Langages langue in Enum.GetValues(typeof(Langages))) { // construire un item basé sur le nom correspondant à la valeur et lui affecter un événement var item = new ToolStripMenuItem( Enum.GetName(typeof(Langages),langue), null, new EventHandler(tsmiLangage_Click)); // Cocher si correspond aux préférences actuelles item.Checked = langue == Langue; // Tag = valeur -> facilite le traitement dans le menu item.Tag = langue; // Placer l'item dans Préférences->Langages tsmiLangue.DropDownItems.Add(item); }

---

Et bien entendu, si j'ajoute un

Code:

---

item.Dispose();

---

après le .add(item), ben évidemment je détruis l'item de menu que je viens de créer, et donc je n'ai plus aucun item créé en sortie de la boucle.

Or, la msdn m'indique que je dois absolument utiliser un dispose :

Citation:

---

Ne supprimez pas d'avertissement de cette règle, à moins que vous ayez appelé une méthode sur votre objet qui appelle Dispose, tel que Close.

---

Ai-je raté quelque chose, est-ce que ma façon de procéder est incorrecte, ou est-ce l'analyseur de code qui est un peu "fantaisiste"?

Merci d'avance
Claude

Pour être clean, il faudrait try catcher ton code avant la .Add, et Disposer ton objet dans le catch. Je suppose que du coup la règle déterminera correctement que dans le seul chemin où la lclasse n'est pas disposée, c'est à dire quand aucune exception n'est levée, une référence à ton objet a été ajoutée.

Ca supposerait que la règle arrive à savoir que Add finit par créer une référence d'un objet vivant (ici ton MdiMain) vers l'objet disposable, chose qu'il me semble bien difficile de déterminer...

Tu peux essayer de réorganiser ton code et faire le Add en tout premier ? Un truc du style

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

var item = new ToolStripMenuItem(...); try { tsmiLangue.DropDownItems.Add(item); } catch { item.Dispose(); item = null; } // là, on est sûrs que tout s'est bien passé : // on a soit une référence, soit l'objet est disposé if (item != null) { item.Checked = langue == Langue; item.Tag = langue; }

---

Citation:

---

Je suppose que du coup la règle déterminera correctement que dans le seul chemin où la lclasse n'est pas disposée, c'est à dire quand aucune exception n'est levée, une référence à ton objet a été ajoutée.

---

J'ai essayé et ça ne change rien, j'ai toujours strictement le même warning.

Citation:

---

Ca supposerait que la règle arrive à savoir que Add finit par créer une référence d'un objet vivant (ici ton MdiMain) vers l'objet disposable, chose qu'il me semble bien difficile de déterminer..

---

Je peux donc présumer que mon code est correct et que c'est l'analyseur de code qui est un peu susceptible?

merci,
Claude