2016-05-08 120 views
0

我有一個程序,其中包含一個發射子彈(小圓圈)的太空船(大圓圈),一段時間後消失。 每次運行代碼的特定部分時,都會生成一個錯誤(但會繼續運行)。錯誤:線程「JavaFX應用程序線程」中的異常

所以我敢肯定,問題是出在以下幾行

private ArrayList<Bullet> bullets = new ArrayList<Bullet>(); 
//[...] 
for (Bullet bullet : bullets) { 
    if (bullet.getLoops() > 50) { 

    bullets.remove(bullet); // specially in this line, 
          // because if I delete this, the error doesn't show up! 
          // I think it has something to do with the ArrayList. 

    } else { 
    bullet.next(); 
    gc.strokeOval(bullet.getX() - 3, bullet.getY() - 3, 6, 6); 
    } 
} 

,這是我得到的錯誤:

Exception in thread "JavaFX Application Thread" java.util.ConcurrentModificationException 
    at java.util.ArrayList$Itr.checkForComodification(ArrayList.java:901) 
    at java.util.ArrayList$Itr.next(ArrayList.java:851) 
    at sample.Main$4.handle(Main.java:131) 
    at javafx.animation.AnimationTimer$AnimationTimerReceiver.lambda$handle$484(AnimationTimer.java:57) 
    at java.security.AccessController.doPrivileged(Native Method) 
    at javafx.animation.AnimationTimer$AnimationTimerReceiver.handle(AnimationTimer.java:56) 
    at com.sun.scenario.animation.AbstractMasterTimer.timePulseImpl(AbstractMasterTimer.java:357) 
    at com.sun.scenario.animation.AbstractMasterTimer$MainLoop.run(AbstractMasterTimer.java:267) 
    at com.sun.javafx.tk.quantum.QuantumToolkit.pulse(QuantumToolkit.java:506) 
    at com.sun.javafx.tk.quantum.QuantumToolkit.pulse(QuantumToolkit.java:490) 
    at com.sun.javafx.tk.quantum.QuantumToolkit.lambda$runToolkit$404(QuantumToolkit.java:319) 
    at com.sun.glass.ui.InvokeLaterDispatcher$Future.run(InvokeLaterDispatcher.java:95) 
    at com.sun.glass.ui.win.WinApplication._runLoop(Native Method) 
    at com.sun.glass.ui.win.WinApplication.lambda$null$148(WinApplication.java:191) 
    at java.lang.Thread.run(Thread.java:745) 

我將很高興提供在其他任何需要的信息請求。

乾杯。 (對不起,我的英語,我是德國人,但我在學校自從幾年有英語。)

PS:如果有人需要mainclass的全碼:

package sample; 

import javafx.animation.AnimationTimer; 
import javafx.application.Application; 
import javafx.event.EventHandler; 
import javafx.fxml.FXMLLoader; 
import javafx.scene.Group; 
import javafx.scene.Scene; 
import javafx.scene.canvas.Canvas; 
import javafx.scene.canvas.GraphicsContext; 
import javafx.scene.input.KeyEvent; 
import javafx.scene.input.MouseEvent; 
import javafx.scene.paint.Color; 
import javafx.scene.text.Font; 
import javafx.scene.text.FontWeight; 
import javafx.stage.Stage; 

import java.util.ArrayList; 

public class Main extends Application { 

    private Player p = new Player(100, 100, 0); 
    private ArrayList<Bullet> bullets = new ArrayList<Bullet>(); 

    private static boolean left; 
    private static boolean right; 
    private static boolean up; 
    private static boolean down; 
    private static boolean shoot; 

    @Override 
    public void start(Stage primaryStage) throws Exception { 
     FXMLLoader.load(getClass().getResource("sample.fxml")); 
     primaryStage.setTitle("Hello World"); 
     Group root0 = new Group(); 
     Scene theScene = new Scene(root0, 500, 500); 
     primaryStage.setScene(theScene); 
     Canvas canvas = new Canvas(500, 500); 
     root0.getChildren().add(canvas); 


     theScene.setOnMouseMoved(
       new EventHandler<MouseEvent>() { 
        public void handle(MouseEvent e) { 
         Vector a = new Vector(p.getX(), -p.getY()); 
         Vector b = new Vector(e.getX(), -e.getY()); 
         Vector c = new Vector(p.getX(), -p.getY() + 1); 
         p.setRotation(Vector.getAngle(a, b, c)); 
        } 
       } 
     ); 
     theScene.setOnKeyPressed(
       new EventHandler<KeyEvent>() { 
        public void handle(KeyEvent e) { 
         String code = e.getCode().toString(); 
         if (code.equals("A")) { 
          left = true; 
         } 
         if (code.equals("D")) { 
          right = true; 
         } 
         if (code.equals("W")) { 
          up = true; 
         } 
         if (code.equals("S")) { 
          down = true; 
         } 
         if (code.equals("SPACE")) { 
          if (!PlayerShot()) shoot = true; 
         } 
        } 
       } 
     ); 
     theScene.setOnKeyReleased(
       new EventHandler<KeyEvent>() { 
        public void handle(KeyEvent e) { 
         String code = e.getCode().toString(); 
         if (code.equals("A")) { 
          left = false; 
         } 
         if (code.equals("D")) { 
          right = false; 
         } 
         if (code.equals("W")) { 
          up = false; 
         } 
         if (code.equals("S")) { 
          down = false; 
         } 
         if (code.equals("SPACE")) { 
          shoot = false; 
         } 
        } 
       } 
     ); 
     GraphicsContext gc = canvas.getGraphicsContext2D(); 

     Font theFont = Font.font("Helvetica", FontWeight.BOLD, 24); 
     gc.setFont(theFont); 
     gc.setStroke(Color.BLACK); 
     gc.setLineWidth(1); 

     new AnimationTimer() { 
      public void handle(long currentNanoTime) { 
       // Clear the canvas 
       gc.setFill(new Color(0.85, 0.85, 1.0, 1.0)); 
       gc.fillRect(0, 0, 500, 500); 

       int x = p.getX(); 
       int y = p.getY(); 
       if (left && x >= 16) { 
        p.setX(x - 3); 
       } 
       if (right && x <= 500 - 16) { 
        p.setX(x + 3); 
       } 
       if (up && y >= 16) { 
        p.setY(y - 3); 
       } 
       if (down && y <= 500 - 16) { 
        p.setY(y + 3); 
       } 
       if (shoot) { 
        bullets.add(new Bullet(p.getX(), p.getY(), p.getRotationInVelocity(), p)); 
        shoot = false; 
       } 
       gc.setStroke(Color.BLUE); 
       gc.strokeLine(p.getX(), p.getY(), p.getX() + Math.sin(p.getRotation()) * 16, p.getY() - Math.cos(p.getRotation()) * 16); 
       gc.strokeOval(p.getX() - 16, p.getY() - 16, 32, 32); 
       gc.setStroke(Color.RED); 
       for (Bullet bullet : bullets) { 
        if (bullet.getLoops() > 50) { 
         bullets.remove(bullet); 
        } else { 
         bullet.next(); 
         gc.strokeOval(bullet.getX() - 3, bullet.getY() - 3, 6, 6); 
        } 
       } 
      } 
     }.start(); 

     primaryStage.show(); 
    } 

    public static void main(String[] args) { 
     launch(args); 
    } 

    // Hat der Spieler gerade eben schon geschossen? 
    public boolean PlayerShot() { 
     for (Bullet bullet : bullets) { 
      if(bullet.getShooter() == p)return true; 
     } 
     return false; 
    } 
} 
其他

類只是處理幾個整型和雙重瓦爾斯

+0

public void next(){ x + = velocity.getY()* 5; y + = -velocity.getX()* 5; loop ++; } //這是處理向前移動的代碼,如果我刪除它,它不會帶有「bullets.remove(bullet);」的部分。所以它不能顯示出來。但那不是我想要的解決方案! – doej1367

+0

謝謝:)! – doej1367

回答

1

你不能這樣做。您正在獲得ConcurrentModificationException

什麼是ConcurrentModificationException

來自官方的Javadoc

This exception may be thrown by methods that have detected concurrent modification of an object when such modification is not permissible.

基本上,你是遍歷列表。如果您決定刪除某個元素,可能會導致不可預知的行爲。爲了避免這種情況,拋出了這個異常。

我該如何解決?

只需使用Iterator代替循環您的ArrayListIterator使用方法add(T element)remove(T element)

+0

你這樣想? 迭代器 bulletsI = bullets.iterator(); (bulletsI.hasNext()){ if(bulletsI.next()。getLoops()> 50)bullets.remove(bulletsI.next());其他{ } else { bulletsI.next()。next(); ()bulletI.next()。getY() - 3,6,6); } } 它只是拋出更多的錯誤在我 – doej1367

+1

但謝謝! 它幫助我進一步瞭解我在正確的軌道上 – doej1367

+0

不可以。你應該每次迭代調用一次。 Next()將使迭代器進入列表中的下一個元素。如果在檢查他至少有一個元素後,每次迭代調用5次,我可以看出爲什麼會出現更多錯誤。只需在這段時間後執行此操作:Bullet currentElement = bulletsI.next();然後在這個對象上做你的東西,而不用在其他地方調用next()。 – Aurasphere

相關問題