Использование LINQ для удаления элементов из списка <T> - PullRequest
604 голосов
/ 12 мая 2009

Скажите, что у меня есть запрос LINQ, такой как:

var authors = from x in authorsList
              where x.firstname == "Bob"
              select x;

Учитывая, что authorsList имеет тип List<Author>, как я могу удалить элементы Author из authorsList, которые возвращаются запросом в authors?

Или, другими словами, как я могу удалить всех равных Бобу из authorsList?

Примечание. Это упрощенный пример для целей вопроса.

Ответы [ 15 ]

1040 голосов
/ 12 мая 2009

Ну, было бы проще исключить их в первую очередь:

authorsList = authorsList.Where(x => x.FirstName != "Bob").ToList();

Однако это просто изменит значение authorsList вместо удаления авторов из предыдущей коллекции. Кроме того, вы можете использовать RemoveAll:

authorsList.RemoveAll(x => x.FirstName == "Bob");

Если вам действительно нужно сделать это на основе другой коллекции, я бы использовал HashSet, RemoveAll и Contains:

var setToRemove = new HashSet<Author>(authors);
authorsList.RemoveAll(x => setToRemove.Contains(x));
122 голосов
/ 12 мая 2009

Было бы лучше использовать List .RemoveAll для достижения этой цели.

authorsList.RemoveAll((x) => x.firstname == "Bob");
43 голосов
/ 19 ноября 2010

Если вам действительно нужно удалить элементы, то как насчет Except ()?
Вы можете удалить на основе нового списка или удалить на лету, вложив Linq.

var authorsList = new List<Author>()
{
    new Author{ Firstname = "Bob", Lastname = "Smith" },
    new Author{ Firstname = "Fred", Lastname = "Jones" },
    new Author{ Firstname = "Brian", Lastname = "Brains" },
    new Author{ Firstname = "Billy", Lastname = "TheKid" }
};

var authors = authorsList.Where(a => a.Firstname == "Bob");
authorsList = authorsList.Except(authors).ToList();
authorsList = authorsList.Except(authorsList.Where(a=>a.Firstname=="Billy")).ToList();
26 голосов
/ 12 мая 2009

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

Но вы можете создать новый список и заменить старый.

var authorsList = GetAuthorList();

authorsList = authorsList.Where(a => a.FirstName != "Bob").ToList();

Или вы можете удалить все элементы в authors за второй проход.

var authorsList = GetAuthorList();

var authors = authorsList.Where(a => a.FirstName == "Bob").ToList();

foreach (var author in authors)
{
    authorList.Remove(author);
}
20 голосов
/ 12 мая 2009

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

static void Main()
{
    List<string> myList = new List<string> { "Jason", "Bob", "Frank", "Bob" };
    myList.RemoveAll(x => x == "Bob");

    foreach (string s in myList)
    {
        //
    }
}
15 голосов
/ 26 марта 2014

Я бродил, если есть разница между RemoveAll и Except и профессионалами использования HashSet, поэтому я сделал быструю проверку производительности:)

using System;
using System.Collections.Generic;
using System.Diagnostics;
using System.Linq;
using System.Text;

namespace ListRemoveTest
{
    class Program
    {
        private static Random random = new Random( (int)DateTime.Now.Ticks );

        static void Main( string[] args )
        {
            Console.WriteLine( "Be patient, generating data..." );

            List<string> list = new List<string>();
            List<string> toRemove = new List<string>();
            for( int x=0; x < 1000000; x++ )
            {
                string randString = RandomString( random.Next( 100 ) );
                list.Add( randString );
                if( random.Next( 1000 ) == 0 )
                    toRemove.Insert( 0, randString );
            }

            List<string> l1 = new List<string>( list );
            List<string> l2 = new List<string>( list );
            List<string> l3 = new List<string>( list );
            List<string> l4 = new List<string>( list );

            Console.WriteLine( "Be patient, testing..." );

            Stopwatch sw1 = Stopwatch.StartNew();
            l1.RemoveAll( toRemove.Contains );
            sw1.Stop();

            Stopwatch sw2 = Stopwatch.StartNew();
            l2.RemoveAll( new HashSet<string>( toRemove ).Contains );
            sw2.Stop();

            Stopwatch sw3 = Stopwatch.StartNew();
            l3 = l3.Except( toRemove ).ToList();
            sw3.Stop();

            Stopwatch sw4 = Stopwatch.StartNew();
            l4 = l4.Except( new HashSet<string>( toRemove ) ).ToList();
            sw3.Stop();


            Console.WriteLine( "L1.Len = {0}, Time taken: {1}ms", l1.Count, sw1.Elapsed.TotalMilliseconds );
            Console.WriteLine( "L2.Len = {0}, Time taken: {1}ms", l1.Count, sw2.Elapsed.TotalMilliseconds );
            Console.WriteLine( "L3.Len = {0}, Time taken: {1}ms", l1.Count, sw3.Elapsed.TotalMilliseconds );
            Console.WriteLine( "L4.Len = {0}, Time taken: {1}ms", l1.Count, sw3.Elapsed.TotalMilliseconds );

            Console.ReadKey();
        }


        private static string RandomString( int size )
        {
            StringBuilder builder = new StringBuilder();
            char ch;
            for( int i = 0; i < size; i++ )
            {
                ch = Convert.ToChar( Convert.ToInt32( Math.Floor( 26 * random.NextDouble() + 65 ) ) );
                builder.Append( ch );
            }

            return builder.ToString();
        }
    }
}

Результаты ниже:

Be patient, generating data...
Be patient, testing...
L1.Len = 985263, Time taken: 13411.8648ms
L2.Len = 985263, Time taken: 76.4042ms
L3.Len = 985263, Time taken: 340.6933ms
L4.Len = 985263, Time taken: 340.6933ms

Как мы видим, лучшим вариантом в этом случае является использование RemoveAll (HashSet)

8 голосов
/ 26 апреля 2013

Это очень старый вопрос, но я нашел очень простой способ сделать это:

authorsList = authorsList.Except(authors).ToList();

Обратите внимание, что поскольку возвращаемая переменная authorsList является List<T>, IEnumerable<T>, возвращаемое Except(), должно быть преобразовано в List<T>.

7 голосов
/ 07 сентября 2010

Вы можете удалить двумя способами

var output = from x in authorsList
             where x.firstname != "Bob"
             select x;

или

var authors = from x in authorsList
              where x.firstname == "Bob"
              select x;

var output = from x in authorsList
             where !authors.Contains(x) 
             select x;

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

6 голосов
/ 01 октября 2013

Скажите, что authorsToRemove - это IEnumerable<T>, который содержит элементы, которые вы хотите удалить из authorsList.

Тогда вот еще один очень простой способ выполнить задачу удаления, заданную ОП:

authorsList.RemoveAll(authorsToRemove.Contains);
5 голосов
/ 12 мая 2009

Я думаю, вы могли бы сделать что-то вроде этого

    authorsList = (from a in authorsList
                  where !authors.Contains(a)
                  select a).ToList();

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

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...