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

Question

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

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

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

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

您可以使用反应式扩展重写代码，并创建“可观察的”，但对于简单的事件处理程序来说，这是一个相当复杂的解决方案。另一种选择是重写代码，如下所示：

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

但接下来的问题是如何执行取消订阅（）？在我看来，未使用的参数不适用于事件处理程序方法。但在声纳探测中可能很难做到这一点。

Answer 1

如果您需要取消订阅，您需要存储委托（删除静态以获得正确的代码，这是从被黑客攻击的控制台应用程序项目中粘贴的）：

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()
{
}

或者使用大规模退订：

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

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

TestEvent = null;

或者只是使用你一直使用的语法，创建一个非匿名方法，就像你上面展示的那样。这种语法没有错。你可以做义务工作

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

为了摆脱警告；）