我最近一直在使用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);
}
}
}
}
不安全版本会导致程序崩溃。
答案 0 :(得分:9)
第一个版本是尝试使事件成为线程安全的。
请参阅C# Events and Thread Safety
实际上,正如在讨论中所说,它不会使事件线程安全。因此,我会使用更短的第二个版本。
编辑:事件线程安全真的很难实现。查看what it might look ...如果您实际上没有处理注册/取消注册事件的多个线程,则不应该浪费时间来保护线程安全。
答案 1 :(得分:0)
第二个版本不是线程安全的。
if (Initialized != null)
{
Initialized(this, new EventArgs());
}
如果最后一个处理程序在if (Initialized != null)
之后取消订阅,那么你将得到空引用异常。