С# все еще привязан к событию после отключения

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

// Simple class to host the Event
class Test
{
  public event EventHandler MyEvent;
}

// Two different methods which will be wired to the Event
static void MyEventHandler1(object sender, EventArgs e)
{
  throw new NotImplementedException();
}

static void MyEventHandler2(object sender, EventArgs e)
{
  throw new NotImplementedException();
}


[STAThread]
static void Main(string[] args)
{
  Test t = new Test();
  t.MyEvent += new EventHandler(MyEventHandler1);
  t.MyEvent += new EventHandler(MyEventHandler2); 

  // Break here before removing the event handler and inspect t.MyEvent

  t.MyEvent -= new EventHandler(MyEventHandler1);      
  t.MyEvent -= new EventHandler(MyEventHandler1);  // Note this is again MyEventHandler1    
}

Если вы прерветесь до удаления обработчика событий, вы можете просмотреть список вызовов в отладчике. См. ниже, есть 2 обработчика, один для MyEventHandler1, а другой для метода MyEventHandler2.

Теперь, после удаления MyEventHandler1 дважды, MyEventHandler2 по-прежнему зарегистрирован, поскольку остался только один делегат, он выглядит немного иначе, он больше не отображается в списке, но пока делегат для MyEventHandler2 не будет удален, на него все равно будет ссылаться событие .

При отсоединении события он должен быть тем же делегатом. Как это:

public class Foo
{
     private MyDelegate Foo = ClientEntityCache_CacheCleared;
     public void WorkItem()
     {
         ClientEntityCache.EntityCacheCleared += Foo;
     }

     public void Dispose()
     {
         ClientEntityCache.EntityCacheCleared -= Foo;
     }
}

Причина в том, что вы используете для этого синтаксический сахар:

public class Foo
{
     public void WorkItem()
     {
         ClientEntityCache.EntityCacheCleared +=
new MyDelegate(ClientEntityCache_CacheCleared);
     }

     public void Dispose()
     {
         ClientEntityCache.EntityCacheCleared -=
new MyDelegate(ClientEntityCache_CacheCleared);
     }
}

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

Вы отцепляете правильную ссылку? Когда вы отцепляете с помощью -=, ошибки не возникает, и если вы отцепляете события, которые не перехвачены, ничего не произойдет. Однако, если вы добавите с помощью +=, вы получите сообщение об ошибке, если событие уже перехвачено. Это всего лишь способ диагностировать проблему, но вместо этого попробуйте добавить события, и если вы НЕ получите сообщение об ошибке, проблема заключается в том, что вы отцепили событие с неправильной ссылкой.

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

Если вы сами вызвали свой метод Dispose, ссылки выйдут за рамки.

Возможно, попробуйте:

 public void Dispose()
    {
      ClientEntityCache.EntityCacheCleared -= ClientEntityCache_CacheCleared;
      // Same thing for 10 other events
    }

Вы создаете новый обработчик событий и удаляете его из delegate, который фактически ничего не делает.

Удалите подписку на событие, удалив ссылку на исходный метод события подписки.

Вы всегда можете просто установить eventhandler = delegate {}; На мой взгляд, это будет лучше, чем null.