Удаление обработки событий в деструкторов в случае как издатель и подписчик держать ссылки на друг друга

Question

Допустим, мы имеем следующую модель:

public class Bar<T>:List<T> 
    { 
     public delegate void CollectionChangedDelegate(); 
     public event CollectionChangedDelegate CollectionChanged; 
    } 

    public class Foo 
    { 
     Bar<object> MyCollection = new Bar<object>(); 

     public Foo() 
     { 
      MyCollection.CollectionChanged += new Bar<object>.CollectionChangedDelegate(MyCollection_CollectionChanged); 
     } 

     public void MyCollection_CollectionChanged() 
     { 
      //Do Something 
     } 

     ~Foo() //Would this work ? 
     { 
      MyCollection.CollectionChanged -= MyCollection_CollectionChanged; 
     } 
    } 

Может деструктор класса Foo называться в этом случае?

Answer 1

Это бессмысленно. Предположим, что есть такая подписка. Теперь, чтобы перейти к методу ~Foo, мы должны быть собраны, поэтому мы должны быть недоступны. Из-за того, как работают события, подписки делают нас доступными (у издателя есть ссылка на подписчика) - поэтому мы можем сделать вывод, что MyCollection is также недоступен. Если бы это было не так, нас не собирали.

И если MyCollection недоступен, то либо он уже собран, либо он находится около собран. Нет необходимости отписываться.

Удалить ~Foo: здесь бессмысленно; на самом деле, это хуже, чем бессмысленный - в дополнение к сервису нецелесообразной цели он заставляет сборщик мусора продвигать этот объект через дополнительный шаг (очередь финализатора).

Вы можете, однако, добавить какой-то детерминированный очистка, которая делает это; как отмечает Йорен, IDisposable может быть полезным - однако трудно сказать, подходит ли это в случае, не зная большего контекста.

Answer 2

Прежде всего, я считаю, что вы пытаетесь написать ObservableCollection
Здесь вы можете найти информацию: http://msdn.microsoft.com/en-us/library/ms668604.aspx.

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

class Foo:Idisposable 
{ 
    public void Dispose(bool b) 
    { 
     MyCollection.CollectionChanged -= MyCollection_CollectionChanged; 
    } 
}