является Еогеаспом с лямбдой склонной к Коммодификации ошибки

Question

ArrayList<IFoo> objs = new ArrayList<>(); 

public void add(IFoo foo) { 
    objs.add(foo); 
} 

public void loop() { 
    objs.forEach(obj -> { 
     obj.doSomething(message); 
    }); 
} 

этот метод подвержены ошибкам ConcurrentModificationException или

я должны сделать копию массива

Должен ли я создать итератор?

или должен ли я отмечать оба метода синхронизации?

Answer 1

Хорошо, поэтому я помещаю все это в комментарии, но я думаю, что это становится слишком длинным, чтобы положить туда, вот и вот ответ.

Что у вас есть это стенография способ сделать

for(Object obj: objs){ ... } 

, который в своей очереди, является сокращенным способом получения итератора и итерации по коллекции. Iterator, который обрабатывает коллекцию, несет ответственность за выброшенные исключения. Итератор ArrayList действительно бросает исключение ConcurrentModificationException, когда к нему открывается итератор после его изменения. Поэтому, если ваш код делает это, тогда да. В противном случае нет.

ПРИМЕЧАНИЕ: ConcurrentModificationException, хотя содержащий слово Concurrent не обязательно имеет какое-либо отношение к многопоточности. Это очень важное различие. Вы можете получить ConcurrentModificationException быть выброшен просто просто сделать это:

objs.forEach(obj -> {objs.remove(0)}); 

Это правда, что множественная Thread s делает это также может привести к этому, но исключение на самом деле не имеет ничего общего с многопоточности. ConcurrentModificationException просто означает, что ArrayList был изменен одновременно во время его доступа, где одновременный в этом контексте означает в то же время, а не более традиционный вычислительный смысл многопоточности.

EDIT:

Я видел комментарий вы сделали о защите его, а также еще про то, что в Lib вы не можете контролировать. Лучший способ действительно защитить его - это, вероятно, сделать копию List в этом случае. Проблема в том, что lib является черным ящиком, и вы не можете контролировать, изменяют ли они коллекцию и, что более метко, блокируют ли они/блокируют/контролируют свои вставки/удаления из коллекции. Вероятно, создание мелкой копии Списка достаточно.См. Collections#copy(List, List) или ArrayList#ArrayList(Collection)

Answer 2

Это будет выброшено, если вы измените ArrayList в doSomething или в другом потоке, как и любая форма итерации.

Но если вы этого не делаете, нет причин беспокоиться.

Answer 3

Да, если вы изменяете список, например, добавляете или удаляете элементы. Однако, если вы изменяете значение/свойства на итерации объекта, вы должны быть хорошими (как вы пытаетесь).

Ниже приведен пример ConcurrentModificationException с традиционным циклом и использованием Lambda, оба из которых подвержены ошибкам.

import java.util.ArrayList; 
import java.util.List; 

public class Temp { 


    public static void main(String[] args) { 
     List<Integer> l = new ArrayList<>(); 
     l.add(1); 

     for(Integer data :l){ 
      l.add(99); 
     } 

     l.forEach(obj -> { 
      l.add(99); 
     }); 
    } 
} 

В вашем случае, когда вы беспокоитесь о каком-то другом потоке модифицирующего коллекции, пожалуйста, создать копию списка итерации, используйте Collections API, чтобы скопировать массив.