MStumpp / pse13ss13-app

shares the application code for the PSE13SS13 project
1 stars 1 forks source link

Bumerangpunkt verschieben/Rundkurs erstellen #194

Closed floscher closed 11 years ago

floscher commented 11 years ago

=> Nur der Wegpunkt wird verschoben, der Rest der Route bleibt wo er war. Manchmal stürzt WalkaRound dabei auch ab

Exception: ConcurrentModificationException in CoordinateUtility:213 aufgerufen durch RouteView:63

themk commented 11 years ago

der Rest der Route bleibt wo er war

Werde ich mir ansehen.

floscher commented 11 years ago

Zu der ConcurrentModificationException (s.o.): Die ist mir bisher schon ein paar Mal untergekommen im Zusammenhang mit Route. Wo das jeweils genau war, kann ich leider nicht mehr genau sagen.

Falls jemand Zeit hat, kann er ja den Code, der mit Route in Verbindung steht auf ThreadSafety untersuchen.

themk commented 11 years ago

..am testen.. Anmerkungen: Wenn man nen Rundkurs erstellt, kommt scheinbar ein zweiter Wegpunkt an der Bumerangstelle hinzu. Werde mir das noch in den Route Klasse näher ansehen. Habe eine Vermutung. @floscher :

Concurrent Exception: Problem tritt noch auf. Grund: in CoordUtil an extractDisplayCoordinatesOutOfRouteInfo wird über die Coordinaten der Route iteriert, während an anderer Stelle an der Route etwas geändert wird. Bin das Problem fürs erste umgegangen, indem ich bei Route.getCoordinates() einen clone der Coordinaten Liste übergebe. Das ist aber gar nicht optimal für die Performance. Daher folgendes:

  1. Falls jemandem dieses Problem nochmal unterkommt, bitte unbedingt melden.
  2. Werde die Sache jetzt in ein eigenes Issue auslagern.
themk commented 11 years ago

Rundkurse werden jetzt beim Verschieben und Löschen von Bumerangpunkten gelöscht.

themk commented 11 years ago

Aus irgendeinem Grund kommt es manchmal vor, dass ein Bumerangpunkt nicht exakt am Rundkurs zu liegen scheint und der Rundkurs dann nicht gelöscht wird. Habe eine Theorie, muss es aber noch austesten. -> Reopen Sesame.

themk commented 11 years ago

Ok, hoffentlich gefixt. Wenn ihr bei der aktuellen Version noch solche Fehler (Rundkurs wird nicht erstellt oder nicht gelöscht) habt, kommentiert rein.