c# - += delegate {} 是将自定义 eventargs 附加到 UI 控件事件的正确方法吗？

Question

我一直在修改事件，以便更好地了解它们在一般情况下的使用。我很惊讶地发现以下内容，所以我可能正走向错误的方向...我所做的本质是在单击按钮时将按钮更改为随机颜色:

Windows 窗体

public Form1()       
{ 

ColorChanges KK = new ColorChanges();

KK.ColorEventHandler += handle_ColorChanges;

button1.Click += delegate { KK.ChangeColor(button1); };

}

事件类

class ColorChanges
{

   *... properties & constructor*

   public void ChangeColor(object sender)
   {
   *... randomly assign color to ColorEventArgs*
   }

   protected virtual void onColorEvent(object sender, ColorEventArgs e)
   {
      EventHandler<ColorEventArgs> ceh = ColorEventHandler;
      {   
      if (ceh != null)
         {
         ceh(sender, e)             
         }
     }
   }

   public event EventHandler<ColorEventArgs> ColorEventHandler;
}

自定义事件参数

public class ColorEventArgs : EventArgs
{
  public Color xColor { get; set; }
}

事件处理程序

public void handle_ColorChanges(object sender, ColorEventArgs e)
{
   if (sender is Button)
      {
       var ButtonSender = (Button)sender;

       ButtonSender.BackColor = e.xColor;
       }
}

所以编辑的问题是:

使用 EventHandler(TEventArgs) 委托(delegate)有用吗？MS 文档表明像

这样的语法

  button1.Click += new EventHandler<AutoRndColorEventArgs>(handle_ColorChanges);

是正确的，但这不会到达我的代码来随机选择颜色和错误

"No overload for 'handle_ColorChanges' matches delegate >'System.EventHandler' "

所以类似

button1.Click += new EventHandler<AutoRndColorEventArgs>(KK.ChangeColor(button1));

或

button1.Click += new EventHandler(KK.ChangeColor(button1));

错误表明需要一个方法，如果我使用

"No overload for 'handle_ColorChanges' matches delegate 'System.EventHandler'"

Lambda 表达式有帮助感谢您的支持性答案

button1.Click += (sender,args) => KK.ChangeColor(s); 

但这不允许取消分配，稍后将需要...

匿名委托(delegate)也有同样的问题

button1.Click += delegate(object sender, EventArgs e)
            { KK.ChangeColor(sender); };

问题的关键是我的颜色方法或其委托(delegate)与按钮委托(delegate)签名(对象、事件)不匹配。我不关心按钮参数，想使用我自己的 如何？

Answer 1

Is the use of the delegate correct?

是的，您正在做的是分配一个 anonymous delegate 作为您的事件处理程序。这是完全有效的，但是，这里的问题是您无法取消分配事件处理程序，因为您没有对它的引用。您可以保留对它的引用并这样做(如果需要)

var clickHandler = delegate { ... };
button1.Click += clickHandler;
...
button1.Click -= clickHandler

如果您需要访问事件处理程序的参数，您需要将它们添加到签名中，例如

button1.Click += delegate (object sender, EventArgs args) { ... }

new EventHandler(SomeHandlerMethod) 构造是一种漫长的做事方式，它与 += SomeHandlerMethod 同义。您的代码当前不起作用，因为当构造函数需要对方法的引用时，您实际上正在尝试调用构造函数内的处理程序

+= new EventHandler<ColorEventArgs>(KK.ChangeColor);

Is there a better structure for this?

是的，您甚至可以使用更少的代码来做到这一点

button1.Click += (s, args) => KK.ChangeColor(button1);