xóa các mục khỏi Danh sách chung <t>

Question

Tôi có phương pháp sau, tôi muốn xóa các mục khỏi bộ sưu tập của mình khớp với Id sản phẩm. Có vẻ khá thẳng về phía trước, nhưng tôi nhận được một ngoại lệ. Về cơ bản, bộ sưu tập của tôi không được đồng bộ hóa. Vì vậy, cách tốt nhất để loại bỏ một mục từ một bộ sưu tập là gì.

public void RemoveOrderItem(Model.Order currentOrder, int productId) 
{ 

    foreach (var orderItem in currentOrder.OrderItems) 
    { 
     if (orderItem.Product.Id == productId) 
     { 
      currentOrder.OrderItems.Remove(orderItem); 
     } 
    } 
} 

Chi tiết ngoại lệ: System.InvalidOperationException: Bộ sưu tập đã được sửa đổi; hoạt động điều tra có thể không thực thi

Answer 1

Sửa đổi bộ sưu tập trong vòng lặp không hoạt động. Để giải quyết vấn đề đó, List có một vài phương pháp cho phép sửa đổi "hàng loạt" của một bộ sưu tập. Trong trường hợp của bạn, hãy sử dụng:

currentOrder.OrderItems.RemoveAll(x => x.Product.Id == productId) 

Answer 2

Bạn không thể sửa đổi bộ sưu tập trong khi lặp lại bộ sưu tập đó. Chỉ cần sử dụng vòng lặp for bình thường thay vì vòng lặp foreach.

Answer 3

Bạn không thể xóa một mục từ một bộ sưu tập bạn đang lặp qua, bạn có thể theo dõi các OrderItem, sau đó loại bỏ nó sau khi bạn hoàn thành vòng lặp

Answer 4

By looping cách này bạn không thể xoá các mục bởi vì trong nó bộ sưu tập nó giữ theo dõi các mục được lưu trữ.

Cách dễ dàng để làm điều này:

authorsList.RemoveAll(x => x.ProductId == productId); 

hoặc

authorsList = authorsList.Where(x => x.ProductId!= productId).ToList(); 

Answer 5

Như bạn nhận ra rằng bạn không thể loại bỏ một mục từ một bộ sưu tập khi bạn đang lặp qua nó. Tôi chắc chắn một người nào đó sẽ có thể cung cấp một giải pháp gọn gàng LINQ nhưng sau sẽ giúp bạn đi ban đầu:

public void RemoveOrderItem(Model.Order currentOrder, int productId) 
{ 
    var selectedOrderItem = null; 
    foreach (var orderItem in currentOrder.OrderItems) 
    { 
     if (orderItem.Product.Id == productId) 
     { 
      selectedOrderItem = orderItem; 
      break; 
     } 
    } 

    if(selectedOrderItem != null) 
     currentOrder.OrderItems.Remove(selectedOrderItem); 
} 

Answer 6

"foreach" cung cấp một "Forward-only read-only" lặp đi lặp lại của một bộ sưu tập.

Để giải quyết sự cố này, bạn có thể sao chép tham chiếu đến bộ sưu tập khác và sau đó lặp lại trên bộ sưu tập đã sao chép và xóa các mục khỏi bộ sưu tập gốc.