このforeachループの何が問題ですか？??? Android

Question

私はforeachループを持っており、ベクトルの各オブジェクトを反復しています。私はコードをデバッグすると、ベクトルの最初のオブジェクトに対して正常に実行されます。しかし、それは2番目のオブジェクトのループを実行しようとすると失敗します。私はですベクトルには複数の要素があります。

for(Object shape : vecForShapes) 
     { 
      currentNode = (Drawable) shape; 
      newNode = getResources().getDrawable(R.drawable.nodered); 
      newNode.setBounds(currentNode.getBounds()); 
      vecForShapes.remove(currentNode); 
       vecForShapes.add(newNode); 
     } 

基本的に私の質問は、なぜこのループが失敗するのですか？私は本当にここで何が間違っているのか分かりません。

P.S.最終的な目的はcurrentNodeをベクターから削除し、newNodeと置き換えて、onDrawの方法でベクター全体を再描画することです。あなたは上の反復コレクションへ/からオブジェクトを削除または追加することはできません

おかげ

Answer 1

。たとえば、vecForShapes.remove(currentNode);はvecForShapesを変更します。したがって、例外はあります。

for (int i = 0; i < vecForShapes.size(); i++) { 
    currentNode = (Drawable) shape; 
    newNode = getResources().getDrawable(R.drawable.nodered); 
    newNode.setBounds(currentNode.getBounds()); 
    vecForSahpes.set(i, newNode); 
} 

これはあなたがエラーなしで欲しいものを行う必要があります。私はあなただったら

は、私はあなたがそのようにしたいの変更を行っているでしょう。

PS：本当にVectorを意味しますか？代わりにArrayListを使用することを真剣にお勧めします。それはかなり優れたパフォーマンスです。

Answer 2

「）（iterator.removeを」Iterableにforeachを使用する場合は、舞台裏でIteratorがあるだろうし、それを反復処理するときには、コレクションを変更することはできませんが、あなたは呼び出すことができます。ですから古い反復モデルfor (Iterator i = v.iterator(); i.hasNext(); .....を使用し、i.remove()と呼ぶことをお勧めします。