я должен заблокировать «событие»?

Question

я должен заблокировать событие в следующем случае:

event foo;

поток A: вызовет foo + = обработчик;

поток B: вызовет foo - = обработчик;

я должен заблокировать foo?

Answer 1

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

private readonly object eventLock = new object();
private EventHandler fooHandler;

public event EventHandler Foo
{
    add
    {
        lock (eventLock)
        {
            fooHandler += value;
        }
    }
    remove
    {
        lock (eventLock)
        {
            fooHandler -= value;
        }
    }
}

private void OnFoo(EventArgs e)
{
    EventHandler handler;
    lock (eventLock)
    {
        handler = fooHandler;
    }
    if (handler != null)
    {
        handler(this, e);
    }
}

Обратите внимание, что если вы используете событие, похожее на поле, например:

public event EventHandler Foo;

тогда вы автоматически получите «блокировку (это)» при добавлении / удалении, хотя вам придется вручную добавлять его при извлечении обработчика перед его вызовом (при условии, что вы хотите удостовериться, что прочитали последнюю написанную значение). Лично я не фанат блокировок «this», но вы можете не возражать - и это, безусловно, упрощает код.