Tái cấu trúc câu lệnh forested lồng nhau

Question

Tôi có một phương thức với bộ sưu tập foreach lồng nhau (lặp một tập hợp các đối tượng và sau đó nhìn vào bên trong từng đối tượng). Tôi thấy trong một cuốn sách là một mô hình tốt để làm cho điều này thanh lịch hơn nhiều nhưng không thể nhớ/tìm thấy ví dụ mã. Làm thế nào khác tôi có thể làm cho điều này gọn gàng hơn?

Mã chỉ là một tuyên bố lồng nhau điển hình foreach vì vậy tôi chưa cung cấp mẫu mã.

Answer 1

Các giải pháp rõ ràng là để san bằng vào phương pháp.

Cũ:

void SubmitOrders() 
{ 
    var orders = GetOrders(); 
    foreach (Order o in orders) 
    { 
     foreach (OrderDetail d in o.Details) 
     { 
      // Blah... 
     } 
    } 
} 

mới:

câu trả lời

void SubmitOrders() 
{ 
    var orders = GetOrders() 
    foreach (Order o in orders) 
    { 
     SubmitOrder(o); 
    } 
} 

void SubmitOrder(Order order) 
{ 
    foreach (OrderDetail d in order.Details) 
    { 
     // Blah... 
    } 
} 

khác ở đây dường như được tập trung vào LINQ, và tôi sẽ đồng ý rằng nếu vòng của bạn không có tác dụng phụ (tức là bạn chỉ cố gắng trích xuất một số thông tin từ vòng lặp trong cùng), sau đó bạn có thể viết lại toàn bộ điều bằng cách sử dụng một hoặc hai câu lệnh LINQ đơn giản. Nếu tác dụng phụ có liên quan, sau đó chỉ cần làm theo các thực hành được kiểm tra theo thời gian của các chương trình con.

Answer 2

Bạn sẽ phải cụ thể hơn về ý nghĩa của mình về "thanh lịch hơn", vì IMO không có gì đặc biệt là không phù hợp về lồng nhau foreach.

Điều đó đang được nói, các phương pháp mở rộng LINQ trong .NET 3.5 trở lên có thể trợ giúp (cụ thể là SelectMany).

public class Foo 
{ 
    public List<string> Strings { get; set; } 
} 

... 

List<Foo> foos = new List<Foo>(); 

foreach(string str in foos.SelectMany(f => f.Strings)) 
{ 
    ... 
} 

Answer 3

Bạn có thể sử dụng toán tử LINQ SelectMany:

foreach (var bar in model.FooCollection.SelectMany(f => f.Bars)) 
{ 
    // Do Stuff 
} 

http://www.hookedonlinq.com/SelectManyOperator.ashx

Answer 4

Bạn đang nghĩ đến việc gì đó như thế này?

public class YourObject{ 
    public List<OtherObject> Others { get; set; } 
} 

public class OtherObject{ 
    public void DoIt(){} 
} 

var theList = new List<YourObject>(); 

theList.ForEach(yo => yo.Others.ForEach(oo => oo.DoIt())); 

Answer 5

Trước:

foreach(Customer c in Customers) 
{ 
    foreach(Order o in c.Orders) 
    { 
    o.Dance(); 
    } 
} 

Sau:

foreach(Order o in Customers.SelectMany(c => c.Orders)) 
{ 
    o.Dance(); 
} 

Answer 6

Khó mà đưa ra câu trả lời mà không có bất kỳ ngữ cảnh hoặc đoạn mã nào. Tuy nhiên, Martin Fowler có một bài viết thực sự tốt về các vòng tái cấu trúc vào các đường ống LINQ có thể được tìm thấy here (đánh giá cao đây là một câu hỏi cũ, vì vậy hy vọng người đọc trong tương lai sẽ được hưởng lợi!).