检查委托是否为null

Question

我正在阅读Essential C#3.0书籍,我想知道这是否是检查代表为空的好方法？:

class Thermostat
{
    public delegate void TemperatureChangeHandler ( float newTemperature );

    public TemperatureChangeHandler OnTemperatureChange { get; set; }

    float currentTemperature;

    public float CurrentTemperature
    {
        get { return this.currentTemperature; }
        set
        {
            if ( currentTemperature != value )
            {
                currentTemperature = value;

                TemperatureChangeHandler handler = OnTemperatureChange;

                if ( handler != null )
                {
                    handler ( value );
                }
            }
        }
    }
}

如果类型是不可变的,解决方案是否会更改？我认为可能具有不变性,你不会遇到这个线程问题.

Answer 1

原始(有些不准确)回复:

关于这一点已经有很多讨论.

简而言之:即使执行此复制/检查null/execute步骤,也无法保证处理程序有效.

问题是,如果OnTemperatureChange在您复制它的时间和执行复制的时间之间取消注册,那么您可能无论如何都不希望执行侦听器.

您也可以这样做:

if (OnTemperatureChange != null )
{
    OnTemperatureChange ( value );
}

并处理空引用异常.

我有时会添加一个不执行任何操作的默认处理程序,只是为了防止空引用异常,但这会非常严重地增加性能影响,尤其是在没有注册其他处理程序的情况下.

更新2014-07-10:

我推荐Eric Lippert.

我的原始回复确实提到了使用默认处理程序,但我不建议使用临时变量,我现在也认为这是一个很好的做法,根据文章.

Answer 2

使用问号进行条件访问：

OnTemperatureChange?.Invoke();

Answer 3

在C. Ross的版本中推荐使用您给出的代码是有原因的。但是，约翰说的对，如果同时注销一个事件，还会有另一个问题。我链接的博客建议处理程序确保即使在未注册后也可以调用它们。