正确实现 EventArgs Empty

Question

我正在尝试正确理解 Events 和 EventArgs，但无法完全了解整个 EventArgs.Empty 属性。

EventArgs 实现：

public static readonly EventArgs Empty;

并允许我们创建一个EventHandler 并使用以下方式调用它：

public event EventHandler<EventArgs> TestHappening;

private void MyMethod()
{
    TestHappening( this, EventArgs.Empty );
}

现在我研究了许多基于 EventArgs 的类，但似乎没有一个类实现了这一点，所以即使我已经阅读了我能找到的所有关于 EventArgs.Empty 的文档，我还是有点盲目。根据文档，“Empty 的值是 EventArgs 的只读实例，相当于调用 EventArgs 构造函数的结果”。

基于此，我创建了以下实现：

public class TestEventArgs : EventArgs
{
    public static readonly TestEventArgs Empty;

    public bool UpdatedValue { get; private set; }

    TestEventArgs()
        : this( false )
    {
    }

    public TestEventArgs( bool updatedValue )
    {
        this.UpdatedValue = updatedValue;
    }
}

public event EventHandler<TestEventArgs> TestHappening;

private void MyMethod()
{
    TestHappening( this, EventArgs.Empty );
}

使用TestEventArgs.Empty 是在实例化一个类还是它到底在做什么？

另外，即使我检查的所有子类都没有使用Empty，它们仍然可以使用它，暴露一个不可用的属性不是很混乱吗？

最后，根据我研究的各种文档，实际实例化EventArgs 的时间有两个主要差异。哪个被认为“更”正确？：

OnTestHappening( new TestEventArgs( false ) );

private void OnTestHappening( TestEventArgs e )
{
    var handler = TestHappening;

    if ( handler != null )
        handler( this, e );
}

对

OnTestHappening( true );

private void OnTestHappening( bool foo )
{
    var handler = TestHappening;

    if ( handler != null )
        handler( this, new TestEventArgs( foo ) );
}

Answer 1

如果您真的需要 Empty 字段，您应该自己考虑。如果您不使用它，则不应创建它。 EventArgs 类没有任何变量或属性，因此每次都创建一个新实例没有意义。在你的情况下，因为你有一个默认值，所以有两个“空”TestEventArgs 更有意义，一个用于true，一个用于false（如果你真的想要并且它在你的场景中是有意义的)。

您在实现中遗漏了我在下面修复的其他一些要点：

public class TestEventArgs : EventArgs
{
    public static readonly TestEventArgs True = new TestEventArgs(true);

    public static readonly TestEventArgs False = new TestEventArgs(false);

    public bool UpdatedValue { get; private set; }

    public TestEventArgs(bool updatedValue)
    {
        this.UpdatedValue = updatedValue;
    }

    public event EventHandler<TestEventArgs> TestHappening;

    private void MyMethod()
    {
        EventHandler<TestEventArgs> eh = TestHappening;

        eh?.Invoke(this, TestEventArgs.True);
    }
}

我改变了什么：

1. 将Empty 实例化为new TestEventArgs，因为这也是EventArgs.Empty 的定义。
2. 我已经实现了事件处理程序的线程安全版本（代码中的第二个示例）。如果订阅事件的事件处理程序列表发生更改，则您的第一个示例不安全。您的第二个是，因此您应该选择该选项。

关于您的最后一点：这取决于您是否打算让调用代表更改对象（它传递一个实例与多个实例）。在您的情况下，由于实例无法更改，所以没那么重要。