Как удалить элементы из общего списка при переборах по нему?

Проследите элементы, которые будут удалены со свойством и удалите их всех после процесса.

using System.Linq;

List<MyProperty> _Group = new List<MyProperty>();
// ... add elements

bool cond = true;
foreach (MyProperty currObj in _Group)
{
    if (cond) 
    {
        // SET - element can be deleted
        currObj.REMOVE_ME = true;
    }
}
// RESET
_Group.RemoveAll(r => r.REMOVE_ME);

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

ArrayList place_holder = new ArrayList();
place_holder.Add("1");
place_holder.Add("2");
place_holder.Add("3");
place_holder.Add("4");

for(int i = place_holder.Count-1; i>= 0; i--){
    if(i>= place_holder.Count){
        i = place_holder.Count-1; 
    }

// some method that removes multiple elements here
}

Я сделал бы как это

using System.IO;
using System;
using System.Collections.Generic;

class Author
    {
        public string Firstname;
        public string Lastname;
        public int no;
    }

class Program
{
    private static bool isEven(int i) 
    { 
        return ((i % 2) == 0); 
    } 

    static void Main()
    {    
        var authorsList = new List<Author>()
        {
            new Author{ Firstname = "Bob", Lastname = "Smith", no = 2 },
            new Author{ Firstname = "Fred", Lastname = "Jones", no = 3 },
            new Author{ Firstname = "Brian", Lastname = "Brains", no = 4 },
            new Author{ Firstname = "Billy", Lastname = "TheKid", no = 1 }
        };

        authorsList.RemoveAll(item => isEven(item.no));

        foreach(var auth in authorsList)
        {
            Console.WriteLine(auth.Firstname + " " + auth.Lastname);
        }
    }
}

, ПРОИЗВОДИТ

Fred Jones
Billy TheKid

Я бы переназначил список из запроса LINQ, который отфильтровывал элементы, которые вы не хотели оставлять .

Использование ToArray () в общем списке позволяет вам удалить (элемент) в вашем общем списке:

        List<String> strings = new List<string>() { "a", "b", "c", "d" };
        foreach (string s in strings.ToArray())
        {
            if (s == "b")
                strings.Remove(s);
        }

Выберите элементы, которые вы делать нужно вместо того, чтобы пытаться удалить элементы, которые не нужны . Это намного проще (и, как правило, более эффективно), чем удаление элементов.

var newSequence = (from el in list
                   where el.Something || el.AnotherThing < 0
                   select el);

Я хотел опубликовать это как комментарий в ответ на комментарий, оставленный Майклом Диллоном ниже, но он слишком длинный и, вероятно, будет полезен в моем ответе:

Лично я бы никогда не удалял элементы один за другим. -one, если требуется удаление, вызовите RemoveAll , который принимает предикат и только один раз переупорядочивает внутренний массив, тогда как Remove выполняет операцию Array.Copy . для каждого удаляемого вами элемента. RemoveAll намного эффективнее.

И когда вы перебираете список в обратном направлении, у вас уже есть индекс элемента, который вы хотите удалить, поэтому было бы намного эффективнее вызвать RemoveAt , потому что Remove сначала выполняет обход списка, чтобы найти индекс элемента, который вы пытаетесь удалить, но вы уже знаете этот индекс.

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

поэтому было бы гораздо эффективнее вызвать RemoveAt , потому что Remove сначала выполняет обход списка, чтобы найти индекс элемента, который вы пытаетесь удалить, но вы уже знаете этот индекс.

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

поэтому было бы гораздо эффективнее вызвать RemoveAt , потому что Remove сначала выполняет обход списка, чтобы найти индекс элемента, который вы пытаетесь удалить, но вы уже знаете этот индекс.

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

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

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

Простое и понятное решение:

Используйте стандартный цикл for, выполняющий в обратном направлении в вашей коллекции и RemoveAt (i) , чтобы удалить элементы .