求助：动态添加带参数的Button点击事件后如何正确移除

问题根源 #

你用lambda表达式包装带参数的ButtonClick方法添加事件时，每次循环都会生成一个全新的委托实例。而你移除时尝试用c.Click -= ButtonClick，一是ButtonClick的签名（带额外string参数）和Click事件要求的EventHandler签名不匹配，二是就算签名匹配，你要移除的委托实例和之前添加的不是同一个，自然移除失败。

有效解决方案：保存委托实例 #

核心思路是把每个按钮对应的委托实例存起来，移除时直接用保存的实例去匹配。

1. 先定义一个字典用来存储按钮和对应的委托：

// 放在窗体类的字段里
private Dictionary<Button, EventHandler> _buttonClickHandlers = new Dictionary<Button, EventHandler>();

2. 修改添加事件的代码，把委托实例保存到字典：

foreach (Control c in Controls)
{
    if (c is Button btn && btn.Name.Length == 7)
    {
        btn.Enabled = true;
        // 提前提取参数，避免闭包捕获循环变量的问题
        string buttonParam = btn.Name.Substring(btn.Name.Length - 1, 1);
        // 创建委托实例并保存
        EventHandler clickHandler = (sender, e) => ButtonClick(sender, e, buttonParam);
        btn.Click += clickHandler;
        _buttonClickHandlers[btn] = clickHandler;
    }
}

注意：这里提前把buttonParam提取出来，避免闭包捕获循环变量btn导致的参数错误问题。

3. 移除事件时，从字典取出对应的委托实例：

foreach (Control c in Controls)
{
    if (c is Button btn && btn.Name.Length == 7 && _buttonClickHandlers.ContainsKey(btn))
    {
        btn.Click -= _buttonClickHandlers[btn];
        // 移除后从字典删除，避免内存占用
        _buttonClickHandlers.Remove(btn);
    }
}

其他注意事项 #

• 如果按钮会被动态删除，记得同步从字典中移除对应的委托，防止内存泄漏。
• 不要直接用匿名lambda表达式添加事件，除非你确定不需要移除它——匿名委托无法直接追踪和移除。

内容的提问来源于stack exchange，提问作者PKYNI