Как изменить код и стоит ли, чтобы не было предупреждения CA1009 анализатора кода?

Question

[sergeyfk]

Имеется такой код, на который анализатор ругается:

public event Action<int> event_SetStatusStripProgressBar_Value;
        private void SetStatusStripProgressBar_Value(object obj)
        {
            if (event_SetStatusStripProgressBar_Value != null)
                event_SetStatusStripProgressBar_Value((int)obj);
        }

Действительно ли стоит переписать данный код? Если да, то как?

Answer 1

[lam0x86]

public sealed class ProgressChangedEventArgs : EventArgs
{
    public ProgressChangedEventArgs (int progress) { this.Progress = progress; }
    public int Progress { get; }
}

public event EventHandler<ProgressChangedEventArgs> StatusStripProgressChanged;
private void RaiseStatusStripProgressChanged(int progress)
{
    StatusStripProgressChanged?.Invoke(new ProgressChangedEventArgs(progress));
}

Comments

[cicatrix]

Вас отправляю туда же.
Не занимайтесь изобретением велосипедов.

Кроме того, чтобы предупреждение ушло, надо соблюдать сигнатуру event (object, EventArgs)
В первом параметре передаётся ссылка на объект, сгенерировавший событие, во втором - наследник от EventArgs, в котором передаются доп. параметры, если необходимо.

[lam0x86]

Писал без Студии, забыл написать this.
Что касается ProgressEventArgs - вы хоть обратили внимание, что этот класс находится в System.Management.dll? Вы всегда добавляете референс на сборку, если вам требуется использовать оттуда двухстрочный класс?
Если уж говорить про правильную имплементацию прогресс-бара, то надо использовать Progress<T>

[cicatrix]

lam0x86, Каюсь, про сборку не посмотрел. Под Winforms она обычно сразу установлена, а кроме как в Winforms этот класс не использовал.
Ваше решение подходит, разве что ссылку на инициирующий объект добавить.

Answer 2

[cicatrix]

Если у вас (или у вашего тимлида) нет ОКР, то можно и suppress сделать.
НО!

Тут при повторном рассмотрении углядел, что вы в obj передаёте не ссылку на объект, который событие отправляет, а какой-то параметр. В этом случае это действительно может ввести в заблуждение (как ввело меня только что, например, когда я подумал, что вы параметры не передаёте, а только ссылку на отправителя события). Я, не зная вашего кода, вижу событие и ОЖИДАЮ стандартного шаблона (особенно если вы используете тип object), и здесь мой шаблон "рвётся" :)

Во-первых, есть класс ProgressEventArgs, который служит ровно той цели, что вы здесь "свелосипедили".

Стиль написания кода - это, если уж честно, догма от Майкрософт, и вещь субъективная. Однако, нельзя считать эту догму плохой. Она задаёт общий шаблон для написания кода и позволяет потом легко свои решения масштабировать.

В общем, это ваше решение. Если код закрытый и никто, кроме вас его больше не увидит, и вы уверены, что и через несколько лет вы, просматривая этот фрагмент, будете чётко знать, что на самом деле в объекте obj типа object вы передаёте в событии целочисленный параметр состояния прогресса, то можете делать suppress этому кейсу.
Если же вы планируете публиковать этот код (это часть Open Source) проекта, то вердикт однозначный - переписывайте.

Comments

[sergeyfk]

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

Answer 3

[Александр Юдаков]

Объяснение здесь:
https://msdn.microsoft.com/ru-ru/library/ms182133.aspx