ReSharper/C#中的"委托减法有不可预测的结果"？

Question

使用myDelegate -= eventHandlerReSharper(版本6)时出现问题:

委托减法具有不可预测的结果

JetBrains在这里解释了这背后的理性.这个解释很有意义,在阅读之后,我怀疑我-对代表的所有用处.

那么如何,

• 我可以写一个非自动事件而不会让ReSharper脾气暴躁吗？
• 或者,是否有更好和/或"正确"的方式来实现这一点？
• 或者,我可以忽略ReSharper吗？

这是简化的代码:

public delegate void MyHandler (object sender);

MyHandler _myEvent;

public event MyHandler MyEvent
{
    add
    {
        _myEvent += value;
        DoSomethingElse();
    }
    remove
    {
        _myEvent -= value; // <-- ReSharper warning here
    }
}

Answer 1

别害怕!ReSharper警告的第一部分仅适用于删除代表列表.在您的代码中,您总是删除一个代理.第二部分讨论了删除重复委托后代理的排序.事件不保证其订阅者的执行顺序,因此它也不会对您产生任何影响.

由于上述机制可能导致不可预测的结果,因此ReSharper在遇到委托减法运算符时会发出警告.

ReSharper发出此警告是因为多播委托减法可能有问题,而不是完全谴责该语言功能.幸运的是,这些陷阱是在边缘情况下,如果你只是在处理简单的事件,你不太可能遇到它们.没有更好的方法来实现自己的add/ remove处理程序,你只需要注意.

我建议将ReSharper对该消息的警告级别降级为"Hint",这样你就不会对他们的警告不敏感,这通常很有用.

Answer 2

您不应该直接使用委托进行加总或减法.而是你的领域

MyHandler _myEvent;

应该被声明为事件.这将解决问题,而不会有您的解决方案的风险,仍然有事件使用的好处.

event MyHandler _myEvent;

使用委托sum或subtract是危险的,因为在简单地分配委托时你可能会丢失事件(根据声明,开发人员不会直接推断这是一个Multicast委托,因为它被声明为事件).举个例子,如果这个问题中提到的属性没有被标记为事件,下面的代码将使两个第一个赋值保持为LOST,因为有人只是分配给了委托(这也是有效的!).

myObject.MyEvent += Method1; 
myObject.MyEvent += Method2;
myObject.MyEvent = Method3;

分配Method3时,我完全丢失了两个初始订阅.事件使用将避免此问题,同时删除ReSharper警告.