Реализовать удаление или завершение?

Question

Class ComponentsContainer ' a component contains other components' 
    Inherits System.ComponentModel.Component 

    Private foo as New Component 
    Private bar as New Component 

Protected Override Sub Finalize() 
    foo.Dispose() ' HERE ? ' 
    bar.Dispose() 
    MyBase.Finalize() 
End Sub 

Protected Overrides Sub Dispose(disposing As Boolean) 
    If disposing Then 
     foo.Dispose() ' OR HERE ? ' 
     bar.Dispose() 
    End If 
    MyBase.Dispose(disposing) 
End Sub 
End Class 

Answer 1

Вы не должны (не обязательно) Dispose управляемые ресурсы из Finalizer:

Protected Override Sub Finalize() 
    ' foo.Dispose() ' 
    ' bar.Dispose() ' 
    MyBase.Finalize() 
End Sub 

И из этого следует, что если ваш класс не имеет неуправляемых ресурсы вам не нужны Finalizer.

Примечание: в вашем классе отсутствует перегрузка Public Sub Dispose().

Edit:

С foo и bar управляются ресурсы (простирающиеся компонент) требуется только метод Protected Overrides Sub Dispose(disposing As Boolean). Версия в вопросе верна. И просто снимите Finalize().

Answer 2

Финализатор должен вызывать этот класс 'Dispose передать false для параметра размещения, а не прямо распоряжаться объектами, которыми владеет этот класс. См. MSDN.

Редактировать: Чтобы ответить на вопрос, удаление принадлежащих им объектов должно быть выполнено в Dispose, а не в Finalize.

Edit 2: Обратите внимание, это означает, что если объект будет завершен без утилизировать, то Dispose только дозвонился (по Finalize) «ложным» параметр, и дочерние объекты не будут захоронены этим класс. Это правильно, потому что они являются управляемыми объектами и будут завершены, когда среда будет выглядеть так, как если бы она не была явно удалена.

Answer 3

Dispose - это когда вы явно хотите освободить некоторые ресурсы, прежде чем сборщик мусора освободит объект.

Finalize автоматически вызывается, когда сборщик мусора обходит вокруг, чтобы освободить объект.

Если у вас есть много объектов, которые держатся за ресурсы, тогда, поскольку вы не должны контролировать сбор мусора, вы должны использовать Dispose.

Из документации рамочной:

Обратите внимание, что даже если вы предоставите явное управление посредством Dispose, вы должны обеспечить неявную очистку с использованием метода Finalize. Финализация обеспечивает резервную копию, чтобы предотвратить утечку ресурсов из-за постоянного утечки, если программист не может вызвать Dispose.

Implementing Finalize and Dispose to Clean Up Unmanaged Resources

Answer 4

Что вы наследующий? Я подозреваю, что это может быть System.ComponentModel.Container, прямо или косвенно.

В этом случае вам ничего не нужно делать. System.ComponentModel.Container automatically содержит любые компоненты, которые он содержит, в своем методе Dispose. Оставьте это в покое - это должен быть самый простой способ реализовать шаблон dispose/finalize.