TableLayoutPanel中OfType<Button>移除按钮仅生效一次的问题

这是个很典型的集合遍历修改陷阱，我来帮你理清楚为什么会出现这种情况，以及怎么正确解决。

为什么一次循环只移除了Submit按钮？ #

当你用foreach (Control ctrl in panel.Controls.OfType<Button>())遍历的时候，OfType<Button>()返回的是一个实时枚举序列，而不是控件集合的副本。当你在循环里执行panel.Controls.Remove(ctrl)时，原控件集合的结构发生了变化，这会打乱枚举器的遍历状态——第一次移除Submit后，枚举器尝试获取下一个元素时，可能因为集合长度变化直接终止了遍历，导致Cancel按钮根本没被遍历到。

你说其他控件（Label、TextBox）一次循环就能移除，其实只是运气好而已——比如这些控件的枚举顺序刚好在移除第一个后，剩下的还能被枚举到，但这种写法本质上是不安全的，哪天控件数量或顺序变了，同样会出现漏删的情况。

而两次循环能生效，是因为第一次循环移除Submit后，第二次循环重新枚举剩下的Cancel按钮，这时候集合只有一个元素，自然能被移除，但这显然不是可靠的解决方案。

正确的移除方式 #

方法1：先把要移除的控件存入临时列表 #

先将所有需要移除的Button提取到一个独立的列表中，再遍历这个列表进行移除操作，这样就不会影响原控件集合的枚举：

private void RemoveExistingButtons(TableLayoutPanel panel)
{
    // 先把所有Button存入临时列表，避免遍历原集合时修改它
    var buttonsToRemove = panel.Controls.OfType<Button>().ToList();
    
    foreach (var btn in buttonsToRemove)
    {
        panel.Controls.Remove(btn);
        
        // 别忘了解除Form对按钮的引用，避免内存泄漏或意外行为
        if (btn == this.AcceptButton)
            this.AcceptButton = null;
        if (btn == this.CancelButton)
            this.CancelButton = null;
    }
}

方法2：倒序遍历原集合 #

从集合的最后一个元素往前遍历，这样移除元素时不会影响前面未遍历的元素索引：

private void RemoveExistingButtons(TableLayoutPanel panel)
{
    // 从后往前遍历，避免移除元素导致索引偏移
    for (int i = panel.Controls.Count - 1; i >= 0; i--)
    {
        if (panel.Controls[i] is Button btn)
        {
            panel.Controls.RemoveAt(i);
            
            // 同步解除AcceptButton/CancelButton的引用
            if (btn == this.AcceptButton)
                this.AcceptButton = null;
            if (btn == this.CancelButton)
                this.CancelButton = null;
        }
    }
}

额外注意点 #

一定要记得解除this.AcceptButton和this.CancelButton的引用——即使你把按钮从panel的Controls中移除了，Form仍然持有这些按钮的引用，可能导致内存泄漏，甚至在某些场景下触发意外的点击事件。

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