SonarLint 规则 S1172“应删除未使用的方法参数”和 EventHandlers

Question

当我创建 EventHandler 方法时，我应该如何修复 SonarLint 规则 S1172“应删除未使用的方法参数”。

public void Subscribe()
{
    MyEvent += OnMyEvent;
}

public void UnSubscribe()
{
    MyEvent -= OnMyEvent;
}

private void OnMyEvent(object sender, EventArgs e)
{
   DoSomething();
}

您可以使用 Reactive Extensions 重写代码并制作“Observables”，但这对于简单的事件处理程序来说是相当复杂的解决方案。另一种选择可能是重写代码，如：

public void Subscribe()
{
   MyEvent += (s,e) => DoSomething();
}

但接下来的问题是你如何处理UnSubscribe()？我认为未使用的参数不适用于事件处理程序方法。但在 SonarLint 中可能很难检测到。

Answer 1

如果您需要退订，您需要store the delegate（删除静态以获得正确的代码，这是从被黑的控制台应用程序项目中粘贴的）：

public static event EventHandler TestEvent;

private static EventHandler saved = (s, e) => DoSomething();

static void Main(string[] args)
{
    TestEvent += saved;
    TestEvent -= saved;
}

internal static void DoSomething()
{
}

或者使用a mass-unsubscribe:

foreach (Delegate d in TestEvent.GetInvocationList())
{
    TestEvent -= (EventHandler)d;
}

或者如果您拥有该活动，您也可以使用它来取消订阅所有：

TestEvent = null;

或者只使用您一直使用的语法并创建一个非匿名方法，如上所示。这种语法没有错。你可以做强制性的

if (sender == null) 
    throw ArgumentNullException(nameof(sender));

摆脱警告；）