的Java IndexOutOfBoundsException異常

Question

我做了一個小射擊，game..It工作正常，但我想還實現了，如果火災相交，他們就會消失。我有兩個列表播放子彈和計算機子彈......但是，如果我從電腦更多的子彈或逆轉。在這裏我的循環

 for (int i = 0; i < cb.size(); i++) { 
     for (int j = 0; j < b.size(); j++) { 
      if (b.get(j).rect.intersects(cb.get(i).rect)) { 

       cb.remove(i); 
       b.remove(j); 


       continue; 

      } 
      if (cb.get(i).rect.intersects(b.get(j).rect)) { 


       b.remove(j); 
       cb.remove(i); 

       continue; 

      } 

     } 

    } 

這是我的遊戲，沃金algoritms ... http://rapidshare.com/files/364597095/ShooterGame.2.6.0.jar

Answer 1

我強烈建議對與打循環計數器從環本身。你現在小心了，稍後你不會小心（「讓我們試着在這裏進行調試」）並最終發現錯誤。

一種解決方案可能是：

• 檢查是否兩個對象相交
• 如果他們這樣做，保存參考到的thingsToRemove
• 一個單獨名單終於，經過thingsToRemove和刪除（或設置爲「空」，或-1或其他）在cb和b列出了相應的元件

Answer 2

最簡單的修改，以避免邏輯錯誤：

for (int i = 0; i < cb.size(); i++) { 
    for (int j = 0; j < b.size(); j++) { 
     if (b.get(j).rect.intersects(cb.get(i).rect)) { 
      cb.remove(i--); 
      b.remove(j--); 
     } 
    } 
} 

Answer 3

運行循環落後也應該做的伎倆：

for (int i = cb.size() - 1; i >= 0; i--) { 
     for (int j = b.size() - 1; j >= 0; j--) { 

---

編輯：該解決方案還可以遇到OOBE的。有一些未處理的情況...所以我必須推薦一個更高等級的解決方案，而不是這個。

Answer 4

正如卡爾的評論指出，第二是否應該是多餘的。

至於你IndexOutOfBounds例外，這是由以下因素導致： 當計算機子彈擊中玩家的子彈，你無論是從列表中刪除。然後使用continue繼續比較相同的計算機項目符號和其餘的項目符號。但是，那個電腦子彈已經被刪除了！因此，我建議你break而不是continue，那麼接下來的計算機子彈檢查被玩家的子彈交集。

於與他的代碼，你應該進一步降低外循環的計數器，當你通過移除子彈減少一個列表的大小羅馬提示。因此，以前是子彈＃4的是第3號子彈。所以在break之後，你不需要增加外環的計數器。

Answer 5

ñ我opinin，你可以用這種方式

for (int i = cb.size() -1; i >= 0 ; i--) { 
      boolean bremoved = false; 
     for (int j = b.size() -1 ; j >=0 ; j--) { 
      if (b.get(j).rect.intersects(cb.get(i).rect) || 
       cb.get(i).rect.intersects(b.get(j).rect)) { 
        bremoved = true; 
       b.remove(j); 
      } 
     } 
     if(bremoved) 
      cb.remove(i); 
    } 

Answer 6

寫你的代碼的問題是，您要更改列表cb的，只要你找到一個交叉點，但隨後繼續使用同一指數的大小。例如，如果cb有3個元素，b有4個，並且第3個（索引= 2）計算機項目符號與第一個玩家項目符號相交，則cb的大小將減小到2。然後，當你繼續檢查第二個玩家項目符合第三個計算機項目符號時，在cb中只剩下兩個項目，並且遊戲崩潰。

簡而言之，循環遍歷列表並同時修改它們是很困難的。

要考慮的另一件事是，例如，如果一個計算機項目符號與兩個玩家項目符號相交，他們三個都應該被刪除。但是，如果在遇到第一個玩家項目符號時刪除了計算機項目符號，則第二個玩家項目符號將保留。

實際上，可能會有子彈鏈彼此接觸，所以在完成所有交叉點之前從列表中刪除任何東西都可能導致錯誤的結果。

經過了漫長的思考過後，我纔會這麼做。它確保所有需要刪除的內容都被刪除，而且不依賴索引刪除。

ArrayList<Bullet> newB = new ArrayList<Bullet>(b); 
ArrayList<Bullet> newCB = new ArrayList<Bullet>(cb); 
for (Bullet pBullet : b) { 
    for (Bullet cBullet : cb) { 
     if (pBullet.rect.intersects(cBullet.rect)) { 
      newB.remove(pBullet); 
      newCB.remove(cBullet); 
     } 
    } 
} 
cb = newCB; 
b = newB;