c# - 移除动态控制

Question

我有一个带有动态添加控件的表单。当用户单击下一步时，必须删除所有动态控件。我的代码只删除了其中的一些。

例如 dtxt2 第一次没有被删除，但如果我第二次运行删除函数，它确实被删除了。

这是添加动态的函数:

void TextboxesStep21()
{
    removeDynamics();
     TextBox dtxt1 = new TextBox();
     dtxt1.Name = "DynamicTextBox1";
     dtxt1.Location = new System.Drawing.Point(336, 125);
     dtxt1.Text = ((diameterOfDrivingWeel + spindleSlope)*10).ToString();
     this.Controls.Add(dtxt1);
     
    TextBox dtxt2 = new TextBox();
    dtxt2.Name = "DynamicTextBox2";
    dtxt2.Location = new System.Drawing.Point(336, 148);
    dtxt2.Text = gearingRatioDen.ToString();
    this.Controls.Add(dtxt2);

    TextBox dtxt3 = new TextBox();
    dtxt3.Name = "DynamicTextBox3";
    dtxt3.Text = gearingRatioNum.ToString(); ;
    dtxt3.Location = new System.Drawing.Point(336, 171);
    this.Controls.Add(dtxt3);


    pictureBox1.SendToBack();
    pictureBox2.SendToBack();
}

这是我的删除函数

void removeDynamics()
{
    foreach (Control x in this.Controls)
    {
        if (x.Name.Contains("Dynamic"))
        {
            this.Controls.Remove(x);
            x.Dispose(); 
        }
    }
}

Answer 1

for example dtxt2 does not get removed the first time but if I run the remove function a second time it does get removed.
... if you have time could you further explain what my fault was?

当你遍历一个集合时，你在后台有一个迭代器，它保存当前位置并根据集合的长度进行检查。所以如果集合有 3 个项目，迭代器在位置 2

[X1] [X2] [X3]  // Length 3
      ^
      |
   iterator

现在你删除第二个元素

[X1] [X3]  // Length 2
      ^
      |
   iterator

迭代器仍然有它的值，但现在它指向下一个元素，不幸的是这是集合中的最后一个。现在循环将递增迭代器并检查它是否已经大于集合中的项目数量。如果是这样，它将取消循环，因为没有更多的项目可以迭代。因此 X3 永远不会被处理

[X1] [X3]      // Length 2
           ^
           |
        iterator

这就是为什么您的第二个文本框首先被跳过的原因。

在迭代过程中从集合中删除项目时，可以使用反向循环。从末尾开始向下到 0:

void removeDynamics()
{
    for (int i = this.Controls.Count - 1; i >= 0; i--)
    {
        Control x = this.Controls[i];
        if (x.Name.Contains("Dynamic"))
        {
            this.Controls.Remove(x);
            x.Dispose();
        }
    }
}

当然你也可以过滤控件集合，首先只获取匹配的项，然后删除它们:

List<TextBox> toBeRemoved = this.Controls.OfType<TextBox>()
                                         .Where(x => x.Name.Contains("Dynamic")).ToList();
toBeRemoved.ForEach(x => { this.Controls.Remove(x); x.Dispose(); });