.NET：每次事件触发时创建新的 EventArgs 是一个好的做法吗？

Question

例如，我有一个基本事件发布方法：

    protected virtual OnSomeEvent(EventArgs e)
    {
        var handler = SomeEvent;
        if (handler != null)
        {
            handler(this, e);
            // handler(this, new EventArgs());// EDIT: Yes it should be
                                           // handler(this, e),
                                           // ignore this one :D
        }
    }

对于重写 OnSomeEvent 并在触发时引发附加事件的派生类：

    protected override OnSomeEvent(EventArgs e)
    {
        base.OnSomeEvent(e);

        if (ExtendedEvent != null)
        {
            OnExtendedEvent(e);
        }
    }

    protected void OnExtendedEvent(EventArgs e)
    {
       // some stuff done
       // new information the ExtendedEventArgs object needs 
       //  is not available until this point

       ExtendedEvent(this, new ExtendedEventArgs(someStuff, someOtherStuff));
    }

如果派生继续这样进行，它将为每一代派生类都需要它。然而，.NET 框架上的 EventArgs 的各种派生似乎并未被设计为可变的（无 setter），这会阻止对象保留 EventArgs 的单个实例并随时修改它。

因此，每次触发此类事件时，它都会为所有涉及的 EventArgs 对象重新分配内存。在图形密集型应用程序中，事件每秒可以触发数十次（例如控件上的 OnPaint 事件），这真的是一个好的做法吗？

我是否应该对 OnExtendedEvent() 进行一些更改并使 ExtendedEventArgs 可变，以便可以执行以下操作？

    protected ExtendedEventArgs extendedArgs = ExtendedEventArgs.Empty;
    protected void OnExtendedEvent(EventArgs e)
    {
       // some stuff done
       // new information the ExtendedEventArgs object needs 
       //  is not available until this point

       extendedArgs.someProperty1 = someStuff;
       extendedArgs.someProperty2 = someOtherStuff;

       ExtendedEvent(this, extendedArgs);
    }

编辑：修复了示例代码，现在应该更清楚了。

Answer 1

首先，如果您只是忽略它，为什么要在您的触发方法中使用 EventArgs 参数呢？这是真正的浪费，但资源消耗比你的方法告诉调用者的谎言问题要小。只要传递参数，您的触发方法可能无论如何都不会具有可访问的相关信息来创建 EventArgs 对象：

protected virtual OnSomeEvent(EventArgs e)
{
    var handler = SomeEvent;
    if (handler != null)
    {
        handler(this, e);
    }
}

因此，现在我们已经知道了，如果您的 EventArgs 对象没有任何有意义的信息可以告诉您的订阅者，只需使用 EventArgs。空的，这就是它存在的意义。您可以对自定义 EventArgs 类遵循相同的模式，但老实说，您无需担心任何事情。创建 EventArgs 对象永远不会成为应用程序的瓶颈，如果是，则说明存在设计问题。

Answer 2

每次触发时我都会创建一个新的不可变对象，因为事件参数中有值。

主要原因是如果在处理现有事件时再次触发新事件会发生什么？

这可能会发生在多线程应用程序中，但甚至可能发生在单个线程上，如以下示例所示：

第一个事件使用以下值触发：

extendedArgs.someProperty1 = "Fire 1";
extendedArgs.someProperty2 = "Fire 1 Other Stuff";

然后，第一个事件处理程序执行某些操作，导致事件再次触发以下参数：

extendedArgs.someProperty1 = "Fire 2";
extendedArgs.someProperty2 = "Fire 2 Other Stuff";

第二个事件的所有事件处理程序都已处理，现在我们返回处理第一个事件的其余事件处理程序。

现在，由于使用了相同的对象，第一个事件的所有事件处理程序现在都将“Fire 2”作为其 someProperty1，因为第二个事件覆盖了这些值。

正如 @nobugz 提到的，不要害怕创建短暂的垃圾。

Answer 3

我对您的 OnExtendedEvent 代码有点困惑 - 您是否打算将事件作为 SomeEvent 重新分派？

当客户端添加事件处理程序时，他们希望能够在处理事件时删除事件处理程序，如下所示：

someObject.SomeEvent += OnSomeEvent;
// ...
private void OnSomeEvent(object sender, EventArgs e)
{
    someObject.SomeEvent -= OnSomeEvent;
}

如果不遵循标准调度实践，此代码将抛出异常，这会让人们大吃一惊使用你的代码。