如何使 Action与事件的委托类型兼容吗？

Question

给出下面的代码：

void LookupBox_Load(object sender, EventArgs e)
{
    Action d = delegate
        {
            if (!_p.AutoClose)
                CloseLookupBox();
        };

    if (this.ParentForm.MdiParent != null)
        this.ParentForm.MdiParent.Deactivate += delegate { d(); };
    else
        this.ParentForm.Deactivate += delegate { d(); };
}

有没有办法省略 delegate { d(); } ?

void LookupBox_Load(object sender, EventArgs e)
{
    Action<object,EventArgs> d = delegate
        {
            if (!_p.AutoClose)
                CloseLookupBox();
        };

    if (this.ParentForm.MdiParent != null)
        this.ParentForm.MdiParent.Deactivate += d;
    else
        this.ParentForm.Deactivate += d;
}

注意：我想内联执行此操作

Answer 1

绝对 - 更改 d 的类型以开始：

EventHandler d = delegate
    {
        if (!_p.AutoClose)
            CloseLookupBox();
    };

匿名方法不仅仅适用于 Func 和 Action...

为了将来的参考，您可以创建一个新的委托基于具有兼容签名的现有代码：

Action<object, EventArgs> d = ...;
EventHandler handler = new EventHandler(d);

但是在这种情况下，这种额外的间接寻址是没有意义的:)

您还可以使用空合并运算符使调用它的代码也稍微简单一些：

Form form = ParentForm.MdiParent ?? ParentForm;
form.Deactivate += d;

因为您只使用 d 一次，您可以内联它，将整个方法变成：

Form form = ParentForm.MdiParent ?? ParentForm;
form.Deactivate += delegate
{
    if (!_p.AutoClose)
        CloseLookupBox();
};

Answer 2

好不了多少，但如果您使用 C# 3.0，您可以这样做：

if (this.ParentForm.MdiParent != null)
    this.ParentForm.MdiParent.Deactivate += (x,y) => d();
else
    this.ParentForm.Deactivate += (x,y) => d();

Answer 3

您应该使用 EventHandler来定义这些而不是 Action代表

  EventHandler<EventArgs> d = delegate        
       {            
            if (!_p.AutoClose)                
               CloseLookupBox();        
       };