Действительно ли вызов является дополнительным методом на “пустой” ссылке (т.е. событие без подписчиков) зло?

Можно всегда объявлять события как это (не, что я рекомендую его):

public event EventHandler<EventArgs> OnClicked = delegate { };

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

можно, вероятно, избавиться от ключевого слова делегата в C# 3.0...

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

мне это кажется одним из глюков C#, который вызывает ошибки, когда люди не знают / забывают проверять на пустой указатель каждый раз.

Сокрытие этой детали реализации кажется хорошим планом, поскольку это не помогает удобочитаемости проверить на пустые указатели каждый раз. Я уверен, что MSFTs скажет, что существует увеличение производительности в не constucting событие, если никто не слушает, но по моему скромному мнению это значительно перевешивается бессмысленными исключениями нулевого указателя / сокращение удобочитаемости в большей части бизнес-кода.

я также добавил бы эти два метода к классу:

    public static void Raise(this EventHandler handler, object sender)
    {
        Raise(handler, sender, EventArgs.Empty);
    }

    public static void Raise<TA>(this EventHandler<TA> handler, object sender, TA args)
        where TA : EventArgs
    {
        if (handler != null)
        {
            handler(sender, args);
        }
    }

Не забывайте использовать [MethodImpl(MethodImplOptions.NoInlining)], еще его возможное, что это не ориентировано на многопотоковое исполнение.

(Чтение, которое где-нибудь давно, помнило его, погугленный и нашло http://blog.quantumbitdesigns.com/tag/events/ )

Почему это было бы злым?

Его цель ясна: Это генерирует событие MyButtonClicked.

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

Это немного тривиально, но это фиксирует мою самую большую жалобу с C#.

В целом, я думаю, что это - фантастическая идея и вероятно украдет его.

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

protected virtual OnSomeEvent(EventArgs e){ }

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

Хотя я бы не назвал его злом , он все же имеет отрицательные последствия, поскольку добавляет ненужные накладные расходы:

При вызове

myEvent.Raise (this, new EventArgs () );

объект EventArgs инициализируется во всех ситуациях, даже если никто не подписался на myEvent.

При использовании

if (myEvent!= null) {
   myEvent(this, new EventArgs());
}

EventArgs инициализируется только в том случае, если кто-то подписался на myEvent.