併發修改異常。在這種情況下如何避免？

Question

考慮下面的代碼：

List tableData = null; 
tableData = new ArrayList(records.size()); 
for (Iterator iter = records.iterator(); iter.hasNext();) { 
      Test record = (Test) iter.next(); 
      Map rowData = createRowData(record); 
      if (rowData != null) { 
       // sorted insert 
       Date newDate = (Date)  rowData.get(TestModel.TIMESTAMP); 
       boolean done = false; 
       for (int row = 0; row < tableData.size(); row++) { 
        Map currentRow = (Map) tableData.get(row); 
        Date currentDate = (Date)  currentRow.get(TestModel.TIMESTAMP); 
        if (currentDate.after(newDate)) { 
         tableData.add(row, rowData); 
         done = true; 
         break; 
        } 
       } 
    if (!done) { 
        tableData.add(rowData); 
       } 
      } 
     } 
    } catch(Exception e) { 
     throw new RuntimeException("Error reading fuel and SMU data", e); 
    } 
    return tableData; 

records這裏有new Vector()正在被初始化其間。

由於ArrayList具有int容量的參數。在這種情況下如何使用CopyOnWriteArrayList？

`STACKTRACE: 
    at com.mincom.explorer.uif.AbstractRequestHandler.handleException(AbstractRequestHandler.java:255) 
    at com.mincom.explorer.uif.AbstractRequestHandler.readData(AbstractRequestHandler.java:137) 
    at com.mincom.jive.service.RequestExecutor$RequestHandlerCommand.mainProcess(RequestExecutor.java:156) 
    at com.mincom.util.executor.AbstractCommand.run(AbstractCommand.java:56) 
    at com.mincom.util.executor.SimpleExecutor$WorkerThread.run(SimpleExecutor.java:156) 
THROWABLE: java.lang.RuntimeException: Error reading fuel and SMU data 
    at minestar.production.presentation.page.fuel.smu.assistant.*.getEventsUsingFilter(*.java:243) 
    at minestar.production.presentation.page.fuel.smu.assistant.*.processReadData(*.java:174) 
    at com.mincom.explorer.uif.AbstractRequestHandler.readData(AbstractRequestHandler.java:129) 
    at com.mincom.jive.service.RequestExecutor$RequestHandlerCommand.mainProcess(RequestExecutor.java:156) 
    at com.mincom.util.executor.AbstractCommand.run(AbstractCommand.java:56) 
    at com.mincom.util.executor.SimpleExecutor$WorkerThread.run(SimpleExecutor.java:156) 
Caused by: java.util.ConcurrentModificationException 
    at java.util.ArrayList$Itr.checkForComodification(Unknown Source) 
    at java.util.ArrayList$Itr.next(Unknown Source) 
    at *.production.presentation.page.fuel.smu.assistant.*.getEventsUsingFilter(*.java:222) 
    ... 5 more` 

我已經修改了這段代碼。請看一看。

Answer 1

您正在修改您正在使用的列表。

有兩種方法可以擺脫這種情況。一種是，創建實際列表的副本，並遍歷這個新列表，但修改舊列表。

List copy = copyOf(myArrayList); 
for(item : copy){ 
    //do stuff 
    //addon original list 
    myArrayList.add(item); 
} 

第二個是，在你的列表上向後迭代。

for(int i = list.size()-1; i >=0; i--){ 
    //add or remove or do stuff here 
} 

第二個解決方案不適用於您，因爲您正在將項目添加到特定索引。但如果你只需要添加/刪除當前搶下項目，這將是罰款，也

注意，這是僞代碼

Answer 2

您遍歷數組列表，然後通過添加特定的指數，即tableData.add(row, rowData);數據修改它修改列表的大小。

我認爲它是領先的java.util.ConcurrentModificationException異常，更好的方法是創建另一個數組列表並向其中添加內容。

Answer 3

您將數據添加到List同時，通過它迭代：

for (int row = 0; row < tableData.size(); row++) { 
    Map currentRow = (Map) tableData.get(row); 
    Date currentDate = (Date)  currentRow.get(TestModel.TIMESTAMP); 
    if (currentDate.after(newDate)) { 
     tableData.add(row, rowData); 
     done = true; 
     break; 
    } 
} 

這就是爲什麼你得到一個ConcurrentModificationException的原因。

CopyOnWriteArrayList不會解決你的問題：我會建議在克隆的clonedTableData您tableData，並同時通過原來的tableData迭代，添加新的記錄，後來分配到tableData「平等」 clonedtableData。 注意：這可能是一個昂貴的練習。

爲什麼我說「CopyOnWriteArrayList不會解決你的問題」的原因是基於它的設計：

此數組在迭代器的生存期內不會改變，所以 干擾是不可能的，迭代器保證不會拋出 ConcurrentModificationException。自迭代器創建了 以來，迭代器不會反映 添加，刪除或更改列表。迭代器本身的元素更改操作（刪除， 集合和添加）不受支持。這些方法拋出 UnsupportedOperationException。

Answer 4

不要在迭代時向列表添加數據。