Discussion :

[madfu]

Bonjour,

Je développe un composant que je teste depuis une application Winform toute simple.

Je référence la DLL dans le projet Winform et j'instancie le composant dynamiquement.

Ce composant redéfinit la méthode Dispose de la manière suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
  protected override void Dispose(bool disposing)
  {
    if (!_disposed)
    {                
        //Libération des resources non managées
        Close();//Echoue lorsque Dispose est appelée depuis le finalizeur
        GC.SuppressFinalize(this);
         _disposed = true;
    }
  }

La méthode Close

CAS1:j'appelle la méthode Dispose de mon composant dans l'évènement OnClosing de la Form..pas de problème (cf code ci-dessous)

Code :

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

1
2
3
4
5
private void Form1_FormClosing(object sender, FormClosingEventArgs e)
{
   sess.Dispose();//no soucis !!!!
}

CAS2:Je ne place aucun appel à Dispose et laisse le GC appeler le destructeur.
Le code rentre bien dans Dispose mais renvoi une erreur que je n'arrive pas à trapper (du type :"l'application machin.exe a rencontré un problème et doit fermer") à l'appel de la méthode Close.

Je peux bien sûr fournir le code de la méthode Close mais en dehors de ça je ne comprends pas quelle est la différence de fonctionnement entre un appel direct à Dispose et un appel de Dispose via un destructeur . Dans les deux cas c'est le même code qui est exécuté non ?!

[Pilloutou]

Salut,

je pense que cela devrait t'aider:

Implémentation des méthodes Finalize et Dispose pour nettoyer des ressources non managées

Bye

[Sphax]

Non le même code ne doit pas être exécuté, en particulier ton GC.SuppressFinalize(this) ne doit pas être exécuté je pense qd tu es dans le thread du finilizer (ça n'a pas de sens, et c'est peut être dangereux). De plus dans le finalizer tu ne dois pas toucher aux autres objets managés car ceux-ci ont peut-être déja été libérés.

[madfu]

Salut,

Pilloutou, je me suis inspiré de ce genre de tutos pour écrire mon code de finalization mais mon problème n'est pas vraiment là..

Mon code dérive de Component et à l'appel de la méthode Dispose je déclenche bien un appel à Dispose(true) qui à son tour déclenche mon appel à la méthode Close qui fonctionne(cf. CAS 1) .

Par contre lorsque je ferme ma fiche je constate également qu'un Dispose(false) est appelé (normal , le GC appelle le destructeur de la classe Component qui lui même appel ma méthode Dispose surchargée) et dans ce cas uniquement ma méthode Close plante...(cf CAS 2)

Le paramètre true ou false n'a pas d'intérêt pour moi car la méthode Close ne s'occupe que de la libération de ressources non managées via l'appel de méthode de DLL.

[madfu]

Effectivement Sphax c'est une bonne remarque pour le GC.SuppressFinalize(this) mais le code plante AVANT à la méthode Close.

Je précise que mon code ne contient AUCUNE ressource non managée.

[Sphax]

Effectivement Sphax c'est une bonne remarque pour le GC.SuppressFinalize(this) mais le code plante AVANT à la méthode Close.

Je précise que mon code ne contient AUCUNE ressource non managée.

Ben donne nous le code de ton Close alors et éventuellement la ligne ou ça plante. Cela dit si tu n'as aucune ressource non managé, pourquoi tu as un finalizer ? Mais le code te ton Close répondra peut être à cette question .

[madfu]

Code :

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

1
2
 
Cela dit si tu n'as aucune ressource non managé, pourquoi tu as un finalizer ?

Je veux libérer toutes mes ressources dans tous les cas et surcharger la méthode Dispose est un bon moyen de le faire proprement je trouve, surtout qu'à terme il yaura à libérer aussi des ressources managées dans mon code.

Mon code contient deux méthodes très simples : Open qui renvoi un handle système et Close qui ferme le handle

Code :

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

1
2
3
4
5
6
7
8
9
10

public MyComponent:Component{
  public void Open(){
    _handle=DLLAPI.SessionOpen();
  }
  public void Close(){
    DLLAPI.SessionClose(_handle);//PLANTE !!!!!
  }
}

Cependant je penche vers un problème de thread spécifique à ma DLL : Je ne suis pas sûr que ma DLL accepte d'ouvrir une session dans un thread et de le fermer depuis un autre (ce qui peut être le cas lorsque le finalizeur est déclenché je crois)...à creuser. Je vais faire des tests en ce sens.