删除多余的委托构造函数调用？

Question

我下载了 ReSharper，它告诉我更改这一行：

dispMap.OnDraw += new EventHandler(dispMap_OnDraw);

更改为这一行：

dispMap.OnDraw += dispMap_OnDraw;

因为第一行是“冗余委托构造函数调用”。

这是真的吗？在自动生成的表单设计器代码中，语法基于第一段代码，当输入 dispMap.OnDraw += 并按 TAB 键时，IDE 会自动生成 new EventHandler(dispMap_OnDraw)< /code>

我只是对这个感到好奇。 ReSharper 有道理吗？

Answer 1

是的，这是正确的。我在几个案例中都这样做过。

委托构造函数调用应该是隐式的；该类型可以从 OnDraw 推断出来，并根据 dispMap_OnDraw 的方法签名进行验证。

此外，这篇 MSDN 文章 中的引用似乎是相关的：

因为 += 运算符只是连接内部调用
一个委托到另一个委托的列表，您可以使用 += 添加一个
匿名方法。请注意，对于匿名事件处理，您不能
使用 -= 运算符删除事件处理方法，除非
匿名方法被添加为处理程序，首先将其存储到
委托，然后向该事件注册该委托。

我相信无论哪种方式都会创建委托实例，但由于在隐式实例化时没有委托的对象引用，因此无法使用 -= 运算符将其删除。

Answer 2

这确实有道理。第二行是第一行的简写。根据您的编码标准/约定，您可以使用其中任何一个，但第一个确实会增加很多噪音。

Answer 3

如果您比较这两种情况下生成的 IL，您会发现它们是相同的。以下是 C# 中的两种情况以及它们产生的 IL。

示例 C#：

namespace EventTest
{
    public class Publisher
    {
        public delegate void SomeEvent(object sender);
        public event SomeEvent OnSomeEvent;
        public event SomeEvent OnOtherEvent;
    }

    public class Subscriber
    {
        public Subscriber(Publisher p)
        {
            p.OnSomeEvent += new Publisher.SomeEvent(Respond);
            p.OnOtherEvent += Respond;
        }

        public void Respond(object sender)
        {

        }
    }
}

这是构造函数的 IL。请注意 IL_000a 至 IL_0028 行。

.method public hidebysig specialname rtspecialname 
        instance void  .ctor(class EventTest.Publisher p) cil managed
{
  // Code size       48 (0x30)
  .maxstack  8
  IL_0000:  ldarg.0
  IL_0001:  call       instance void [mscorlib]System.Object::.ctor()
  IL_0006:  nop
  IL_0007:  nop
  IL_0008:  ldarg.1
  IL_0009:  ldarg.0
  IL_000a:  ldftn      instance void EventTest.Subscriber::Respond(object)
  IL_0010:  newobj     instance void EventTest.Publisher/SomeEvent::.ctor(object,
                                                                          native int)
  IL_0015:  callvirt   instance void EventTest.Publisher::add_OnSomeEvent(class EventTest.Publisher/SomeEvent)
  IL_001a:  nop
  IL_001b:  ldarg.1
  IL_001c:  ldarg.0
  IL_001d:  ldftn      instance void EventTest.Subscriber::Respond(object)
  IL_0023:  newobj     instance void EventTest.Publisher/SomeEvent::.ctor(object,
                                                                          native int)
  IL_0028:  callvirt   instance void EventTest.Publisher::add_OnOtherEvent(class EventTest.Publisher/SomeEvent)
  IL_002d:  nop
  IL_002e:  nop
  IL_002f:  ret
} 

结论：我认为没有任何理由更改您的代码，它们是等效的。

Answer 4

它工作得很好，我有 DevExpress，它也告诉我同样的事情！