Неясная ConcurrentModificationException на отдельных списках

Question

в классе Я эти 2 метода:

public void notifyResult(List<Road> result) { 
    ArrayList<RoadOverlay> tempRoads = new ArrayList<>(); 

    for(Road road:result){ 
    // Computes and stores the overlays 
    // ... 
     tempRoads.add(new RoadOverlay()); 
    } 
    //switch field when update is done 
    this.positions = tempRoads; 
    } 
} 

private void drawRoadsVO(GL gl) { 
    // keep a reference on the current instance of the field 
    // to avoid concurrent modification in notifyResult 
    ArrayList<RoadOverlay> positionsCopy = this.positions; 
    int vertexCount = 0; 
    for (RoadOverlay road : positionsCopy) { //ConcurrentModificationException here 
      // ... 
     } 
} 

Оба метода проложены в разных потоках. Внутри рендеринга я не делаю никаких изменений в списке, и, насколько я понимаю, я работаю над отдельными списками, так как это может привести к CME? Я борюсь за это, и любой ключ приветствуется. Насколько это возможно, я стараюсь избежать штрафа за использование синхронизации.

С уважением

Отредактировано комментарий в коде

Answer 1

Это, как правило, лучше, чтобы сделать полную копию списка

ArrayList<RoadOverlay> positionsCopy = new ArrayList<Integer>(this.positions); 

Что касается ConcurrentModificationException, ваш код выглядит нормально, может быть вы изменяете текущий this.positions в некоторых других методах класса? Обратите внимание, что переменная this.positions должна быть объявлена ​​как volatile.

Вот next() метод ArrayList итератора

public E next() { 
    checkForComodification(); 
    int i = cursor; 
    if (i >= size) 
     throw new NoSuchElementException(); 
    Object[] elementData = ArrayList.this.elementData; 
    if (i >= elementData.length) 
     throw new ConcurrentModificationException(); 
    cursor = i + 1; 
    return (E) elementData[lastRet = i]; 
} 

final void checkForComodification() { 
    if (modCount != expectedModCount) 
     throw new ConcurrentModificationException(); 
} 

ConcurrentModificationException может быть брошено только в том случае, то Листом elementData структурно модифицированного

Структурные изменения являются те, которые изменяют размер списка, или иным образом нарушить его таким образом, что выполняемые итерации могут давать неверные результаты.

Answer 2

проблема внутри drawRoadsVO метод ваши позиции varCopy на самом деле не копия, а ссылка на тот же объект this.positions.

Для того, чтобы сделать фактическую копию вы должны сделать

ArrayList<RoadOverlay> positionsCopy = new ArrayList(this.positions);