Утилизация объектов в Destructor

Question

У меня есть объект, который имеет одноразовый объект в качестве члена.

public class MyClass 
{ 
    private MyDisposableMember member; 

    public DoSomething 
    { 
     using (member = new MyDisposableMember()) 
     { 
      // Blah... 
     } 
    } 
} 

Там может быть много методов в MyClass, все требуя using заявление. Но что, если я это сделал?

public class MyClass 
{ 
    private MyDisposableMember member = new MyDisposableMember(); 

    public DoSomething 
    { 
     // Do things with member :) 
    } 

    ~MyClass() 
    { 
     member.Dispose(); 
    } 
} 

Как вы можете видеть, member в настоящее время расположен в деструкторе. Будет ли это работать? Есть ли проблемы с этим подходом?

Answer 1

В идеале, Dispose() должен был быть вызван до завершения. Лучше будет следовать типичному шаблону размещения и позволить пользователю правильно распоряжаться объектом() и иметь финализатор Утилизировать его , если dispose еще не был вызван.

В этом случае, поскольку вы инкапсулируете IDisposable, вам действительно не нужно реализовывать финализатор вообще. (В момент финализации ваш инкапсулированный член будет завершен, поэтому вам не нужно завершать свой объект - он просто добавляет служебные данные.) Подробнее см. Это blog article I wrote об инкапсуляции IDisposable.

Answer 2

Возможно, вы должны сделать MyClass орудие IDisposable. Внутри метода Dispose() вызовите member.Dispose();. Таким образом, программист может иметь контроль над тем, когда элемент будет удален.

Answer 3

Единственное, что я вижу неправильно (и это не ошибка) - это тот факт, что в используемом выражении вы явно распоряжаетесь объектом в этот момент времени (когда вы вызываете свою функцию/метод). Деструктор не может быть вызван, они вызываются автоматически. Поэтому на данный момент может потребоваться некоторое время для удаления члена. Лучше реализовать интерфейс IDisposeable для MyClass.

Answer 4

НЕ ДЕЛАЙТЕ ЭТО!

ГХ будет делать это за вас (косвенно, как объект распоряжаться или другой будет содержать деструктор)

MyDisposableMember может даже быть утилизированы ГХ еще до того, как выбросить его - то, что происходит, то не может быть что вы намеревались сделать.

Еще хуже: добавление деструктора (или финализатора) к классу требует дополнительного времени при утилизации объекта (гораздо больше времени, так как объект останется в памяти по меньшей мере для одного цикла сбора данных и, возможно, даже продвинут к следующему поколению).

Это было бы совершенно бесполезно и даже обратное.

Answer 5

Следуя шаблону Microsoft - ваш лучший выбор, так что пользователи вашего класса будут иметь полный контроль над тем, когда он будет удален.

public class MyClass : IDisposable 
{ 
    private MyDisposableMember member = new MyDisposableMember(); 

    public DoSomething 
    { 
     // Do things with member :) 
    } 

    ~MyClass() 
    { 
     Dispose(false); 
    } 

    public void Dispose() 
    { 
     Dispose(true); 
     GC.SuppressFinalize(this); 
    } 

    protected virtual void Dispose(bool disposing) 
    { 
     if (disposing) // Release managed resources 
     {   
      member.Dispose(); 
     } 

     // Release unmanaged resources 
    } 
} 

Answer 6

В первом примере элемент не является частью состояния объекта, так как вы инстанцировании это каждый раз, когда он используется и утилизации сразу после. Так как это не часть государства, не моделируйте его как таковое, просто используйте локальную переменную, когда это необходимо.

В более общем, вы должны поместить всю логику утилизации в Dispose() и реализовать IDisposable затем использовать вас класс вместе с использованием или примерка наконец