リストの同時変更例外

Question

ここでどこが間違っているのか誰にでも教えてもらえますか？

File promoCountFile = new File(RaconTours.PATH + "promocodeCount.txt"); 
      if (promoPlistPath.exists()) { 
       try { 
        ObjectInputStream inStream = new ObjectInputStream(new FileInputStream(promoPlistPath)); 
        ObjectInputStream promoStream = new ObjectInputStream(new FileInputStream(promoCountFile)); 
        promoobj = (ArrayList<HashMap<String, Object>>) promoStream.readObject(); 
        obj = (ArrayList<HashMap<String, Object>>) inStream.readObject(); 
        for (HashMap<String, Object> tmpObj : obj) { 
         promoTourname = (String) tmpObj.get("promoTour"); 
         promocodeID = (String) tmpObj.get("promocode"); 
         if (promoTourname.equals(currentTour.getObjtourName())) { 
          //if the condition is met, remove the entry from the file 
          for (HashMap<String, Object> promoTemp : promoobj) { 
           promoTourCount = (Integer) promoTemp.get("promocodeTourCount"); 
          } 

          obj.remove(tmpObj); 
          --promoTourCount; 

          ObjectOutputStream out = new ObjectOutputStream(new FileOutputStream(promoPlistPath)); 
          out.writeObject(obj); 
          out.close(); 

          ObjectOutputStream promoout = new ObjectOutputStream(new FileOutputStream(promoCountFile)); 
          HashMap<String, Object> promoCountDict = new HashMap<String, Object>(); 
          promoobj.remove(0); 
          promoCountDict.put("promocodeTourCount",promoTourCount); 
          promoobj.add(promoCountDict); 
          promoout.writeObject(promoobj); 
          promoout.close(); 


         } 
        } 


        if (obj.size() == 0 || promoTourCount == 0) { 
         promoPlistPath.delete(); 
         promoCountFile.delete(); 
        } 
       } catch (Exception e) { 
        e.printStackTrace(); 
       } 

      } 

ここでは、forループが2回目以降に繰り返し実行されている間に同時に変更例外が発生しています。

私はループの繰り返しとして毎回ファイル内のpromoTourCountの値を更新しようとしています。しかし、私はそれをやり遂げていません。私は位置0に存在する物体を削除し、その位置に新しいものを追加していた複数のオブジェクトを追加することを回避するので（promoobj.remove（0）;）

Plzを私にあなたが変更している

Answer 1

を助けますあなたが反復しているコレクション。これによりエラーが発生します。

for (HashMap<String, Object> tmpObj : obj) { 

をしかし、ここで、それから削除します：あなたがオーバーOBJここ反復

obj.remove(tmpObj); 

を私はあなたのために終了した場合にのみ別のコレクションに削除して、マップからそれらを削除するには、あなたがアイテムを保存することをお勧めしますループ。

編集：< < コード例を追加する>>

List<Integer> toRemove = new LinkedList<Integer>(); 
for (int i = 0; i < obj.size(); i++) { 
    HashMap<String, Object> tmpObj = obj.get(i); 

    if (/* something */) { 
     /* ... */ 
     /* obj.remove(tmpObj); replaced with*/ 
     toRemove.add(0, i); // notice that I add bigger indices first 
    } 
} 
// Here we make the removal from bigger indices to smaller ones 
// Notice that we iterate and remove from different collections. 
for (Integer indexToDelete : toRemove) { 
    obj.remove(indexToDelete); 
} 

あなたは要素を削除したいとき、これは基本的な考え方です。ただし、objを変更したものをすぐにforループで出力する必要があります。その後、おそらくインデックスのハックのビットは、あなたのためのより良い仕事をします：

for (int i = 0; i < obj.size(); i++) { 
    HashMap<String, Object> tmpObj = obj.get(i); 

    if (/* something */) { 
     /* ... */ 
     /* obj.remove(tmpObj); replaced with*/ 
     obj.remove(i); // We erase the element but this time we do not use enhanced for loop which is ok. 
     i--; // we decrease the index because th enumber of elements decreased. Index hack. Ugly. 
     System.out.println(obj); // modified obj :) 
    } 
}