Tự động xóa các phần tử khỏi Danh sách

Question

Tôi gặp sự cố khi xóa các phần tử của danh sách trong khi lặp qua danh sách. Mã số:

For (WebElement element: list){ 
    if (!element.isEnabled() || !element.isSelected()){ 
     list.remove(element); 
    } 
} 

Tôi nhận được ConcurrentModificationException, mà tôi hoàn toàn hiểu. Tôi đang xóa một mục khỏi danh sách trong khi trong vòng lặp đi qua danh sách. Trực giác, điều đó sẽ làm hỏng việc lập chỉ mục của vòng lặp.

Câu hỏi của tôi là, cách khác, tôi nên xóa các phần tử không phải là enabled hoặc selected khỏi danh sách này?

Answer 1

Cách đơn giản nhất để loại bỏ các yếu tố từ một danh sách trong một vòng lặp là sử dụng một ListIterator và loại bỏ các yếu tố sử dụng thường xuyên iterator.remove()

Answer 2

Sửa đổi một danh sách trong khi iterating qua nó, theo một cách bên ngoài của việc sử dụng lặp, kết quả trong hành vi không xác định. Bạn sẽ phải sử dụng một trình lặp một cách rõ ràng:

Iterator<WebElement> iter = list.iterator(); 
while (iter.hasNext()) { 
    WebElement element = iter.next(); 
    if (!element.isEnabled() || !element.isSelected()) { 
     iter.remove(); 
    } 
} 

Xem this question để biết thêm.

Answer 3

Kết quả ConcurrentModificationException thực tế là cú pháp cho mỗi cú pháp chỉ là đường cú pháp để sử dụng giao diện Iterator.

Danh sách trình vòng lặp có thuộc tính "không nhanh", có nghĩa là bất kỳ thay đổi nào được thực hiện cho danh sách ngoài giao diện do trình vòng lặp cung cấp, ngay lập tức vô hiệu hóa trình lặp nói trên. Việc cố gắng sử dụng trình vòng lặp không hợp lệ sẽ kích hoạt ngoại lệ của bạn.

@Claudiu đã đăng mã này, nhưng để rõ ràng, tôi cũng sẽ đặt nó ở đây. Để làm những gì bạn đang cố gắng làm, bạn sẽ phải thả cú pháp ưa thích và sử dụng một Iterator trần.

Iterator<WebElement iter = list.iterator(); 
while (iter.hasNext()) { 
    WebElement element = iter.next(); 
    if (!element.isEnabled() || !element.isSelected()) { 
     iter.remove(); 
    } 
} 

Answer 4

Những người khác đã đề xuất sử dụng trình lặp danh sách. Điều đó đã chứng tỏ hữu ích cho tôi, nhưng tiếc là nó dựa trên một phương pháp, remove(), được coi là tùy chọn bởi giao diện Iterable<E>.

Quoth Javadoc, nevermore (tôi nhấn mạnh):

trống remove()

Xóa khỏi bộ sưu tập cơ bản các yếu tố cuối cùng được trả về bởi iterator (hoạt động không bắt buộc).

Để khắc phục điều đó đã chứng minh hữu ích hơn đối với tôi là danh sách xóa.

List<E> removed = new ArrayList<E>(); 
for(E element : list) { 
    if(someCondition) removed.add(element); 
} 
list.removeAll(removed); 

Điều này mang lại cho bạn một lịch sử về những gì bạn đã xóa, giống như phương pháp xóa.