Question

我在Windows窗体应用程序中有一个方法，试图从面板中删除2个文本框。

在方法中，我遍历面板中的所有控件。总是应该将2个面板移除并添加在一起，但在移除时，当我按下按钮时，它会随机移除1个或2个容器。

以下是删除文本框的代码：

private void removeRows()
        {
            string descName = "Desc" + (textBoxCounter - 1).ToString();
            string costName = "Cost" + (textBoxCounter - 1).ToString();

            if (textBoxCounter >= 0)
            {
                foreach (Control c in costItems.Controls)
                {
                    if (c.Name == descName)
                    {
                        // Remove the control from the panel and dispose of it
                        panel.Controls.Remove(c);
                        c.Dispose();

                    }
                    if(c.Name == costName)
                    {
                        // Remove the control from the panel and dispose of it
                        panel.Controls.Remove(c);
                        c.Dispose();
                    }
                }

                // Decrement the counter
                // This happens only once since two controls need to be removed
                if (textBoxCounter == 0)
                    textBoxCounter = 0;
                else
                    textBoxCounter--;
            }
            else
                MessageBox.Show("There are no more rows to remove", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);

            testlabel1.Text = textBoxCounter.ToString();
            testlabel2.Text = panel.Controls.Count.ToString();
        }

以下是添加按钮的代码：

private void addRows(string desc, string cost)
    {
        if (textBoxCounter >= maxExpenses)
        {
            MessageBox.Show("Maximum number of expenses entered", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
        }
        else
        {
            TextBox Desc = new TextBox();
            TextBox Cost = new TextBox();

            // Give the text boxes names
            Desc.Name = "Desc" + textBoxCounter.ToString();
            Cost.Name = "Cost" + textBoxCounter.ToString();

            // Format the text boxes
            Desc.Width = panel.Width / 2;
            Cost.Width = panel.Width / 4;

            // Add the items to the costItems panel
            panel.Controls.Add(expenseDesc);
            panel.Controls.Add(expenseCost);

            // Add the items to the expenses dictionary
            panel.Add(Desc, Cost);

            // Increment the text box counter variable
            textBoxCounter++;
            testlabel1.Text = textBoxCounter.ToString();
            testlabel2.Text = costItems.Controls.Count.ToString();
        }
    }

要了解一些信息。将始终添加和删除2个文本框，它们彼此相关。 textBoxCounter初始化为0，因此前两个boxe名称将为“Desc0”和“Cost0”。

当我第一次按下按钮删除行时，会删除一个文本框，然后如果我再按一次它可能会删除2，它可能只删除1。

我尝试了调试，我注意到，遍历面板中所有控件的foreach循环似乎循环了一小段控件的全部数量。

对我的代码的任何帮助都会很棒。

Answer 1

您的问题是由foreach引起的，修改foreach中的集合可能会导致一些意外行为。您只想删除预先知道名称的TextBoxes ，那么为什么不使用方法ControlCollection.RemoveByKey？

如果您要删除上次添加的文本框（Desc...和Cost...），请执行以下操作：

panel.Controls.RemoveByKey(descName);
panel.Controls.RemoveByKey(costName);

如果你想删除所有添加的textBoxes（假设你有其他类型的TextBoxes，否则我们可以使用一点LINQ来轻松删除所有文本框）：

for(int i = 0; i < textBoxCounter; i++){
   panel.Controls.RemoveByKey("Desc" + i);
   panel.Controls.RemoveByKey("Cost" + i);
}

Answer 2

您的代码有两个问题：您正在处理无法处理的内容，并且您正在以错误的方式迭代一个集合（您正在修改它）。您可以执行以下操作删除所有Controls：

panel.Controls.Clear();

依靠索引向后迭代：

for (int i = panel.Controls.Count - 1; i >= 0; i--)
{
    panel.Controls.RemoveAt(i);
}

关于Dispose，您可以根据需要使用它，但不需要使用Remove：

for (int i = panel.Controls.Count - 1; i >= 0; i--)
{
    panel.Controls[i].Dispose();
}

PS：我问过与此相同的东西，得到-6。维护这个问题的原因之一正是对其他人有帮助（我看到你用来删除互联网控件的代码，我知道很多人都在使用它）。确实很讽刺。

从面板中删除文本框的方法

2 个答案: