Sửa đổi một từ điển mà tôi đang lặp lại qua

Question

foreach(BruteforceEntry be in Entries.Values) 
{ 
    if (be.AddedTimeRemove <= now) 
     Entries.Remove(be.IPAddress); 
    else if (be.Unbantime <= now && be.Unbantime.Day == DateTime.Now.Day) 
     Entries.Remove(be.IPAddress); 
} 

Một ngoại lệ được ném:

Bộ sưu tập đã được sửa đổi; hoạt động điều tra có thể không thực hiện được.

Vì lý do nào đó, nó không còn nữa.

Tôi biết bạn không thể xóa nội dung nào đó, trong khi lặp lại theo cách này. Câu hỏi của tôi là: Làm thế nào để giải quyết nó?

Answer 1

Bạn không thể sửa đổi bộ sưu tập bạn đang lặp lại. Trong trường hợp này, một giải pháp tốt hơn sẽ là để tạo ra một danh sách các mục để loại bỏ bằng cách duyệt qua từ điển, và sau đó lặp qua rằng danh sách, loại bỏ các mục từ điển:

List<string> removals = new List<string>();      
DateTime now = DateTime.Now; 
foreach(BruteforceEntry be in Entries.Values) 
{ 
    if (be.AddedTimeRemove <= now || 
     (be.Unbantime <= now && be.Unbantime.Day == DateTime.Now.Day)) 
    { 
     removals.Add(be.IPAddress); 
    } 
} 
foreach (string address in removals) 
{ 
    Entries.Remove(address); 
} 

Lưu ý rằng nếu bạn đang sử dụng .NET 3.5, bạn có thể sử dụng một truy vấn LINQ để diễn tả phần đầu tiên:

List<string> removals = (from be in Entries.Values 
         where be.AddedTimeRemove <= now || 
           (be.Unbantime <= now && 
           be.Unbantime.Day == DateTime.Now.Day) 
         select be.IPAddress).ToList(); 

Answer 2

Bạn không thể sửa đổi bộ sưu tập khi lặp lại bộ sưu tập với foreach. Thay vào đó, lặp lại nó với một vòng lặp for.

Answer 3

bạn có thể thay đổi foreach(BruteforceEntry be in Entries.Values)-foreach(BruteforceEntry be in new List<BruteforceEntry>(Entries.Values))

bằng cách đó bạn không sửa đổi bộ sưu tập của bạn, nhưng ra có một bản sao của nó.

Answer 4

sôi xuống vấn đề của bạn ...

foreach(... ... in Entries.Values) 
{ 
     Entries.Remove(...); 

} 

Như những người khác đã nói bạn đang sửa đổi iterator trong khi iterating.

Bạn có thể, như @ David cho biết, sử dụng vòng lặp for thay thế, nhưng hãy đảm bảo bắt đầu ở cuối (lặp lại ngược lại).

Answer 5

Chỉ cần đặt: bạn không thể xóa mục nhập khỏi bộ sưu tập trong khi bạn đang lặp lại nó.

Một workaround có thể là tạo ra một bản sao cạn của bộ sưu tập (ví dụ sử dụng ToList) và duyệt qua rằng:

foreach(BruteforceEntry be in Entries.Values.ToList()) 
{ 
    // modify the original collection 
} 

Answer 6

Dường như câu hỏi của bạn là về việc tại sao một ngoại lệ là không bị ném nơi nó từng là. Vì các câu trả lời khác nói chung, bạn thường không thể thay đổi một bộ sưu tập mà bạn đang lặp lại, nhưng bạn đang lặp qua bộ sưu tập Giá trị, mà tôi tin là bản sao của các giá trị trong từ điển, thay vì tham chiếu đến bộ sưu tập từ điển chính chinh no. Vì vậy, nó không còn có vấn đề lặp lại và sửa đổi cùng một điều.

Answer 7

này (theo ý kiến ​​của tôi) là cách dễ nhất:

Dictionary<String, String> A = new Dictionary<string, string>(); //Example Dictionary 
A.Add("A", "A"); //Example Values 
A.Add("B", "B"); 
A.Add("C", "C"); 

for (int i = A.Count - 1; i >= 0; i--) //Loop backwards so you can remove elements. 
{ 
    KeyValuePair<String, String> KeyValue = A.ElementAt(i); //Get current Element. 
    if (KeyValue.Value == "B") A.Remove(KeyValue.Key); 
} 

Trong trường hợp của bạn:

for (int i = Entries.Count - 1; i >= 0; i--) 
{ 
    KeyValuePair<String, BruteforceEntry> KeyValue = Entries.ElementAt(i); 
    if (KeyValue.Value.AddedTimeRemove <= now) 
     Entries.Remove(KeyValue.Key); 
    else if (KeyValue.Value.Unbantime <= now && KeyValue.Value.Unbantime.Day == DateTime.Now.Day) 
     Entries.Remove(KeyValue.Key); 
}