Listiterator в игре для Android - PullRequest
Новая
       14

Listiterator в игре для Android

0 голосов
/ 21 июля 2011

У меня есть два ArrayList, которые я использую, один для вражеских спрайтов, а другой для пуль.Когда я запускаю приложение, оно время от времени падает на эмуляторе, и в течение нескольких уровней оно падает на устройстве при столкновении.Журнал постоянно говорит мне, что у меня есть простая ошибка ListIterator.Я хотел знать, как мне реализовать ListIterator.Я не использовал ListIterator, я искал информацию и примеры, но я немного запутался, как на самом деле это сделать.Функция add () вызывается в методе onTouch моего класса представления, и столкновение происходит внутри моего класса потока в его собственных методах столкновения.

Журнал: 

07-20 19:57:46.604: ERROR/AndroidRuntime(234): Uncaught handler: thread Thread-9 exiting due to uncaught exception
07-20 19:57:46.613: ERROR/AndroidRuntime(234): java.util.ConcurrentModificationException
07-20 19:57:46.613: ERROR/AndroidRuntime(234):     at java.util.AbstractList$SimpleListIterator.next(AbstractList.java:64)
07-20 19:57:46.613: ERROR/AndroidRuntime(234):     at com.android.hitmanassault.HitmanView$HitmanThread.startGame(HitmanView.java:333)
07-20 19:57:46.613: ERROR/AndroidRuntime(234):     at com.android.hitmanassault.HitmanView$HitmanThread.gameStart(HitmanView.java:290)
07-20 19:57:46.613: ERROR/AndroidRuntime(234):     at com.android.hitmanassault.HitmanView$HitmanThread.updateGame(HitmanView.java:393)
07-20 19:57:46.613: ERROR/AndroidRuntime(234):     at com.android.hitmanassault.HitmanView$HitmanThread.run(HitmanView.java:237)

Проверка столкновения: 

private void startGame(){
            synchronized(mSurfaceHolder){
                for(Beam bullet: beam){
                    for(Sprite sprite: sprites){
                        if(checkCollision(sprite, bullet)){
                            sprites.remove(sprite);
                            beam.remove(bullet);
                            mScore = mScore + 1;
                            break;
                        }               
                    }
                }
            }       
        }

Метод столкновения: 

public boolean checkCollision(Sprite sprite, Beam bullet){

            boolean retValue = false;
            int SpriteX = sprite.getX();
            int SpriteY = sprite.getY();
            int SpriteXS = sprite.getX() + sprite.getWidth();
            int SpriteYS = sprite.getY() + sprite.getHeight();  
            int BeamX = bullet.getX();
            int BeamY = bullet.getY();
            int BeamXS = bullet.getX() + bullet.getBitmap().getWidth();
            int BeamYS = bullet.getY() + bullet.getBitmap().getHeight();


            if ((BeamX >= SpriteX && BeamX <= SpriteXS) || (BeamXS >= SpriteX && BeamXS <= SpriteXS)) {
                if ((BeamY >= SpriteY && BeamY <= SpriteYS) || (BeamYS >= SpriteY && BeamYS <= SpriteYS)) {
                    retValue = true;
                }
            }   
            return retValue;
        }

1 Ответ

2 голосов
/ 21 июля 2011

Если вы добавите элемент в список, пока другой поток выполняет итерацию списка, вы получите ConcurrentModificationException .

Вы можете попробовать ConcurrentLinkedQueue , но в зависимости от вашеготребование к модели согласованности игры, вам следует переосмыслить архитектуру.

Это проблема. 

for(Sprite sprite: sprites) { <------- Iterate
    if(checkCollision(sprite, bullet)) {
        sprites.remove(sprite); <-------- Modify
...

Быстрое решение: вы можете собрать все элементы, которые хотите удалить, а затем удалить их последля цикла. 

ArrayList<Beam> toBeRemoveBeams = new ArrayList<Beam>();
ArrayList<Sprite> toBeRemoveSprites = new ArrayList<Sprite>();
for(Beam bullet: beam){
    for(Sprite sprite: sprites){
        if(checkCollision(sprite, bullet)){
            toBeRemoveBeams.add(beam);
            toBeRemoveSprites.add(sprite);
            mScore = mScore + 1;
            break;
        }               
    }
}
beam.removeAll(toBeRemoveBeams);
sprites.removeAll(toBeRemoveSprites );
...