Implementieren IDisposable auf einer versiegelten Klasse

Question

Ich glaube nicht, dass diese Frage zuvor gestellt wurde. Ich bin ein wenig verwirrt über die beste Möglichkeit, IDisposable auf einer versiegelten Klasse zu implementieren - speziell eine versiegelte Klasse, die nicht von einer Basisklasse erbt. (Das heißt, eine "reine versiegelte Klasse", die meine Bezeichnung ist.)

Vielleicht stimmen einige von Ihnen darin überein, dass die Richtlinien für die Implementierung IDisposable sehr verwirrend sind. Das heißt, ich möchte wissen, dass die Art, wie ich vorhabe, IDisposable zu implementieren, ausreichend und sicher ist.

Ich mache einige P/Invoke-Code, der eine IntPtr durch Marshal.AllocHGlobal zuweist und natürlich möchte ich den nicht verwalteten Speicher, den ich erstellt habe sauber zu entsorgen. So denke an Ich bin so etwas wie dies

using System.Runtime.InteropServices; 

[StructLayout(LayoutKind.Sequential)] 
public sealed class MemBlock : IDisposable 
{ 
    IntPtr ptr; 
    int length; 

    MemBlock(int size) 
    { 
      ptr = Marshal.AllocHGlobal(size); 
      length = size; 
    } 

    public void Dispose() 
    { 
      if (ptr != IntPtr.Zero) 
      { 
       Marshal.FreeHGlobal(ptr); 
       ptr = IntPtr.Zero; 
       GC.SuppressFinalize(this); 
      } 
    } 

    ~MemBlock() 
    { 
      Dispose(); 
    }  
} 

Ich gehe davon aus, dass, weil MemBlock vollständig abgedichtet ist und nie von einer anderen Klasse abgeleitet wird, dass ein virtual protected Dispose(bool disposing) Implementierung ist nicht erforderlich.

Ist der Finalizer auch unbedingt erforderlich? Alle Gedanken willkommen.

Answer 1

Der Finalizer ist als Fallback-Mechanismus erforderlich, um eventuell nicht verwaltete Ressourcen freizugeben, wenn Sie vergessen haben, Dispose anzurufen.

Nein, Sie sollten keine virtual Methode in einer sealed Klasse deklarieren. Es würde überhaupt nicht kompilieren. Es wird auch nicht empfohlen, neue protected Mitglieder in sealed Klassen zu deklarieren.

Answer 2

Ein kleiner Zusatz; in der allgemeine Fall, ein gemeinsames Muster ist eine Dispose(bool disposing) Methode, so dass Sie wissen, ob Sie in Dispose sind (wo mehr Dinge verfügbar sind) vs Finalizer (wo Sie wirklich keine anderen verbundenen verwalteten Objekte berühren sollten) .

Zum Beispiel:

public void Dispose() { Dispose(true); } 
~MemBlock() { Dispose(false); } 
void Dispose(bool disposing) { // would be protected virtual if not sealed 
    if(disposing) { // only run this logic when Dispose is called 
     GC.SuppressFinalize(this); 
     // and anything else that touches managed objects 
    } 
    if (ptr != IntPtr.Zero) { 
      Marshal.FreeHGlobal(ptr); 
      ptr = IntPtr.Zero; 
    } 
} 

Answer 3

Von Joe Duffy's Weblog:

Für versiegelte Klassen, dieses Muster müssen nicht gefolgt werden, das heißt, Sie sollten einfach Ihre Finalizer implementieren und Entsorgen mit den einfachen Methoden (zB ~ T() (Finalisieren) und Dispose() in C#). Wenn Sie die letztere Route wählen, sollte Ihr Code immer noch den folgenden Richtlinien in Bezug auf Implementierung der Finalisierung und Dispose Logik entsprechen.

Also ja, du solltest gut sein.

Sie brauchen den Finalizer wie Mehrdad erwähnt. Wenn Sie es vermeiden möchten, können Sie sich SafeHandle ansehen. Ich habe nicht genug Erfahrung mit P/Invoke, um die korrekte Verwendung vorzuschlagen.

Answer 4

Sie können virtuelle Methoden in einer versiegelten Klasse nicht deklarieren. Wenn Sie auch geschützte Member in einer versiegelten Klasse deklarieren, erhalten Sie eine Compilerwarnung. Du hast es also richtig implementiert. Der Aufruf von GC.SuppressFinalize (this) aus dem Finalizer ist aus offensichtlichen Gründen nicht erforderlich, kann aber nicht schaden.

Ein Finalizer ist wichtig, wenn es um nicht verwaltete Ressourcen geht, da sie nicht automatisch freigegeben werden. Sie müssen dies im Finalizer tun, wenn das Objekt automatisch aufgerufen wird, nachdem das Objekt gesammelt wurde.