Zugriff synchronisieren

[Guest]

Hi, ich bastele an einem kleinen 2D-Spiel mit BufferStrategy.

Das Spiel besteht aus dem Thread, der rendert und dann gibt es da noch die Event Listener.
Jedenfalls befinden sich die Objekte, die gerendert werden müssen in einem Vector. Allerdings werden diesem Vektor Objekte hinzugefügt und aus ihm welche gelöscht.

Weil das häufig parallel passiert bekomme ich eine "ConcurrentModificationException". Ich dachte ich könnte das über ein einfaches "synchronize" lösen, aber das hat nicht funktioniert. Ich zeige euch mal etwas Code:

public class Editor extends Thread{
    
	private BufferedCanvas canvas;
     
        private Vector<Shape> shapes = new Vector<Shape>();
        private Shape currentShape;
        
        //...

	public void run() {
		while (true) {
			Graphics2D g = canvas.getGraphicsContext();
			g.setColor(Color.WHITE);
			g.fillRect(0,0,800,600);
                        g.setColor(Color.RED);
                        
                        if (currentShape != null) g.draw(currentShape);
                        for (Shape shape: shapes) g.draw(shape);
                        
                        canvas.draw();
			try { Thread.sleep(10); } catch (Exception e) {}
		}
	}

        
        private class MouseInputHandler extends MouseAdapter{           
            public void mousePressed(MouseEvent e){
                currentShape = new Line2D.Float(e.getPoint(),e.getPoint());
            }
           
            public void mouseReleased(MouseEvent e){
              if (currentShape != null) {
                  synchronized (currentShape){
                    synchronized (shapes){
                        shapes.add(currentShape);                      
                    }
                    currentShape = null;
                  }
              }  
            }
        }       
}

Wie ihr seht kann auf das Shape "currentShape" und den Vekor "Shapes" von zwei Seiten zugegriffen werden. Wie lös ich das Problem, dass es dabei "knallt"?
"InvokeLater" kann ich hier nicht verwenden, weil ich selber neuzeichne, oder?

 

[SlaterB]

die run-Operation muss auch
synchronized (shapes){
}
benutzen, dann ist doppelter Zugriff ausgeschlossen,

nur ein synchronized bringt nix

 

[Guest]

Funktioniert, danke!

Trotzdem noch eine Frage: Kann es nur bei schreibendem oder auch bei lesendem Zugriff von zwei Seiten zu Problemen kommen?

 

[SlaterB]

dürfte nur bei Modification, Änderung so sein

 

[Wildcard]

Vector ist bereits synchronisiert.
Dein Problem ist die foreach Schleife bzw. der Iterator den sie verwendet.
Während ein Iterator läuft, darf die Collection nicht (bzw. nur über den Iterator) verändert werden.
Mit Threads hat das also nur indirekt zu tun. Ohne foreach würdest du diese Exception nicht bekommen.
Aber dein Programm an sich ist ziemlich daneben.
Nur ein einziger Thread darf zeichnen und das ist der Event Dispatcher Thread.

Hier wird der Fehler beschrieben den du begehst.
http://www.java-forum.org/de/topic46550_zeichnen-swing-tutorial.html

 

[Guest]

Okay, super, danke!

Gut dann pack ich das in den EDT, ist ja an für sich kein Problem. Hat im Übrigen so auch keine Probleme gegeben.

 

[Guest]

Nochmal eine Nachfrage:

Das heißt ich kann beide "synchronized" weglassen, wenn ich die for-each ersetze?

 

[Wildcard]

kannst du