เหตุใด List <T> .ForEach จึงอนุญาตให้แก้ไขรายการได้

ถ้าฉันใช้:

var strings = new List<string> { "sample" };
foreach (string s in strings)
{
  Console.WriteLine(s);
  strings.Add(s + "!");
}

Addในforeachพ่น InvalidOperationException (เก็บได้รับการปรับเปลี่ยนการดำเนินการแจงนับไม่อาจดำเนินการ) ซึ่งผมคิดว่าตรรกะเนื่องจากเรามีการดึงพรมจากภายใต้เท้าของเรา

อย่างไรก็ตามหากฉันใช้:

var strings = new List<string> { "sample" };
strings.ForEach(s =>
  {
    Console.WriteLine(s);
    strings.Add(s + "!");
  });

มันยิงตัวเองที่เท้าทันทีโดยการวนซ้ำจนกว่ามันจะพ่น OutOfMemoryException

นี้มาเป็นแปลกใจให้ฉันเป็นฉันคิดเสมอว่า List.ForEach เป็นอย่างใดอย่างหนึ่งเพียงเสื้อคลุมสำหรับหรือforeach ใครมีคำอธิบายเกี่ยวกับวิธีการและสาเหตุของพฤติกรรมนี้หรือไม่?for

(Inpired byEach loop for a Generic List ซ้ำไม่รู้จบ )

เป็นเพราะForEachวิธีการนี้ไม่ได้ใช้ตัวแจงนับมันจะวนforซ้ำรายการด้วยการวนซ้ำ:

public void ForEach(Action<T> action)
{
    if (action == null)
    {
        ThrowHelper.ThrowArgumentNullException(ExceptionArgument.match);
    }
    for (int i = 0; i < this._size; i++)
    {
        action(this._items[i]);
    }
}

(รหัสที่ได้รับจาก JustDecompile)

เนื่องจากไม่ได้ใช้ตัวแจงนับจึงไม่เคยตรวจสอบว่ารายการมีการเปลี่ยนแปลงหรือไม่และforจะไม่ถึงเงื่อนไขสิ้นสุดของลูปเนื่องจาก_sizeจะเพิ่มขึ้นทุกครั้งที่ทำซ้ำ

List<T>.ForEachถูกนำไปใช้forภายในดังนั้นจึงไม่ใช้ตัวนับและอนุญาตให้แก้ไขคอลเลกชัน

เนื่องจาก ForEach ที่แนบมากับคลาส List ภายในจะใช้ for loop ที่แนบโดยตรงกับสมาชิกภายในซึ่งคุณสามารถดูได้โดยดาวน์โหลดซอร์สโค้ดสำหรับ. NET framework

http://referencesource.microsoft.com/netframework.aspx

โดยที่ foreach loop เป็นอันดับแรกและสำคัญที่สุดของการเพิ่มประสิทธิภาพคอมไพลเลอร์ แต่ยังต้องดำเนินการกับคอลเล็กชันในฐานะผู้สังเกตการณ์ด้วยดังนั้นหากมีการแก้ไขคอลเล็กชันจะมีข้อยกเว้น

เราทราบเกี่ยวกับปัญหานี้ แต่เป็นการกำกับดูแลเมื่อเขียนขึ้นในตอนแรก น่าเสียดายที่เราไม่สามารถเปลี่ยนแปลงได้เนื่องจากตอนนี้จะป้องกันไม่ให้โค้ดที่ใช้งานได้ก่อนหน้านี้ทำงาน:

        var list = new List<string>();
        list.Add("Foo");
        list.Add("Bar");

        list.ForEach((item) => 
        { 
            if(item=="Foo") 
                list.Remove(item); 
        });

ประโยชน์ของวิธีนี้นั้นเป็นที่น่าสงสัยตามที่Eric Lippertชี้ให้เห็นดังนั้นเราจึงไม่ได้รวมไว้ใน. NET สำหรับแอปสไตล์ Metro (เช่นแอป Windows 8)

เดวิดคีน (ทีม BCL)