MVC Helper SuppressFinalize

Version imprimable

Bonjour,

J'aimerai savoir pourquoi il faut appeler le GC.SuppressFinalize(this).

Tout les codes que je trouve l'utilise mais impossible pour moi de trouver pourquoi...

Une petit aide serai simpa ;)

Bonne journée

Code du helper MVC

Code:

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
 
namespace MvcPanelExample.Models.Html
{
    using System;
    using System.Diagnostics.CodeAnalysis;
    using System.Web;
 
    public class MvcPanel : IDisposable
    {
        #region Fields
 
        private bool disposed;
        private readonly HttpResponseBase httpResponse;
 
        #endregion
 
        #region CTOR
 
        public MvcPanel(HttpResponseBase httpResponse)
        {
            if (httpResponse == null)
            {
                throw new ArgumentNullException("httpResponse");
            }
            this.httpResponse = httpResponse;
        }
 
        #endregion
 
        #region Methods
 
        public void Dispose()
        {
            Dispose(true /* disposing */);
            GC.SuppressFinalize(this); /* <===== Here */
        }
 
        protected virtual void Dispose(bool disposing)
        {
            if (!this.disposed)
            {
                this.disposed = true;
                this.httpResponse.Write("</div>");
            }
        }
 
        public void EndPanel()
        {
            Dispose(true);
        }
 
        #endregion
    }
}

18/11/2011, 16h19
tomlev
Effectivement, dans le pattern "Dispose" standard, on appelle généralement SuppressFinalize dans le Dispose pour empêcher le finaliseur d'être exécuté à nouveau. Mais dans ton cas, ça ne sert à rien puisque tu ne définis pas de finaliseur...

En fait, quand on implémente IDisposable, on définit souvent un finaliseur (aussi appelé "destructeur") pour appeler Dispose avant que l'objet soit ramassé par le garbage collector (cf. cet article sur MSDN). Ca permet de s'assurer que l'objet sera disposé à un moment donné, même si Dispose n'a pas été appelé explicitement. Dans ton cas, ça donnerait quelque chose comme ça :
Code:

1 2 3 4 ~MvcPanel() { Dispose(false); }
Puisque le finaliseur ne fait rien d'autre qu'appeler Dispose, ça ne sert à rien de l'exécuter si l'objet a déjà été disposé, donc on indique au GC ne ne pas appeler le finaliseur sur cet objet.

En l'occurrence, je ne pense pas que tu aies besoin de définir un finaliseur ; en effet, si c'est le GC qui déclenche le Dispose, ce sera beaucoup trop tard pour écrire dans la HttpResponse, qui aura sans doute été fermée depuis longtemps...

Donc tu peux virer le GC.SuppressFinalize, puisque tu n'as pas de finaliseur.
21/11/2011, 09h19
elnfrancois

Merci beaucoup pour cette réponse

bonne journée