Question

我有这段代码：

       foreach (var item in ListView1.Items)
            {
                ListView1.Items.Remove(item);
                ListView21.Items.Add(item);
            }

循环停在一半的项目上？

有什么想法吗？

修改

好吧，也许这是我的错误，我需要澄清这是来自Infrajistics的UltraListView控件，我无法将项目添加到另一个列表，除非我将其删除或从原始列表中克隆它。

但是，谢谢，大多数关于不修改循环内列表的注释是正确的，所以这段代码有效：

           foreach (var item in listView1.Items)
            {
                var i = item.Clone(true);
                listView2.Items.Add(i);
            }
            listView1.Items.Clear();

谢谢，

Answer 1

您无法修改迭代集合，它应该以异常（或未定义的行为）消亡。尝试制作数组的副本：

   foreach (var item in ListView1.Items.ToArray())
   {
       ListView1.Items.Remove(item);
       ListView21.Items.Add(item);
   }

编辑：

实际上，您的示例代码可以通过编写：

来实现

ListView21.Items.AddRange(ListView1.Items);
ListView1.Items.Clear();

（实际上并不完全是你在做什么，但给出相同的结果，我认为它不会打扰你在两个列表视图中暂时拥有相同的内容）。从.NET2.0开始支持后者，第一个解决方案需要linq，因此需要.NET3.5。

Answer 2

您正在修改正在循环的集合。尝试从上到下使用for语句（从索引最高的项目到0）。

for (int i = ListView1.Items.Count - 1; i >= 0; i--)
{
    var item = ListView1.Items[i];
    ListView1.Items.Remove(item);
    ListView21.Items.Insert(0, item);
}

Answer 3

它会导致运行时异常，抱怨在迭代它时无法修改集合。您必须改为使用for循环。

for(int index = Items.Count; index > 0; index--)
{
    .......
    // use Add and RemoveAt
}

编辑：正如其他人所说。如果您只需要将项目从一个集合移动到另一个集合。 AddRange和Clear会更好。

Answer 4

您是否收到任何异常或错误消息？在一个集合中循环并从同一个集合中删除项目总是一个坏主意。

Answer 5

这看起来像WinForms列表视图控件，所以：

ListViewItem[] items = ListView1.Items.ToArray();
ListView1.Items.Clear();
ListView21.Items.AddRange(items);

Answer 6

为什么不CopyTo()到新列表然后Clear()项？

Answer 7

您正在遍历所有项目，删除所有项目，然后将它们添加到另一个列表中。正如其他人所评论的那样，您无法从for-each中的列表中删除项目。为什么不循环遍历所有项目以将它们添加到其他列表中，然后一次性删除它们？

foreach (var item in ListView1.Items)
{
    ListView21.Items.Add(item);
}
ListView1.Items.Clear();   // remove all

PS：这是一个ASP.NET列表视图还是WinForms列表视图？

Answer 8

那是因为你正在改变循环内的集合。使用正常for循环，如下所示：

for(int i=0; i < ListView1.Items.Count-1; i++)
{
       ListView21.Items.Add(ListView1.Items[i]);
       ListView1.Items.RemoveAt(i);
}

Foreach没有经过所有项目？

8 个答案: