C# почему List.Remove() прерывает 'for loop'?

Question

Я действительно не знаю, почему цикл for только петли один раз. Я думаю, что логика правильная, ниже - код.

// the list is named 'dataList' 

for(int i = 0 ; i < dataList.Count; i ++) 
{ 
    string[] data = dataList[i].Split('+'); 
    string[] wsno = data[0].Split(':'); 
    if(wsno[1].Equals(tbWorkSheet.Text)) 
    { 
     dataList.Remove(dataList[i]) <<<< remove string that has the same number 
     //data removed 
     //for loop ends up here idk why.. 
    } 
} 

dataList.Count будет размером с список.

дело такое. Я хочу удалить несколько строк, хранящихся в списке, каждая строка имеет свой номер группы. Итак, цикл for будет зацикливаться до последнего. но когда он найдет строку с тем же номером с нужной, она выполнит оператор if, который должен удалить его.

Answer 1

Это очень плохая идея. Удалив значения из вашего списка, вы перепрыгнете через следующий элемент в своем списке.

Например у вас есть 3 пунктов в списке с индексом 0, 1, 2.

Первая итерация: я = 0 Вы удаляете элемент с индексом 0. Список теперь элементы с индексом 0, 1 (кулак элемент удален).

Вторая итерация: i = 1. Обратите внимание, как элемент, расположенный в индексе 1, перемещается в 0. Пока наш индексный счетчик переходит в 1 от 0. Таким образом, мы «пропустим» этот элемент.

Если у вас есть только 2 элемента в своем списке, он сломается после первого удаления. Это потому, что счет уменьшается, а ваш индекс увеличивается.

Обращаясь в обратном порядке, вы устраните эту проблему.

List<int> list = new List<int> { 2, 1 }; 
for (int i = list.Count - 1; i >= 0; i--) 
{ 
    list.RemoveAt(i); 
} 

Edit: Как Ролинг упоминает вы можете идти вперед, если вы уменьшить счетчик при удалении элементов.

for (int i = 0; i < list.Count; i++) 
{ 
    list.RemoveAt(i--); 
} 

Answer 2

Вы не можете изменить список в цикле for. Вы можете сохранить данные, которые хотите удалить в другом списке, а затем удалить их после выхода из цикла.

Answer 3

Ты забыл уменьшить i после удаления объекта. Это пропустит следующий пункт, поскольку все последующие позиции будут перенумерованы на 1.

Answer 4

Loop назад по списку:

for(int i = dataList.Count; i >= 0 ; i--) 
{ 
    string[] data = dataList[i].Split('+'); 
    string[] wsno = data[0].Split(':'); 
    if(wsno[1].Equals(tbWorkSheet.Text)) 
    { 
     dataList.RemoveAt(dataList[i]) <<<< remove string that has the same number 
     //data removed 
     //for loop ends up here idk why.. 
    } 
} 

Answer 5

Вы можете использовать List.RemoveAll универсальный метод. Метод удаляет из списка все элементы, соответствующие предикату. Метод подписи следующее:

INT List.RemoveAll (Predicate матч)

возвращает количество элементов, которые были удалены.

См MSDN: http://msdn.microsoft.com/en-US/library/wdka673a(v=vs.110).aspx

Answer 6

Я предпочел бы сделать новый список, а не изменяя оригинальную коллекцию.

Может быть сделано аккуратно с помощью LINQ:

dataList = 
    dataList 
     .Select(x => new{ 
      dataItem = x, 
      secondWsno = x.Split('+').First().Split(':').Skip(1).First() 
     }) 
     .Where(x => !x.secondWsno.Equals(tbWorkSheet.Text)) 
     .ToList();