C#类订阅自己发布的事件是否是一种糟糕的形式？
我可能只是有点神经质，但我经常发现自己处于这样的情况：我有一个发布事件的类，我发现从类本身（例如在构造函数中）订阅这个事件很方便，而不仅仅是从外部类订阅 对我来说这听起来很合理，但我还是忍不住感到这是一种糟糕的做法，原因很简单，我总是面临这样一个问题：“为什么不在触发事件的代码中执行事件处理程序中提供的操作？”C#类订阅自己发布的事件是否是一种糟糕的形式？,c#,events,C#,Events,我可能只是有点神经质，但我经常发现自己处于这样的情况：我有一个发布事件的类，我发现从类本身（例如在构造函数中）订阅这个事件很方便，而不仅仅是从外部类订阅 对我来说这听起来很合理，但我还是忍不住感到这是一种糟糕的做法，原因很简单，我总是面临这样一个问题：“为什么不在触发事件的代码中执行事件处理程序中提供的操作？” 订阅自己的活动有什么缺点吗？我看这没有问题。但如果处理同一类中的事件，也可以重写事件方法： protected override void OnClick(Eventargs e) {
订阅自己的活动有什么缺点吗？我看这没有问题。但如果处理同一类中的事件，也可以重写事件方法：
protected override void OnClick(Eventargs e)
{
base.OnClick(e);
}
这会更有效，并使您能够在必要时吞下事件（只需不调用base.OnClick（））。问题在于“someHandler”将更改对象的状态。是否希望在事件运行任何“外部”代码之前或之后更改此状态
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, ...);
}
}
}
幸运的是，4.0已经改变了这一点，但如果您使用的是以前的版本，您可能会体验到这一点 如果以普通System.Windows.Form类为例，
当您想要处理表单加载事件（使用visual studio designer）时，它会被处理 在课堂上的形式本身
this.Load += new System.EventHandler(this.Form1_Load);
private void Form1_Load(object sender, EventArgs e)
{
}
所以我认为这根本不是问题 如果您的button类应该是第一个接收click事件的类，那么您应该在event方法中编写代码，例如：
protected virtual void OnClick(EventArgs e)
{
//insert your code here
if(this.Click != null)
{
this.Click(this, e);
}
}
也就是说：如果你能避免这种情况，我会的。如果您只是为内部订阅制作一个看起来像是坏代码味道的事件。C#3中的部分方法在这方面有很大帮助，因为它们使机器生成端在用户生成端调用少量通知函数变得简单和低成本，而不必费事发布事件。+1这是一个很好的观点，但在承包商或UI控件中订阅时这不是一个问题（也是单螺纹的）@Ian：我认为很少有人会遇到这种微妙的错误，但是很少有人会遇到这种错误。如果你在类的ctor中订阅自己的事件，那么就不可能遇到线程问题。此外，它允许你指定你的代码是在任何其他事件处理程序之前运行还是在aft之前运行erwards（或者两者兼而有之）。在我的例子中，这是我自己的自定义按钮类——与该类名没有特定的相关性；它可以是任何东西。因此没有可重写的OnClick方法（除非我的C#事件知识中有一个漏洞）。@vargonian:那么我建议你创建一个（
protected virtual void OnLoad（EventArgs e）；按钮private voidboolOnLoadOnLoadOnLoad