C# 类订阅自己发布的事件是不是很糟糕？

Question

我可能只是有点神经质，但我经常发现自己有一个发布事件的类，并且我发现从类本身（例如在构造函数中）订阅这个事件很方便，而不仅仅是订阅外部类。

这对我来说听起来很合理，但我不禁感到这是一种糟糕的做法，原因很简单，我总是面临这样一个问题：“为什么不执行你在触发事件的代码中的事件处理程序？”

public class Button
{
   public Button()
   {
      this.Click += someHandler; // bad practice?
   }

   public event EventHandler Click;

   public void HandleInput()
   {
      if (someInputCondition)
      {
         // Perform necessary actions here rather than 
         // subscribing in the constructor?
         this.Click(this, ...);
      }
   }
}

订阅自己的活动有什么缺点吗？

Answer 1

这对我来说听起来很合理，但我不禁感到这是一种糟糕的做法，原因很简单，我总是面临这样一个问题：“为什么不执行你在触发事件的代码中的事件处理程序？”

要回答这个问题，请考虑部分类场景。假设您有一个基类型 B。您运行一个自动化工具，该工具通过将 B 扩展为派生类 D 来装饰 B。您的工具会生成一个分部类，以便使用 D 的开发人员可以根据自己的目的进一步自定义它。

在这种情况下，当 B 声明的事件或 D 的机器生成端由 D 的机器生成端引发时，D 的用户编写端希望注册被调用似乎是完全合理的.

这是我们多年前设计 VSTO 时遇到的场景。事实证明，在 C# 中做到这一点并不难，但在 VB 中让它全部工作却相当棘手。我相信 VB 已经对其事件订阅模型进行了一些调整，以使其更容易。

也就是说：如果你能避免这种情况，我会的。如果您只是为内部订阅制作一个看起来很糟糕的代码气味的事件。 C# 3 中的部分方法在这方面有很大帮助，因为它们使机器生成端可以轻松且低成本地调用用户生成端的小通知函数，而不必费心发布事件。

Answer 2

我认为这没有问题。但是如果你在同一个类中处理事件，你也可以重写事件方法：

protected override void OnClick(Eventargs e)
{
   base.OnClick(e);
}

这更有效，并且让您能够在必要时吞下事件（只需不调用 base.OnClick()）。

Answer 3

由于内部优化，在执行此操作时有一个非常 exotic problem。由于优化添加/删除事件处理程序不是线程安全的。它仅适用于您的示例中声明类型使用的事件。

幸运的是，4.0 对此进行了更改，但如果您使用的是以前的版本，您可以体验到这一点。

Answer 4

问题是“someHandler”会改变你的对象的状态。您希望在事件运行任何“外部”代码之前或之后更改此状态吗？

目前尚不清楚如果您订阅该事件，状态会在什么时候发生变化，但是在“HandleInput()”中调用它可以更清楚地了解何时调用它。

（另外调用“HandleInput()”、“OnClick”并使其成为虚拟以便子类可以覆盖它更正常）

说了这么多，一般情况下订阅自己的事件也没什么大不了的；在表示表单的 UI 类中，这很常见，否则它往往会让很多阅读您的代码的人“感到惊讶”。

Answer 5

如果你的按钮类应该是第一个接收点击事件的，你应该在事件方法中编写你的代码，例如：

protected virtual void OnClick(EventArgs e)
{
    //insert your code here

    if(this.Click != null)
    {
        this.Click(this, e);
    }
}

但如果你的班级不是第一个接收者，你可以正常订阅事件。

Answer 6

如果以普通的 System.Windows.Form 类为例，
当您想处理 Form_Load 事件（使用 Visual Studio 设计器）时，它会被处理 在表单本身的类中！

this.Load += new System.EventHandler(this.Form1_Load);

 private void Form1_Load(object sender, EventArgs e)
 {
 }

所以我认为这根本不是问题！。