Почему кто-то должен быть подписан, чтобы событие произошло?

Что ж, каноническая форма такова:

void OnMadeSound()
{
    if (MadeSound != null)
    {
        MadeSound(this, new EventArgs());
    }
}

public void Fall() {  OnMadeSound(); }

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

Очень дзен, а?

Вы должны проверить значение null, если хотите вызвать событие:

protected void OnMyEvent()
{
    if (this.MyEvent != null) this.MyEvent(this, EventArgs.Empty);
}

Было бы неплохо, если бы вам не пришлось с этим возиться, но это перерыв.

Джеймс представил хорошее техническое обоснование, я также хотел бы добавить, что я видел, как люди использовали это преимущество: если ни один подписчик не слушает событие, они предпримут действия, чтобы зарегистрировать его в коде или что-то подобное. Простой пример, но подходящий в этом контексте.

Вам нужно понимать, что на самом деле делает ваше объявление события. Он объявляет как событие, так и переменную. Когда вы ссылаетесь на него внутри класса, вы просто ссылаетесь на переменную, которая будет иметь значение null, когда нет подписчиков.

Какой смысл создавать событие, если никто не слушает? Технически, это то, как C# решил реализовать это.

В C# событие - это делегат с некоторыми особыми перьями. Делегат в этом случае можно рассматривать как связанный список указателей функций (на методы-обработчики подписчиков). Когда вы «запускаете событие», каждый указатель функции вызывается по очереди. Первоначально делегат - это нулевой объект, как и все остальное. Когда вы выполняете + = для первого действия подписки, вызывается Delegate.Combine, который создает экземпляр списка. (Вызов null.Invoke () вызывает исключение null - когда событие запускается.)

Если вы все еще чувствуете, что «этого не должно быть», используйте вспомогательный класс EventsHelper, как упоминалось здесь, со старым и улучшенным «публикацией защитных событий» http://weblogs.asp.net/rosherove/articles/DefensiveEventPublishing.aspx.

Еще один хороший способ обойти это без необходимости проверять значение null:

class Tree
{
    public event EventHandler MadeSound = delegate {};

    public void Fall() { MadeSound(this, new EventArgs()); }

    static void Main(string[] args)
    {
        Tree oaky = new Tree();
        oaky.Fall();
    }
}

Обратите внимание на анонимный делегат - возможно, это небольшое снижение производительности, поэтому вам нужно выяснить, какой метод (проверка на null или пустой делегат) лучше всего работает в вашей ситуации.

Спасибо за ответы. Я понимаю, почему возникает исключение NullReferenceException и как его обойти.

Gishu said

What is the point of raising an event if no one is listening?

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

Возможно, лучше спросить: если поле делегата объявлено с ключевым словом события перед ним, почему компилятор не переводит все экземпляры:

MadeSound(this, EventArgs.Empty)

к

if (MadeSound != null) { MadeSound(this, EventArgs.Empty); }

за кулисами так же, как и другие сокращения синтаксиса? Количество шаблонных методов проверки нуля OnSomeEvent, которые люди должны писать вручную, должно быть колоссальным.

Рекомендуемый шаблон (.net 2.0+)

public class MyClass
{
    public event EventHandler<EventArgs> MyEvent; // the event

    // protected to allow subclasses to override what happens when event raised.
    protected virtual void OnMyEvent(object sender, EventArgs e)
    {
        // prevent race condition by copying reference locally
        EventHandler<EventArgs> localHandler = MyEvent;
        if (localHandler != null)
        {
            localHandler(sender, e);
        }
    }
    public void SomethingThatGeneratesEvent()
    {
        OnMyEvent(this, EventArgs.Empty);
    }
}

Я вижу много рекомендаций для пустого делегата {} в инициализаторе, но я полностью с этим не согласен. Если вы будете следовать приведенному выше шаблону, вы будете проверять event != null только в одном месте. Инициализатор пустого делегата {} - пустая трата времени, потому что это дополнительный вызов для каждого события, он тратит впустую память и все равно может завершиться ошибкой, если MyEvent был установлен в значение null в другом месте моего класса.

* Если ваш класс закрыт, вы бы не сделали OnMyEvent() виртуальным.

В этом случае было бы полезно использовать метод расширения.

public static class EventExtension
{
    public static void RaiseEvent<T>(this EventHandler<T> handler, object obj, T args) where T : EventArgs
    {
        if (handler != null)
        {
            handler(obj, args);
        }
    }
}

Затем его можно использовать, как показано ниже.

public event EventHandler<YourEventArgs> YourEvent;
...
YourEvent.RaiseEvent(this, new YourEventArgs());