VB.NET中AddHandler绑定按钮事件后，ColorDialog为何打开两次？

嘿，这个问题我太熟了——你遇到的是事件重复绑定的坑！每次调用OpenForm方法，都会给那8个按钮的Click事件再绑定一次cmbColor_Click处理程序，事件绑定是可以叠加的，所以绑定两次的话，点击一次按钮就会触发两次方法，弹窗自然就弹两次啦。

最常见的原因是你的OpenForm被调用了不止一次，比如在窗体的Load事件里不小心触发了多次，或者其他地方重复调用了这个方法。下面给你几个解决思路：

思路1：确保OpenForm只执行一次 #

先检查调用OpenForm的代码位置，比如是不是在窗体初始化时被多次触发，或者在其他逻辑里重复调用了。只要保证这个方法只跑一次，每个按钮的Click事件就只会绑定一次，问题就解决了。

思路2：绑定前先移除旧绑定（更稳妥） #

如果没法保证OpenForm只调用一次，那我们可以在添加新绑定前，先移除可能存在的旧绑定，避免重复叠加。修改你的OpenForm方法：

Public Sub OpenForm()
    ' 先移除已有绑定，再添加新绑定，避免重复
    RemoveHandler cmbColor1.Click, AddressOf cmbColor_Click
    AddHandler cmbColor1.Click, AddressOf cmbColor_Click
    
    RemoveHandler cmbColor2.Click, AddressOf cmbColor_Click
    AddHandler cmbColor2.Click, AddressOf cmbColor_Click
    
    RemoveHandler cmbColor3.Click, AddressOf cmbColor_Click
    AddHandler cmbColor3.Click, AddressOf cmbColor_Click
    
    RemoveHandler cmbColor4.Click, AddressOf cmbColor_Click
    AddHandler cmbColor4.Click, AddressOf cmbColor_Click
    
    RemoveHandler cmbColor5.Click, AddressOf cmbColor_Click
    AddHandler cmbColor5.Click, AddressOf cmbColor_Click
    
    RemoveHandler cmbColor6.Click, AddressOf cmbColor_Click
    AddHandler cmbColor6.Click, AddressOf cmbColor_Click
    
    RemoveHandler cmbColor7.Click, AddressOf cmbColor_Click
    AddHandler cmbColor7.Click, AddressOf cmbColor_Click
    
    RemoveHandler cmbColor8.Click, AddressOf cmbColor_Click
    AddHandler cmbColor8.Click, AddressOf cmbColor_Click
End Sub

这样哪怕OpenForm被多次调用，每次都会先清空之前的绑定，再重新绑定一次，就不会有重复触发的问题了。

思路3：批量处理按钮（优化代码） #

另外，为了让代码更简洁，你可以遍历窗体上的目标按钮来批量绑定，不用重复写8次相同的代码：

Public Sub OpenForm()
    ' 遍历窗体中所有以cmbColor开头的按钮
    For Each ctrl As Control In Me.Controls
        If TypeOf ctrl Is Button AndAlso ctrl.Name.StartsWith("cmbColor") Then
            Dim btn As Button = DirectCast(ctrl, Button)
            RemoveHandler btn.Click, AddressOf cmbColor_Click
            AddHandler btn.Click, AddressOf cmbColor_Click
        End If
    Next
End Sub

这样不管以后你加多少个类似的按钮，这段代码都能自动处理，维护起来也方便得多。

最后提醒一句：如果你的窗体是MDI子窗体，或者存在多次加载的场景，尽量把事件绑定放在窗体初始化后只执行一次的地方，比如窗体的Load事件（要确保Load只触发一次哦）。

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