代码分析CA2000:这段代码可以吗？

Question

看一下这段代码，我相信它解决了CA2000，但我想确保我没有忽略一些东西。基本上，这段代码根据在TreeView中选择的内容加载一个新的Control。然后显示该Control，并且该Node可见/可用，直到选择该TreeView中的另一个and为止。

private void Something(object sender, TreeViewEventArgs e)
{
    ProjectTreeNode node = (e.Node as ProjectTreeNode);

    foreach (Control c in optionsPlaceholderPanel.Controls)
        c.Dispose();

    optionsPlaceholderPanel.Controls.Clear();

    if (node != null)
    {
        //ProjectOptions inherits from Control and is therefore IDisposable
        ProjectOptions options = new ProjectOptions(node.Project);

        ShowOptionsPanel(options);
    }
}

private void ShowOptionsPanel(Control control)
{
    optionsPlaceholderPanel.Controls.Add(control);
    control.Dock = DockStyle.Fill;
}

因此，基本上Control始终在作用域中，直到加载了新的Control来代替它。当我这样做的时候，我正在处理之前加载的Control，所以我认为在这种情况下忽略CA2000是安全的。另外，当Form最终关闭并释放optionsPlaceholderPanel时，这也会释放子控件，对吧？

Answer 1

foreach (Control c in optionsPlaceholderPanel.Controls)
    c.Dispose();

不，这段代码有一个bug。它本身是由ControlCollection类中的错误触发的。foreach循环正在修改面板的Controls集合。这通常会产生一个InvalidOperationException，“集合已被修改，枚举操作可能无法执行”，但该类忘记了这样做。

对该控件的Dispose()调用将其从集合中移除。实际上，您将只处理其他所有控件。这应该有副作用，它们在面板上仍然可见。Ymmv.修复：

        for (int ix = optionsPlaceholderPanel.Controls.Count - 1; ix >= 0; --ix)
            optionsPlaceholderPanel.Controls[ix].Dispose();

或者效率较低，尽管您永远看不到其中的区别：

        while (optionsPlaceholderPanel.Controls.Count > 0)
             optionsPlaceholderPanel.Controls[0].Dispose();

否则代码就没问题，CA2000往往会产生错误的警告。