C# Почему мой оператор блокировки висит?

Question

Я следил за некоторыми учебниками о том, как создавать аксессоры событий custem. Это код, у меня есть:

event ControlNameChangeHandler IProcessBlock.OnControlNameChanged 
{ 
    add 
    { 
     lock (ControlNameChanged) 
     { 
      ControlNameChanged += value; 
     } 
    } 
    remove 
    { 
     lock (ControlNameChanged) 
     { 
      ControlNameChanged -= value; 
     } 
    } 
} 

На данный момент код достигает в надстройке statament lock(ControlNameChanged), ничего не происходит. Код больше не работает. Однако мое приложение все еще работает. Он не замерзает или что-то в этом роде.

Что пошло не так?

Answer 1

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

Поэтому я хотел бы использовать отдельный объект запирающий так:

private readonly object controlNameChangedSync = new object(); 

event ControlNameChangeHandler IProcessBlock.OnControlNameChanged 
{ 
    add 
    { 
    lock (controlNameChangedSync) 
    { 
     ControlNameChanged += value; 
    } 
    } 
    remove 
    { 
    lock (controlNameChangedSync) 
    { 
     ControlNameChanged -= value; 
    } 
    } 
} 

Примечание: ссылка на событие изменения при выполнении += или -= на делегата.

Answer 2

и -=изменение делегат. Таким образом, ваши методы «Добавить и удалять» блокируются на разных объектах каждый раз, и у вас вообще нет синхронизации.

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

Answer 3

Ваш код эквивалентен

event ControlNameChangeHandler IProcessBlock.OnControlNameChanged { 
    add { 
     try { 
      Monitor.Enter(ControlNameChanged); 
      ControlNameChanged = ControlNameChanged + value; 
     } 
     finally { 
      Monitor.Exit(ControlNameChanged); 
     } 
    } 
    remove { 
     try { 
      Monitor.Enter(ControlNameChanged); 
      ControlNameChanged = ControlNameChanged - value; 
     } 
     finally { 
      Monitor.Exit(ControlNameChanged); 
     } 
    } 
} 

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

private object padlock = new object(); 
event ControlNameChangeHandler IProcessBlock.OnControlNameChanged { 
    add { 
     lock (padlock) { 
      ControlNameChanged += value; 
     } 
    } 
    remove { 
     lock (padlock) { 
      ControlNameChanged -= value; 
     } 
    } 
}