Vous devez disposer des objets IDisposable avant de les repeupler?

Question

En supposant que j'ai une méthode dans mon modèle d'architecture de commande qui modifie le contenu du chemin graphique comme ceci: (GraphicsPath IDisposable)

(ce qui est purement un exemple non testé, rapide)

public void DoSomething(ref GraphicsPath path) 
{ 
    if(path != null) 
     { 
     List<PointF> pts = new List<PointF>(); 
     foreach(PointF pt in path.PathPoints) 
     { 
     //again, just a silly example. 
     float y = pt.X; 
     float x = pt.Y; 
     pts.Add(new PointF(x, y)); 
     } 
    path.Dispose(); //<-- Do I need this? 
    path = new GraphicsPath(pts.ToArray(), path.PathTypes); 
    } 
} 

Do I besoin de disposer le chemin avant de définir le chemin égal au nouveau chemin? Si oui, pourquoi?

Answer 1

Oui. Puisque vous transmettez la variable de chemin par référence, vous "orphelin" l'instance précédente lorsque vous la réattribuez. Nettoyer avec Dispose() est la bonne chose à faire.

Answer 2

Ouais, vous devriez probablement.

Si l'objet est jetable, cela signifie qu'il a des ressources à nettoyer (comme un fichier ouvert ou une base de données ou autre).

appel disposer, nettoie les vers le haut (dans un monde idéal)

Answer 3

Sans discuter le mérite de votre fonction, oui - vous devez appeler Dispose afin que l'objet peut se nettoyer des ressources dont elle a alloué le long du chemin . Une autre raison - au moment où vous lui attribuez quelque chose, vous assumez la responsabilité de sa durée de vie, vous devez donc vous assurer que tout ce qu'il utilisait avant est nettoyé.

Answer 4

Il s'agit d'une question difficile à répondre. Si le code consommateur doit être responsable de la gestion de la durée de vie de l'objet, il doit appeler Dispose() à la place de votre code. La raison est que le code pourrait être consommer maintenir une autre référence à l'instance, en plus de celui qu'il a passé à votre code, et il peut attendre de pouvoir continuer à utiliser cette référence après votre méthode a été appelée. Ou, si la sémantique de votre classe indique qu'elle gérera la durée de vie de l'objet une fois qu'il a été passé, alors il devrait appeler Dispose() comme vous l'avez décrit. En règle générale, chaque fois qu'un objet implémente IDisposable, vous devez toujours appeler explicitement sa méthode Dipose() lorsque vous en avez fini (ou utiliser la référence dans un bloc using). Après avoir dit ce qui précède, vous voudrez peut-être revoir votre design car la sémantique qu'il représente n'est pas intuitive.