C # Action / Вопрос о стиле делегата
Что считается лучшим стилем для определения события:
public event Action<object, double> OnNumberChanged;
или же
public delegate void DNumberChanged(object sender, double number);
public event DNumberChanged OnNumberChanged;
Первый занимает меньше времени, но делегат дает имена параметрам. Когда я набираю это, я думаю, что номер 2 - победитель, но я могу ошибаться.
Изменить: другой (третий) подход является победителем. Читай ниже.
Ответы на вопрос(5)
Как и во всех вопросах о стиле кодирования. Выберите тот, который вы предпочитаете или который предпочитает ваша команда, и сохраняйте его последовательным на протяжении всего проекта. До тех пор, пока каждый, кто нуждается в этом, может прочитать его эффективно, у вас все будет хорошо.
Ни 1, ни 2. Третий вариант - победитель
public event EventHandler<NumberChangedEventArgs> NumberChanged;
Вы нарушаете ряд рекомендаций по стилю для разработки в C #, таких как использование типа для аргументов событий, который не расширяет EventArgs.
Да, вы можете сделать это таким образом, так как компилятору все равно. Однако люди, читающие ваш код, будут делать WTF.
Не создавайте новый тип, если вам это не нужно. Я думаю, что это лучше
public event Action<object, double> OnNumberChanged;
Причина в том, чтоAction а такжеFunc Семейства делегатов существуют для того, чтобы служить этой цели и уменьшить необходимость создания новых типов делегатов разработчиками.
Обычно я придерживаюсь использования производного класса EventArgs в качестве аргумента. Это делает код намного более последовательным.
У меня есть класс:
public class ApplyClickedEventArgs : EventArgs
{
...
}
и обработчик:
void cpy_ApplyClicked(object sender, ApplyClickedEventArgs e)
{
...
}
Декларация:
public event EventHandler<ApplyClickedEventArgs> ApplyClicked;
Я думаю, что вариант 1 лучше, если бы я выбрал, но IIRC, официальные правила для событий, утверждают, что ваш второй параметр должен быть класс с именемXxxEventArgsи должен иметьEventArgs в его цепочке наследования.