правильный способ петлить через список и удалять элементы

Question

Я написал функцию, чтобы пройти через список и удалять элементы списка, если выполняются некоторые условия. Моя программа врезалась в нее, и через некоторое время я пришел к выводу, что внешний цикл for, проходит через все элементы в списке. В то же время список предметов может быть короче.

// Lijst is a list of a struct that contains a value .scanned and .price 

for (int i = 0; i < Lijst.Count; i++) 
    { 
    if (Lijst[i].scanned == false) 
    { 
    // (removed deletion of list item i here) 
    if (Lijst[i].price > (int)nudMinimum.Value) 
     { 
     Totaal++; 
     lblDebug.Text = Totaal.ToString(); 
     } 
    Lijst.RemoveAt(i); //<-moved to here 
    } 
    } 

Теперь я задаюсь вопросом, что правильно делать, не получая индекс из ошибок диапазона.

Answer 1

Вы могли бы искать для этого

for (int i = Lijst.Count - 1 ; i >= 0 ; i--) 
{ 
    if (Lijst[i].scanned == false) 
    { 
     if (Lijst[i].price > (int)nudMinimum.Value) 
     { 
      Totaal++; 
      lblDebug.Text = Totaal.ToString(); 
     } 
     Lijst.RemoveAt(i); 
    } 
} 

Вопрос в комментарии:

почему бы другое направление для работы контура?

Потому что, когда цикл запущен от нуля до графа Возникает ситуация, когда индекс недоступен для удаления, и счет все еще остается. Например:

Если у вас есть 10 элементов в списке, цикл начинается с 0 и будет удалять 0,1,2,3,4, а теперь левый элемент равен 5, а индекс также равен 5, это также удалит этот элемент , После этого, когда значение цикла достигает 6, а элемент слева равен 4. Тогда это создаст проблему. и это вызовет ошибку. т.е. индекс вне диапазона

Answer 2

List<string> list = new List<string>(); 
     list.Add("sasa"); 
     list.Add("sames"); 
     list.Add("samu"); 
     list.Add("james"); 
     for (int i = list.Count - 1; i >= 0; i--) 
     { 

      list.RemoveAt(i); 

     } 

How to Delete Items from List

Answer 3

Обычно, если вы хотите удалить из списка все элементы, которые соответствуют предикат, вы бы использовать List<T>.RemoveAll(), например:

List<int> test = Enumerable.Range(0, 10).ToList(); 

test.RemoveAll(value => value%2 == 0); // Remove all even numbers. 

Console.WriteLine(string.Join(", ", test)); 

Однако , кажется, вам нужно сделать некоторую дополнительную обработку. У вас есть два варианта:

1. Сделайте это в два шага; сначала используйте RemoveAll(), чтобы удалить ненужные элементы, затем перейдите по списку, чтобы обработать остальные предметы отдельно.
2. Петля назад от List.Count-1 до 0 вместо.

Answer 4

Проблема заключается в том, что при удалении элемента номер 5 список становится короче, а номер элемента 6 теперь равен 5, число 7 становится 6-м и т. Д. Однако, если вы запустите цикл назад, число будет сохранено как и ожидалось.

for(int i = donkeys.Count - 1; i >= 0; i++) 
    if(donkeys[i] == some condition here) 
    donkeys.RemoveAt(i); 

Однако, это подход, похожий на босса. Есть лучшие способы. У вас есть ответ, но я хотел бы предложить подход на основе LINQ.

int Totaal = Lijst 
    .Where(item => item.scanned) 
    .Where(item => item.price > (int)nudMinimum.Value) 
    .Count(); 

Lijst = Lijst.Where(item => !item.scanned).ToList() 

Кроме того, в качестве примечания стороны, интересно, находите ли вы ниже более читаемым. Рассмотрим следующее различное наименование (как относительно языка, так и капитализации).

List<Item> items = ...; 
int minimum = (int)nudMinimum.Value; 

int total = items 
    .Where(item => item.scanned) 
    .Where(item => item.price > minimum) 
    .Count(); 

items = items 
    .Where(item => !item.scanned) 
    .ToList(); 

Answer 5

Почему бы не написать List<T>.RemoveAll()?

https://msdn.microsoft.com/en-us/library/wdka673a(v=vs.110).aspx

В вашем случае

Lijst.RemoveAll(item => some condition); 

Э.Г.

// Count all the not scanned items each of them exceeds nudMinimum.Value 
    lblDebug.Text = Lijst 
    .Where(item => !item.scanned && item.price > (int)nudMinimum.Value) 
    .Count() 
    .ToString(); 

    // Remove all not scanned items 
    Lijst.RemoveAll(item => !item.scanned); 

Answer 6

Ваш код является некоторым, как не в надлежащем формате. сначала вы удалили элемент списка, а затем попытались поймать цену за удаленный элемент. Как это возможно.

поэтому вы можете написать таким образом.

for (int i = 0; i < Lijst.Count; i++) 
    { 
    if (Lijst[i].scanned == false) 
    { 
     if (Lijst[i].price > (int)nudMinimum.Value) 
     { 
     Totaal++; 
     lblDebug.Text = Totaal.ToString(); 
     } 
    Lijst.RemoveAt(i);   
    } 
    } 

Answer 7

здесь вы идете

// 1. Count items 
lblDebug.Text = Lijst.Count(x => x.price > (int)nudMinimum.Value && !x.scanned).ToString(); 
//2. Remove items 
Lijst.RemoveAll(x => !x.scanned); 

Answer 8

Сначала Вы удаляете элемент с индексом я, а затем использовать его. Сначала вы должны выполнить свой процесс с элементом, имеющим индекс i, а затем удалить его. Ваш код будет выглядеть следующим образом:

for (int i = 0; i < Lijst.Count; i++) 
    { 
    if (Lijst[i].scanned == false) 
    { 

    if (Lijst[i].price > (int)nudMinimum.Value) 
     { 
     Totaal++; 
     lblDebug.Text = Totaal.ToString(); 
     } 
Lijst.RemoveAt(i); 
    } 
    }