Rätselhaftes Verhalten von Collections

[Jay1980]

Servus,

ich habe drei Klassen einmal eine Gui Klasse die den MouseListener von Swing implementiert und die dann die Methode addPoint der Logikklasse bei einem Mausklick abfeuert. Die dritte Klasse ist ein abgeleitetes JPanel, das dann den Zeichenvorgang übernimmt.

Die Punkte liegen in den Klassenattributen in ArrayLists aber ich kann mir gerade nicht erklären warum, es so ist, dass meine Debugmeldungen ein so wirres Ergebnis zurückliefern ... es scheint, dass ich völlig daneben liege, wenn es um die tiefen oder flachen Kopien geht und ob ich nun auf der Referenz arbeite oder auf einer Wertkopie.

Ich habe mal ein kompilierbares Beispiel angehangen.

Anwendung unter Anwendung - Geometrie - ConvexHull starten. Ich habe die Debugmeldungen drin gelassen, bei den ersten beiden JOptionPanes sieht man wie es laufen sollte, wenn man mittels Linksklick aber einen Punkt der Zeichenfläche zufügt, sollten die anderen sichtbar bleiben - tun sie aber nicht?!

Danke falls sich das jemand ansieht, ich bin mit meinem Latein aktuell ziemlich durch...

EDIT: das mit dem Beispiel mache ich zum ersten Mal, hoffe das habe ich so gemacht wie es sein sollte, ansonsten bin ich für jegliche Tipps dankbar. - (geänderte) Datei ist zwei Posts weiter unten.
EDIT2: ich habe vier Posts weiter unten die Links zum Pastebin gesetzt, jetzt kann man die Klassen einfach sehen oder via c&p in die IDE nehmen. Das mit dem File-Export habe ich zum wiederholten Male nicht gesch... bekommen.

 

[SlaterB]

in dem Jar sind nur .class-Dateien, was soll man denn damit machen?

 

[Jay1980]

Ha, man lernt nie aus ...

habe das File mal geändert und beim Export in Eclipse einen Haken mehr gesetzt. Jetzt sollte es klappen.

 

[SlaterB]

die Klasse JobaConvexHull fehlt, da kann man recht wenig machen,

von der wird eine ArrayList an das Panel weitergereicht,
neue Punkte anscheinend nur in die JobaConvexHull eingefügt,
ob die sie nun auch in die ArrayList des Panels schreib oder nicht, ist ohne Code nicht ersichtlich, aber das müsste man doch genau debuggen können?

 

[Jay1980]

Danke Slater fürs darauf aufmerksam machen, dass immer noch nicht alles da ist, schwere Geburt.

Ich werde nun mal die Links zu den drei Klassen in einem pastebin posten, dann kann sie jeder sehen oder via copy & paste in seine IDE nehmen.

Das Synchronisieren habe ich mir so gedacht: Ich arbeite auf der Collection in der Klasse JobaConvexHull, dann bevor es ans Zeichnen geht, werden die Collections gecleart, etwa diese, die die alten vorab berechneten Werte speicherten, damit diese wieder frisch sind. Damit die Collections aus der Klasse JobaDrawPanelConvexHull nicht die Klassen aus JobaConvexHull beeinflussen, bezieht die Zeichenpanelklasse alle Daten aus einer Kopie des JobaConvexHullObjekts, hier 'jchcopy'.

Also hier die drei Klassen:
JobaGui
Java pastebin - collaborative debugging tool

Klasse JobaConvexHull:
Java pastebin - collaborative debugging tool

Klasse JobaDrawPanelConvexHull
Java pastebin - collaborative debugging tool

 

[SlaterB]

hier mal DebuggingCode:

		// neue Daten mit initialize() schicken
		System.out.println("vor init: "+pointsToDrawJobaGui);
		jdpch.initialize(flag, pointsToDrawJobaGui, newCoordinateSystemMaxXJobaGui, 
                    newCoordinateSystemMaxYJobaGui); // jdpch-clear wird hier  automatisch vorgeschalten ...
		System.out.println("nach init: "+pointsToDrawJobaGui);

Ausgabe:

vor init: [java.awt.Point[x=96,y=270], java.awt.Point[..]
nach init: [java.awt.Point[x=96,y=270], java.awt.Point[..]
vor init: [java.awt.Point[x=96,y=270], java.awt.Point[..]
nach init: []

beim zweiten initialize-Aufruf leert also das Panel selber die Liste,
ich vermute, dass es im Panel an
clear( String flag )
und dort an
pointsToDraw.clear();
liegt,

denn dieses alte pointsToDraw scheint genau die aktuelle Punkt-Liste zu sein,
also in der Richtung weiterarbeiten,
entweder die alte Liste nicht leeren (was stört sich das Panel daran?) oder immer eine neue Liste übergeben
oder das Panel legt sich selber eine Kopie der Liste an und darf die dann auch leeren oder was auch immer