Question

Here is my code:

foreach (OrderItem item in OrderInfo.order)
{
     orderItemViews.Single(i => i.numericUpDown.Name == item.id.ToString()).numericUpDown.Value = item.count;
}

It gives an exception.
I know that I can’t change the collection inside foreach
How can I change this code to make it work? Best of all if it would be LINQ code.

exception says that “The collection was modified”. sorry can’t provide real message of exception because it in non-english

sorry guys. I’ve found where collection is changing. It was inside *numericUpDown_ValueChanged* handler. anyway I’ve got an answer. thank you

Answer 1

You can use ToList(), Like this :

foreach (OrderItem item in OrderInfo.order.ToList())
{
     orderItemViews.Single(i => i.numericUpDown.Name == item.id.ToString()).numericUpDown.Value = item.count;
}

Or use normal for loop :

for (int i = 0 ; i < OrderInfo.order.Count; i++)
{
     OrderItem item = OrderInfo.order[i];
     orderItemViews.Single(i => i.numericUpDown.Name == item.id.ToString()).numericUpDown.Value = item.count;
}

Tip : Performance wise, It’s better to use the second way.