Tại sao List <T> .ForEach cho phép danh sách của nó được sửa đổi?

Nếu tôi sử dụng:

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

các Addtrong foreachném một InvalidOperationException (Bộ sưu tập đã được sửa đổi; hoạt động liệt kê không thể thực hiện), mà tôi cân nhắc hợp lý, vì chúng ta đang kéo tấm thảm từ dưới chân chúng ta.

Tuy nhiên, nếu tôi sử dụng:

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

nó nhanh chóng tự bắn vào chân bằng cách lặp lại cho đến khi ném ra OutOfMemoryException.

Điều này làm tôi ngạc nhiên, vì tôi luôn nghĩ rằng List.ForEach chỉ là một trình bao bọc cho foreachhoặc cho for.
Có ai có lời giải thích cho cách thức và lý do tại sao của hành vi này?

(Được thúc đẩy bởi vòng lặp ForEach cho một Danh sách chung được lặp lại liên tục )

Đó là bởi vì ForEachphương thức này không sử dụng enumerator, nó lặp qua các mục bằng một forvòng lặp:

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]);
    }
}

(mã lấy được bằng JustDecompile)

Vì trình điều tra không được sử dụng, nó không bao giờ kiểm tra xem danh sách đã thay đổi hay chưa và điều kiện kết thúc của forvòng lặp không bao giờ đạt được vì _sizenó được tăng lên ở mỗi lần lặp.

List<T>.ForEachđược thực hiện thông qua forbên trong, vì vậy nó không sử dụng trình điều tra và nó cho phép sửa đổi tập hợp.

Vì ForEach được gắn vào lớp List bên trong sử dụng vòng lặp for được gắn trực tiếp với các thành viên bên trong của nó - bạn có thể thấy điều này bằng cách tải xuống mã nguồn cho .NET framework.

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

Trong đó vòng lặp foreach trước hết là tối ưu hóa trình biên dịch nhưng cũng phải hoạt động chống lại bộ sưu tập với tư cách là người quan sát - vì vậy nếu bộ sưu tập được sửa đổi, nó sẽ ném ra một ngoại lệ.

Chúng tôi biết về vấn đề này, đó là một sự giám sát khi nó được viết ban đầu. Rất tiếc, chúng tôi không thể thay đổi nó vì nó sẽ ngăn mã hoạt động trước đây chạy:

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

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

Bản thân tính hữu ích của phương pháp này còn bị nghi ngờ như Eric Lippert đã chỉ ra, vì vậy chúng tôi đã không đưa nó vào cho các ứng dụng kiểu .NET for Metro (tức là các ứng dụng Windows 8).

David Kean (BCL Team)