Mieux Coder: Corriger les fuites

Version imprimable

Bonjour à tous, suite à un énième:

Citation:

Erreur lors de la création d'un Handler de fenêtre

Je viens vous demander de m'aider à remettre en question ma manière coder, en gros il faut libérer la mémoire dés qu'on utilise plus une variable c'est ça?

Car j'ai du mal avec .net profiler ou encore dottrace, je vous demande quels sont les points principaux dans la manière de coder pour éviter ce genre d'erreurs?
Depuis quelques temps, je m'efforce de mettre des Dispose sur tout mes datareader et sur quelques autres variables. Ou faut -il également els placer?

Que rajouter d'autres?

Mon erreur apparait dans une fonction (assez longue et complexe qui crée un calendrier avec 20 personnes sur match aller retour) et ensuite je lui demande de supprimer le controle de chargement (qui se rajoute par dessus en attendant que ça soit finit) et d'afficher l'usercontrol de la page d'accueil (qui récupère les dernières news à son chargement).

Le debuggueur bug ici, dan smon form1.designer:

L'erreur que le debuggueur renvoie:
Code:

1 2 3 4 5 6 7 8 protected override void Dispose(bool disposing) { if (disposing && (components != null)) { components.Dispose(); } base.Dispose(disposing); }
Citation:

System.NullReferenceException was unhandled by user code
Message=La référence d'objet n'est pas définie à une instance d'un objet.
Source=System.Windows.Forms
StackTrace:
à System.Windows.Forms.ListView.OnHandleDestroyed(EventArgs e)
à System.Windows.Forms.Control.WmDestroy(Message& m)
à System.Windows.Forms.Control.WndProc(Message& m)
à System.Windows.Forms.ListView.WndProc(Message& m)
à System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
à System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
à System.Windows.Forms.NativeWindow.DebuggableCallback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
à System.Windows.Forms.UnsafeNativeMethods.IntDestroyWindow(HandleRef hWnd)
à System.Windows.Forms.UnsafeNativeMethods.DestroyWindow(HandleRef hWnd)
à System.Windows.Forms.NativeWindow.DestroyHandle()
à System.Windows.Forms.Control.DestroyHandle()
à System.Windows.Forms.Control.Dispose(Boolean disposing)
à System.Windows.Forms.ContainerControl.Dispose(Boolean disposing)
à System.Windows.Forms.Form.Dispose(Boolean disposing)
à BM2010.frmSaisiesBoutons.Dispose(Boolean disposing) dans C:\Users\Victor\Documents\PROG\PROJET BASKET MANAGER\CODE\C#\BM2010\BM2010\Form1.Designer.cs:ligne 20
à System.ComponentModel.Component.Dispose()
à System.Windows.Forms.ApplicationContext.Dispose(Boolean disposing)
à System.Windows.Forms.Application.ThreadContext.DisposeThreadWindows()
InnerException:

Mais l'erreur apparait également lorsque je lance mon application (je récupère également les news au chargement) et dans les deux cas l'user control des news ne se charge qu'à moitié, j'en déduis que ça vient de là mais alors pouvez vous me dire ce que j'ai fait pour que ça plante?
Parce que j'imagine que ce genre d'erreurs est récurrent, peut etre n'avez vous pas besoin de mon code.

D'après mes recherches c'est juste que trop de trucs se chargent en meme temps mais je fais comment pour charger autrement?

Pour mon code je sais pas lequel poster parce qu'en fait j'ai une listview et un tableau, et seul la listview se charge donc ça peut venir du designer non?

Voilà, ce post a donc double vocation, à savoir m'aider à comprendre comment coder plus proprement mais également pour résoudre ce problème qui apparait assez fréquemment.

Merci d'avance, r0seBa

20/03/2010, 00h49
Spaeda
Le problème semble venir d'un objet null.

Il faut absolument prendre l'habitude de tester les objets que tu utilise, surtout quand ces derniers sont charger via des sources externes.
Code:

1 2 3 4 5 6 object obj = maClass.GetNews(); if(obj != null) { //Je peux travailler avec mon objet }

LE problème peut en effet venir de là puisque j'utilise la meme variable pour mon objet (c'est peut etre pas la meilleure solution en fait): Je l'ai déclaré en statique au début de mon code.

Je sais pas si c'est la meilleure solution vu que si je fais ça pour toutes les pages ça fait beaucoup.

En fait je vois pas comment il pourrait être null parce que pour le moment je faisais ça pour après avoir créé le calendrier:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 this.Controls.Remove(chargement); chargement.Dispose(); panel1.Controls.Clear(); this.accueil_titre.Text = RM.GetString("ACCUEIL_MESSAGERIE"); if (show_messagerie == null) { show_messagerie = new messagerie(); } show_messagerie.Location = new Point(0, 0); show_messagerie.Dock = DockStyle.Fill; panel1.Controls.Add(show_messagerie);
avec show_messagerie déclaré comme suit au début de mon code:

Code:

public static messagerie show_messagerie;

Merci

23/03/2010, 17h28
r0seBa

Vous pensez que ça vient de mon control, des taches que j'effectue dedans? ou de la manière de l'ajouter?

J'arrive vraiment pas à corriger ce bug :s

Citation:

Envoyé par r0seBa

Depuis quelques temps, je m'efforce de mettre des Dispose sur tout mes datareader et sur quelques autres variables. Ou faut -il également els placer?

Dès que t'as fini de t'en servir. Une façon simple de coder en Ado.Net est la suivante :
Code:

1 2 3 4 5 6 7 8 9 10 11 using (DbConnection conn = gnagna.CreateConnection()) { conn.Open(); using(DBCommand cmd = conn.CreateCommand()) { cmd.CommandText = "select bidule from truc"; using(DbDataReader reader = cmd.ExecuteNonQuery()) { } } }
Le mot clé using t'assure que la méthode de l'objet en question sera bien appelée à la sortie du bloc, y compris en cas d'exception.

Donc : OUI, dès que tu crées une instance d'une classe IDisposable, il faudra à un moment appeler Dispose. Soit explicitement, soit en le mettant dans un bloc "using".

our ton cas particuliers des forms : le designer de forms crée une variable privée nommée components. Si tu rajoutes un objet là dedans, il sera disposé en même temps que la form.

24/03/2010, 22h16
r0seBa

Bonjour, merci de ta réponse.

Mais alors d'après tes explications ça veut dire que si je met Dispose() c'est équivalent? donc au final ça ne résoud pas mon problème, même si ça répond parfaitement à ma question pour coder plus proprement, je tacherais d'essayer de m'en servir. Merci

EN utilisant le débuggueur je trouve l'erreur ici:
EDIT: Finalement elle est pas là je réédite mon message dés que j'ai trouvé d'ou elle venait
25/03/2010, 09h53
Philippe Vialatte

bon, mes 2 cts.

Citation:

Mon erreur apparait dans une fonction (assez longue et complexe qui crée un calendrier avec 20 personnes sur match aller retour) et ensuite je lui demande de supprimer le controle de chargement (qui se rajoute par dessus en attendant que ça soit finit) et d'afficher l'usercontrol de la page d'accueil (qui récupère les dernières news à son chargement).

Si la fonction est longue est complexe, splitte la en plusieurs sous-fonctions moins longues et complexes, pense SRP...ca te permettra de suivre plus facilement le cycle de vie de tes objets...

Citation:

LE problème peut en effet venir de là puisque j'utilise la meme variable pour mon objet (c'est peut etre pas la meilleure solution en fait): Je l'ai déclaré en statique au début de mon code.

Mal ! On est en .Net, pas en C ;).
Une bonne pratique est de déclarer tes variables le plus prés possible de l'endroit ou tu les utilise.

Pour ton problème, le plus simple, c'est de commencer par commenter tout ton code, et dé-commenter au fur et à mesure ;)

Si tu as un controleur de code source (j'espère pour toi...), tu tagges ton fichier courant, et comme ca tu n'as pas de risque ;)