C#提升事件

时间:2012-02-17 14:46:29

标签: c# events

我最近一直在使用C#工作,我注意到大多数代码都会在我公司的代码中引发事件:

EventHandler handler = Initialized;

if (handler != null)
{
    handler(this, new EventArgs());
}

我真的不明白为什么相反,你不能这样做:

if (Initialized != null)
{
    Initialized(this, new EventArgs());
}

编辑:

有些值得深思,我试着对此做一些测试:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading;

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            Test t = new Test(true);

            while(true)
            {
                t.Ev += new EventHandler(t_Ev);
                t.Ev -= new EventHandler(t_Ev);
            }
        }

        static void t_Ev(object sender, EventArgs e)
        {
        }
    }

    public class Test
    {
        private readonly bool m_safe;

        public Test(bool safe)
        {
            m_safe = safe;

            Thread t = new Thread(Go);
           t.Start();
        }

        private void Go()
        {
            while (true)
            {
                if(m_safe)
                {
                    RaiseSafe();
                }
                else
                {
                    RaiseUnsafe();
                }
            }
        }

        public event EventHandler Ev;

        public void RaiseUnsafe()
        {
            if(Ev != null)
            {
                Ev(this, EventArgs.Empty);
            }
        }

        public void RaiseSafe()
        {
            EventHandler del = Ev;

            if (del != null)
            {
                del(this, EventArgs.Empty);
            }
        }
    }
}

不安全版本会导致程序崩溃。

2 个答案:

答案 0 :(得分:9)

第一个版本是尝试使事件成为线程安全的

请参阅C# Events and Thread Safety

实际上,正如在讨论中所说,它不会使事件线程安全。因此,我会使用更短的第二个版本。

编辑:事件线程安全真的很难实现。查看what it might look ...如果您实际上没有处理注册/取消注册事件的多个线程,则不应该浪费时间来保护线程安全。

答案 1 :(得分:0)

第二个版本不是线程安全的。

if (Initialized != null)
{
    Initialized(this, new EventArgs());
}

如果最后一个处理程序在if (Initialized != null)之后取消订阅,那么你将得到空引用异常。