Xóa điều khiển trong vòng lặp

Question

Hôm qua tôi đã viết một đoạn mã để xóa tất cả các điều khiển trong một biểu mẫu đáp ứng các tiêu chí nhất định. Viết nó ngây thơ, đây là những gì tôi nghĩ ra.

for (int i = 0; i < this.Controls.Count; ++i) 
{ 
    if (this.Controls[i].Name.Length == 2) 
    { 
     this.Controls.Remove(this.Controls[i); 
    } 
} 

Nhưng điều đó xảy ra là mã sai. Sau đó, tôi đổi thành:

foreach (Control ctr in this.pbBoardImage.Controls) 
{ 
    if (ctr.Length == 2) 
    { 
     this.Controls.Remove(ctr); 
    } 
} 

Nhưng vẫn không đúng. Tôi biết rằng cách chính xác sẽ là:

for (int i = this.Controls.Count - 1; i >= 0; i--) 
{ 
    if (this.Controls[i].Name.Length == 2) 
    { 
     this.Controls.Remove(this.Controls[i]); 
    } 
} 

Tuy nhiên nó vẫn không cảm thấy thanh lịch. Tôi không thể sử dụng List.RemoveAll, vì this.Controls không phải là một List. Vì vậy, tôi có thể yêu cầu một cách thanh lịch hơn, tốt hơn là không sử dụng một vòng lặp?

Answer 1

Không chắc chắn lý do bạn không thích câu trả lời này ... Tôi đã đánh dấu quan trọng RemoveAt; Tuy nhiên, như một sự thay thế trong NET 3.5/C# 3.0: LINQ:

 var qry = from Control control in Controls 
        where control.Name.Length == 2 
        select control; 

     foreach(var control in qry.ToList()) { 
      Controls.Remove(control); 
     } 

---

(bản gốc)

Bạn không thể Remove trong foreach - nó phá vỡ các iterator. Một cách tiếp cận phổ biến ở đây là để lặp ngược:

for (int i = this.Controls.Count - 1; i >= 0; i--) { 
    if (this.Controls[i].Name.Length == 2) { 
     this.Controls.RemoveAt(i); // <=========== *** RemoveAt 
    } 
} 

Điều này tránh sự "tắt bởi một" vấn đề vv