ใช้ LINQ เพื่อลบองค์ประกอบออกจากรายการ <T>


654

บอกว่าฉันมีคำสั่ง LINQ เช่น:

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

เมื่อระบุว่าauthorsListเป็นประเภทList<Author>ฉันจะลบAuthorองค์ประกอบจากauthorsListสิ่งที่ส่งคืนโดยแบบสอบถามได้authorsอย่างไร

หรืออีกวิธีหนึ่งฉันจะลบ Bob ที่เท่ากับของชื่อทั้งหมดได้authorsListอย่างไร

หมายเหตุ: นี่เป็นตัวอย่างที่ง่ายสำหรับวัตถุประสงค์ของคำถาม

คำตอบ:


1138

มันจะง่ายกว่าถ้าจะแยกพวกมันตั้งแต่แรก:

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

อย่างไรก็ตามนั่นจะเป็นการเปลี่ยนค่าauthorsListแทนที่จะลบผู้แต่งออกจากคอลเลกชันก่อนหน้า หรือคุณสามารถใช้RemoveAll:

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

หากคุณจำเป็นต้องทำตามคอลเลกชันอื่นฉันจะใช้ HashSet, RemoveAll และประกอบด้วย:

var setToRemove = new HashSet<Author>(authors);
authorsList.RemoveAll(x => setToRemove.Contains(x));

14
อะไรคือสาเหตุของการใช้ HashSet สำหรับคอลเล็กชันอื่น
123 456 789 0

54
@LeoLuis: ทำให้การContainsตรวจสอบรวดเร็วและช่วยให้คุณประเมินลำดับได้เพียงครั้งเดียว
Jon Skeet

2
@LeoLuis: ใช่การสร้าง HashSet จากลำดับจะประเมินเพียงครั้งเดียว ไม่แน่ใจว่าสิ่งที่คุณหมายถึงโดย "ชุดคอลเลกชันที่อ่อนแอ"
Jon Skeet

2
@ AndréChristofferAndersen: คุณหมายถึงอะไร "ล้าสมัย"? มันยังใช้งานได้ หากคุณมีList<T>มันก็ดีที่จะใช้มัน
Jon Skeet

4
@ AndréChristofferAndersen: มันจะดีกว่าที่จะใช้authorsList = authorsList.Where(x => x.FirstName != "Bob")
Jon Skeet

133

มันจะเป็นการดีกว่าถ้าใช้List <T> ลบทั้งหมดเพื่อทำสิ่งนี้ให้สำเร็จ

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

8
@Reed Copsey: พารามิเตอร์ lambda ในตัวอย่างของคุณอยู่ในวงเล็บคือ (x) มีเหตุผลทางเทคนิคสำหรับสิ่งนี้หรือไม่? ถือว่าเป็นการฝึกที่ดีหรือไม่?
Matt Davis

24
ไม่จำเป็นต้องใช้กับ> พารามิเตอร์ 1 ตัว ด้วยพารามิเตอร์เดียวมันเป็นตัวเลือก แต่มันช่วยรักษาความมั่นคง
Reed Copsey

48

หากคุณต้องการลบรายการจริงๆแล้วจะมีอะไรบ้างยกเว้น ()?
คุณสามารถลบตามรายการใหม่หรือลบทันทีโดยซ้อน 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();

Except()เป็นวิธีเดียวที่จะไปกลางคำสั่ง LINQ IEnumerableไม่มีมิได้Remove() RemoveAll()
Jari Turkia

29

คุณไม่สามารถทำสิ่งนี้กับตัวดำเนินการ 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);
}

12
RemoveAll()ไม่ใช่ตัวดำเนินการ LINQ
Daniel Brückner

ขอโทษด้วย. คุณถูกต้อง 100% น่าเสียดายที่ฉันไม่สามารถย้อนกลับ downvote ของฉันได้ ขอโทษด้วยกับเรื่องนั้น.
ไชโคเฮน

Removeเป็นเมธอดList< T> ไม่ใช่เมธอดSystem.Linq.Enumerable
DavidRR

@ แดเนียลแก้ไขฉันถ้าฉันผิดเราสามารถหลีกเลี่ยง. ToList () จากที่ไหน condtion สำหรับตัวเลือกที่สอง เช่นรหัสด้านล่างจะทำงาน varhorsList = GetAuthorList (); ผู้แต่ง var =horsList.Where (a => a.FirstName == "Bob"); foreach (ผู้แต่ง var ในผู้เขียน) {authorList.Remove (ผู้เขียน); }
Sai

ใช่มันใช้งานได้ การเปลี่ยนเป็นรายการจะต้องใช้เฉพาะในกรณีที่คุณต้องการให้รายการส่งผ่านไปยังวิธีการบางอย่างหรือหากคุณต้องการเพิ่มหรือลบรายการอื่น ๆ นอกจากนี้ยังอาจเป็นประโยชน์หากคุณต้องระบุลำดับหลายครั้งเพราะคุณจะต้องประเมินค่าที่อาจเกิดขึ้นที่เงื่อนไขหนึ่งครั้งหรือหากผลลัพธ์อาจเปลี่ยนไประหว่างการแจงนับสองครั้งตัวอย่างเช่นเนื่องจากเงื่อนไขขึ้นอยู่กับเวลาปัจจุบัน หากคุณต้องการใช้ในวงเดียวคุณไม่จำเป็นต้องเก็บผลลัพธ์ไว้ในรายการก่อน
Daniel Brückner

20

วิธีแก้ปัญหาง่าย ๆ :

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

    foreach (string s in myList)
    {
        //
    }
}

วิธีการลบ "Bob" และ "Jason" ฉันหมายถึงหลายรายการในรายการสตริง?
Neo

19

ฉันสงสัยว่าถ้ามีความแตกต่างระหว่าง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)


รหัสนี้: "l2.RemoveAll (HashSet <string> ใหม่ (toRemove) .Contains);" ไม่ควรคอมไพล์ ... และถ้าการทดสอบของคุณถูกต้องพวกเขาแค่สองสิ่งที่ Jon Skeet แนะนำไว้แล้ว
ปาสกาล

2
l2.RemoveAll( new HashSet<string>( toRemove ).Contains );รวบรวมได้เพียง FYI
AzNjoE

9

นี่เป็นคำถามที่เก่ามาก แต่ฉันพบวิธีง่ายๆในการทำสิ่งนี้:

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

หมายเหตุว่าตั้งแต่ตัวแปรผลตอบแทนauthorsListเป็นList<T>ที่IEnumerable<T>ส่งกลับโดยต้องได้รับการดัดแปลงเป็นExcept()List<T>


7

คุณสามารถลบได้สองวิธี

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;

ฉันมีปัญหาเดียวกันถ้าคุณต้องการผลลัพธ์ง่าย ๆ ตามเงื่อนไขของคุณแล้ววิธีแก้ปัญหาแรกดีกว่า


ฉันจะตรวจสอบ "Bob" หรือ "Billy" ได้อย่างไร
Si8

6

สมมติว่าauthorsToRemoveเป็นIEnumerable<T>องค์ประกอบที่มีองค์ประกอบที่คุณต้องการลบออกauthorsListที่มีองค์ประกอบที่คุณต้องการลบออกจาก

จากนั้นนี่เป็นอีกวิธีที่ง่ายมากในการลบภารกิจตามที่ขอโดย OP:

authorsList.RemoveAll(authorsToRemove.Contains);

5

ฉันคิดว่าคุณสามารถทำอะไรเช่นนี้

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

แม้ว่าฉันจะคิดว่าวิธีการแก้ปัญหาที่ให้ไปแล้วสามารถแก้ปัญหาได้ในทางที่อ่านง่ายขึ้น


4

ด้านล่างเป็นตัวอย่างการลบองค์ประกอบออกจากรายการ

 List<int> items = new List<int>() { 2, 2, 3, 4, 2, 7, 3,3,3};

 var result = items.Remove(2);//Remove the first ocurence of matched elements and returns boolean value
 var result1 = items.RemoveAll(lst => lst == 3);// Remove all the matched elements and returns count of removed element
 items.RemoveAt(3);//Removes the elements at the specified index

1

LINQ มีต้นกำเนิดในการเขียนโปรแกรมการทำงานซึ่งเน้นการเปลี่ยนแปลงไม่ได้ของวัตถุดังนั้นจึงไม่มีวิธีในการอัพเดทรายการดั้งเดิมในสถานที่

หมายเหตุเกี่ยวกับการเปลี่ยนแปลงไม่ได้ (นำมาจากคำตอบ SO อื่น):

นี่คือความหมายของการไม่เปลี่ยนรูปจากวิกิพีเดีย

ในการเขียนโปรแกรมเชิงวัตถุและฟังก์ชันการทำงานวัตถุที่ไม่เปลี่ยนรูปเป็นวัตถุที่มีสถานะไม่สามารถแก้ไขได้หลังจากที่มันถูกสร้างขึ้น


0

ฉันคิดว่าคุณเพียงแค่ต้องกำหนดรายการจากรายการผู้แต่งไปยังรายการใหม่เพื่อให้มีผล

//assume oldAuthor is the old list
Author newAuthorList = (select x from oldAuthor where x.firstname!="Bob" select x).ToList();
oldAuthor = newAuthorList;
newAuthorList = null;

0

ในการรักษาโค้ดให้คล่อง (หากการปรับให้เหมาะสมโค้ดไม่สำคัญ) และคุณจะต้องดำเนินการเพิ่มเติมในรายการ:

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

หรือ

authorsList = authorsList.Where(x => !setToRemove.Contains(x)).<do_some_further_Linq>;
โดยการใช้ไซต์ของเรา หมายความว่าคุณได้อ่านและทำความเข้าใจนโยบายคุกกี้และนโยบายความเป็นส่วนตัวของเราแล้ว
Licensed under cc by-sa 3.0 with attribution required.