Thực hiện IDisposable trên lớp kín

Question

Tôi không nghĩ câu hỏi này đã được hỏi trước đây. Tôi là một chút bối rối về cách tốt nhất để thực hiện IDisposable trên một lớp kín - đặc biệt, một lớp niêm phong mà không kế thừa từ một lớp cơ sở. (Đó là, một "lớp học kín tinh khiết" là thuật ngữ của tôi.)

Có thể một số bạn đồng ý với tôi rằng hướng dẫn thực hiện IDisposable rất khó hiểu. Điều đó nói rằng, tôi muốn biết rằng cách tôi dự định thực hiện IDisposable là đủ và an toàn.

Tôi đang thực hiện một số mã P/Invoke phân bổ IntPtr đến Marshal.AllocHGlobal và tự nhiên, tôi muốn xóa sạch bộ nhớ không được quản lý mà tôi đã tạo. Vì vậy, tôi đang nghĩ đến một cái gì đó như thế này

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(); 
    }  
} 

Tôi giả định rằng vì MemBlock là hoàn toàn kín và không bao giờ xuất phát từ một lớp khác mà thực hiện một virtual protected Dispose(bool disposing) là không cần thiết.

Ngoài ra, người hoàn tất có cần thiết không? Tất cả những suy nghĩ đều được chào đón.

Answer 1

Trình hoàn thiện là cần thiết làm cơ chế dự phòng để cuối cùng không có tài nguyên không được quản lý nếu bạn quên gọi Dispose.

Không, bạn không nên khai báo phương thức virtual trong lớp học sealed. Nó sẽ không biên dịch chút nào. Ngoài ra, bạn không nên khai báo các thành viên protected mới trong các lớp sealed mới.

Answer 2

Một bổ sung nhỏ; trong chung trường hợp, một mô hình phổ biến là có một phương pháp Dispose(bool disposing), để bạn biết liệu bạn đang ở trong Dispose (nơi mà mọi thứ hơn có sẵn) vs finalizer (nơi bạn không nên thực sự chạm vào bất kỳ đối tượng quản lý kết nối khác) .

Ví dụ:

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

Từ Joe Duffy's Weblog:

Đối với các lớp niêm phong, mô hình này cần không được tuân thủ, có nghĩa là bạn nên chỉ đơn giản là thực hiện finalizer của bạn và Vứt với các phương pháp đơn giản (ví dụ: ~ T() (Hoàn thành) và Vứt bỏ() trong C#). Khi chọn tuyến đường thứ hai, mã của bạn vẫn phải tuân thủ các nguyên tắc bên dưới về việc thực hiện quyết toán và xử lý logic.

Vì vậy, có, bạn nên tốt.

Bạn cần trình kết thúc như đã đề cập. Nếu bạn muốn tránh nó, bạn có thể xem SafeHandle. Tôi không có đủ kinh nghiệm với P/Invoke để đề xuất cách sử dụng chính xác.

Answer 4

Bạn không thể tuyên bố phương pháp ảo trong một lớp niêm phong. Việc khai báo các thành viên được bảo vệ trong một lớp được niêm phong sẽ cho bạn một cảnh báo trình biên dịch. Vì vậy, bạn đã triển khai chính xác. Calling GC.SuppressFinalize (this) từ bên trong finalizer là không cần thiết vì lý do hiển nhiên, nhưng nó không thể làm hại.

Có trình hoàn thiện là cần thiết khi giao dịch với tài nguyên không được quản lý, vì chúng không được giải phóng tự động, bạn phải làm điều đó trong trình hoàn thiện được gọi tự động sau khi đối tượng đã được thu thập rác.