C# this.Controls.Remove 问题

Question

这段代码有什么问题？

for (int w = 0; w < this.Controls.Count; w++)
{
    if (this.Controls[w] is TransparentLabel)
    {
        la = (TransparentLabel)this.Controls[w];
        if (la.Name != "label1")
        {
            la.Visible = false;
            la.Click -= new System.EventHandler(Clicked);
            this.Controls.Remove(this.Controls[w]);
            la.Dispose();
         }
     }
}

我想清除屏幕上的标签，但它不起作用。

Answer 1

将 for 更改为：

for (int w = this.Controls.Count - 1; w >= 0; w--)

否则，您可能会收到有关修改控件的错误。否则，如果这没有帮助，并且控件在屏幕上，那么它将与您的 if 语句评估有关。调试将有助于解决这个问题。

Answer 2

我认为代码没有删除所有预期的控件？这是因为您要从 Control 集合中删除一个项目，然后递增 w。

您应该在 this.Controls.Remove(...); 之后调用 w--;

如果您不调用 w--;删除该控件后，您将跨过取代索引 w 处的控件的控件。

还要补充一点，您真的需要调用以下内容吗？

la.Visible = false;
la.Click -= new System.EventHandler(Clicked);
la.Dispose();

当您删除控件时，它将变得不可见并且无论如何都无法单击。如果您不重新添加它，它将超出范围并被 GC 收集。

为了满足批评者的要求，正确的做法应该是通过 ControlCollection 向后工作。 Brian 在他的答案中对此进行了介绍。

Answer 3

怀疑 CF 是否支持 LINQ，因此您可以执行下一步：

this.Controls
  .OfType<TransparentLabel>()
  .Where(c => c.Name != "label1")
  .ToList()
  .ForEach(c => this.Controls.Remove(c));

Answer 4

ctrl.Visible = false;

它解决了我遇到的同样的问题。呈现页面时没有 HTML 输出。