Предпочтительный способ объявить события

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

Когда я начинал помещать события в свои классы, я использовал стандартный способ:

public event EventHandler<MyEventArgs> MyEvent;

Это означало, что для подписки на событие потребуется такой метод:

void HandleThatEvent(object sender, MyEventArgs args){...}

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

Поэтому я перешел к объявлению своих собственных типов делегатов

public delegate void MyEventHandler(SomeClass argument);

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

eventImplmentor.MyEvent += HandleThatEvent;
.
.
.
void HandleThatEvent(/*oh, um, what arguments does it take? Intellisense isn't telling me*/)

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

Так что теперь вместо этого я просто использую Action, Action<T>или любой другой шаблон подходит.

public event Action<SomeClass> MyEvent;

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

Мой вопрос, после всего этого: есть ли лучшая практика для объявления событий в C #? Должен ли я вернуться на EventHandler<T>путь, или это Action<T>приемлемо?

Для простой внутренней обработки событий есть такие, которые просто используют Actionили Action<T>, как вы предлагаете. Я склонен использовать стандартный шаблон, в том числе Sender, даже для внутренних событий, потому что вы никогда не знаете, когда позже вы захотите выставить класс или событие, и я бы не хотел наказания за необходимость рефакторинга метода события просто для того, чтобы сделать это публично.

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

Для чего бы это ни стоило, я потратил некоторое время на это и придумал другой шаблон обработки событий : Подпись события в .NET - Использование строго отправленного типа «Отправитель»? , Цель здесь состояла не в том, чтобы удалить Отправителя, а в том, чтобы сделать его строго типизированным TSenderвместо «слабо типизированным» System.Object. Это работает очень хорошо; однако при этом вы теряете поддержку IntelliSense, поэтому возможен неудачный компромисс.

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