引发自定义事件时出错

Question

我有一个会写日志的课程。该类需要引发一个事件（在下面未说明的特定情况下），该事件将由一个类对其做出反应。我有下面的代码，但是一旦我尝试引发事件，我就会在指示的行上收到一个错误，即

对象引用未设置为对象的实例

知道我错过了什么吗？

//1. Class where event is registered
    public class LogEvent
    {
        public delegate void WriteLogEventHandler(object Sender, WriteLogEventArgs e);
        public event WriteLogEventHandler WriteLog;

        public class WriteLogEventArgs : EventArgs
        {
            public string Message { get; set; }

            public WriteLogEventArgs(string message) : base()
            {
                Message = message;
            }
        }

        //Raise the event.
        internal void OnWriteLog(WriteLogEventArgs e)
        {
             WriteLog(this, e);    //Error here.  Seems like WriteLog is null
        }

//2. Class where event is raised.
public class Logs
{
    public static void WriteLog(string message)
    {
        LogEvent.WriteLogEventArgs args = new LogEvent.WriteLogEventArgs(message);
        new LogEvent().OnWriteLog(args);
    }
}

//3. Class where event should be consumed
public class MyClass()
{
    private LogEvent _logEvent;
    public MyClass()
        {
            //Subscribe to event:
            _logEvent = new LogEvent();
            _logEvent.WriteLog += (sender, args) => { DoSomething(args.Message); };
        }

   public void DoSomething(string message)
   { ... }
}

Answer 1

两个问题：

• 无论是否有人订阅，您都在发起该事件。不要那样做 - 如果您在 WriteLog 为空时调用 WriteLog(this, e)，您将得到一个 NullReferenceException。在 C# 6 中很容易避免这种情况：

  WriteLog?.Invoke(this, e);
  

• 您在与引发事件的实例不同的LogEvent 实例上订阅事件。这更像是一个设计问题——单个日志事件拥有订阅者列表是没有意义的。相反，您应该有一个 Logger 或类似的有订阅者（通过事件），然后每个 LogEvent 被传递给这些订阅者。您将创建一个 Logger，订阅它，然后在同一个实例上调用WriteLog。