c# - 即使已设置，变量也会导致 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;
}

有 2 个方法同时挂接到事件，这可能是原因吗？如果是这样，我该如何避免呢？还是传递引用列表不是从 Hook 方法获取值的方式？

谢谢!

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

看，这是程序的一部分，它使用了通过反射加载的插件，并且有可能是我在调试之前忘记将更新的插件 dll 复制到插件目录的可能性很小。 ……呵呵。 ^^;对不起!但至少我的代码现在使用了最佳实践；P 非常感谢那个，我会把它标记为答案!

Answer 1

这里有多个问题。

传递 List<T>作为 ref 参数不是一个好方法。 List<T>可以在不涉及 ref 的情况下进行修改/out只需使用标准 Add/Remove该类型上已有的方法。

您正在为事件处理程序使用非标准形式。坚持EventHandler<TEventArgs>比较常规其中 TEventArgs是从 EventArgs 派生的某个类.从事件处理程序来回传递的数据应使用自定义属性或方法处理 EventArgs类。

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

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