C# 委派没有被删除

Question

在程序中，我使用图片框创建了多个面板。图片框被委派为可点击。我希望用户可以选择从订单中删除其中一个面板/图片框。现在，如果它被删除并且订单被重新排列，那么在被删除的图片框保留其委托顺序之后的所有图片框。因此，在被删除的那个之后单击其中的任何一个，会跳到它旁边的那个（即单击#9，它将转到#10）。我需要删除重新排序的委托并正确地重新委托它们。我试过了：

int z2 = z;
var myClickDelegate = (EventHandler)delegate { clicked(z2, null); };
PicBx[z].Click += myClickDelegate;

创造和

PicBx[z].Click -= myClickDelegate;

删除

还有

int z2 = z;
PicBx[z].Click -= delegate { clicked(z2, null); };

但他们都没有删除原始委托。

Answer 1

您的第一种方法应该有效，但后一种方法不应该有效。
因为当您执行 PicBx[z].Click -= delegate { clicked(z2, null); }; 方法时，您并没有删除旧的委托，而是创建了一个新的委托，然后将其移除。

在您的第一次尝试中应该可以：

private void SomeMethod()
{
    var myClickDelegate = (EventHandler)delegate { clicked(z2, null); };
    PicBx[z].Click += myClickDelegat;
    //Do extra work
    PicBx[z].Click -= mayClickDelegat;
}

编辑：与您的评论配对： 我注意到您只是在您的委托中添加clicked(z2, null)，所以我假设您只是首先创建委托只是为了传递该int z2 来表示图片框索引。
您可以使用pictureBox.Tag 将该索引与图片框本身放在一起，并在点击事件中从标签中获取int：

int z2 = z;
picBx[z].Tag = z2;//here we embedded the number with the picture box.
PicBx[z].Click += clicked;
...
PicBx[z].Click -= clicked;

所以在点击事件中：

private void clicked(object sender, EventArgs e)
{
    PictureBox pictureBox = sender as PictureBox;

    if (pictureBox != null)
    {
        int number = Convert.ToInt32(pictureBox.Tag);
        ...
    }
}

Edit2：与您的 cmets 配对，您似乎拥有不同的 clicked 方法签名：

private void clicked(int tes,..
{
    Pnl[tes].BackColor = Color.Red;
}

这里我们只改成：

private void clicked(object sender, EventArgs e)
{
    PictureBox pictureBox = sender as PictureBox;//when user clicks on picture box it will be the sender parameter.

    if (pictureBox != null)
    {
        //we add number to each of picture boxes at there tags. "picBx[z].Tag = z2"
        int tes = Convert.ToInt32(pictureBox.Tag);
        pnl[tes].BackColor = Color.Red;
    }
}

Answer 2

您没有尝试删除同一个委托。您应该尝试将您的代表存储在某个地方：

this.MyDelegate = delegate { clicked(z2, null); };

PicBx[z].Click += this.MyDelegate;


...

...


PicBx[z].Click -= this.MyDelegate;

但是像这样玩代理可能会导致糟糕的设计选择。您应该定义一个bool，您将在其中确定是否执行您的委托。

public void OnClick(object sender, ...)
{
    if(myBool)
    {
        ...
    }
}

Answer 3

从您对 Jalal 的回答看来，您实际上可能需要这样的东西

How to remove all event handlers from a control

Answer 4

这里有两个问题。

首先，正如其他人所指出的，您需要删除添加到事件处理程序中的同一委托实例。

第二个是您使用数组来存储图片框，并且您的处理程序使用索引来查找图片框，因此当您从数组中删除图片框时，所有图片框的事件处理程序数组将指向错误的图片框。

这是怎么做的：

第 1 步：更改您的事件处理程序以获取图片框的实例，而不是数组的索引。

第 2 步：创建一个类级字典，以通过图片框保存您的委托引用：

Dictionary<PictureBox, EventHandler> _pictureBoxHandlers =
    new Dictionary<PictureBox, EventHandler>();

第 3 步：编写您的订阅代码，如下所示：

//After the picture box is added to `PicBx`
var pb = PicBx[z];
var myClickDelegate = (EventHandler)delegate { clicked(pb, null); };
_pictureBoxHandlers[pb] = myClickDelegate;
pb.Click += myClickDelegate;

第 4 步：编写您的退订代码，如下所示：

//Before the picture box is removed from `PicBx`
var pb = PicBx[z];
if (_pictureBoxHandlers.ContainsKey(pb))
{
    var myClickDelegate = _pictureBoxHandlers[pb];
    pb.Click -= myClickDelegate;
    _pictureBoxHandlers.Remove(pb);
}

看看情况如何。