变量导致NullReferenceException,即使它已设置

Question

也许我正在做一些非常错误的事情,但不知何故,这段代码总是崩溃并出现NullReferenceException:

public class IncomingMessageEventData : EventArgs
{
    public IncomingMessageEventData(SpecialClasses.IncomingMessageData _msg, List<string> _toreturn)
    {
        msg = _msg;
        ToReturn = _toreturn;
    }
    public SpecialClasses.IncomingMessageData msg { get; set; }
    public List<string> ToReturn { get; set; }
}
public delegate void IncomingMessageHook(IncomingMessageEventData Args);
public event IncomingMessageHook InComingMessage;
public string NewMessage(string[] _message, System.Net.IPEndPoint RemoteIP)
{
    if (InComingMessage != null)
    {
        IncomingMessageEventData data = new IncomingMessageEventData(new SpecialClasses.IncomingMessageData(_message, RemoteIP), new List<string>());
        string ToReturn = "";
        InComingMessage(data);
        foreach (var item in data.ToReturn)
        {
if (item.Length > 0)
    ToReturn = item;
        }
        return ToReturn;
    }
    else return null;
}

有两种方法同时挂钩事件,这可能是原因吗？如果是这样,我该如何避免呢？或者传递一个ref List只是不是从钩子方法获取值的方法？

谢谢!

编辑:更新代码.哪个有效!......我想我知道我做错了什么.

看,这个程序是使用通过Reflection加载的插件的一部分.在调试之前我可能有一点点忘记将更新的插件dll复制到插件目录... 呵呵.^^; 抱歉! 但至少我的代码现在使用最佳实践; P非常感谢那个,我会把它标记为答案!

Answer 1

这里有很多问题.

传递a List<T>作为ref参数不是一个好方法.在List<T>已经可以不涉及修改ref/ out只使用标准Add/ Remove对类型已经可用的方法.

您正在使用非标准表单来处理事件处理程序.坚持某些类派生的EventHandler<TEventArgs>地方更为常规.要从事件处理程序来回传递的数据应使用自定义类上的属性或方法来处理.TEventArgsEventArgsEventArgs

您的事件处理程序逻辑不是线程安全的.您需要捕获事件处理程序的本地副本,以便考虑在执行空检查后有人取消订阅的情况.这是典型的模式:

// Capture the handler in a local
EventHandler<MyEventArgs> handler = this.MyEvent;
if (handler != null)
{
    // Invoke using the local copy
    handler(this, new MyEventArgs(/* ... */));
}