MStumpp / pse13ss13-app

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

Löschen von Wegpunkten geht nicht (Auf Karte per Fling) #198

Open themk opened 11 years ago

themk commented 11 years ago

Fehler siehe oben. Teile des Programms reagieren dann auch für eine Weile nicht mehr. Oder zumindest lassen sich die noch angezeigten Wegpunkte nicht anklicken.

Im Moment vermute ich, dass der Fling nicht richtig erkannt wird. @Profreak Liegt es daran? Oder hat das was mit der neuen move Methode zu tun?

lootwitch commented 11 years ago

bin an der Stelle am umbauen wegem dem abfangen den Finger zoom ... es funktioniert noch aber hab vorhin gesehndas jemand die Geschwindigkeit die erreicht werden muss geändert hat ... wird noch gemacht

lootwitch commented 11 years ago

so das löschen geht wieder doch ein Problem bleibt und zwar: @themk Ich erklär mal kurz was: Der Fling wird erst erkannt wenn der Finger vom Display runter kommt. Das bedeutet aber auch das zuerst moveActiveWaypointMoveOnly zuerst aufgeufen wird bevor erkannt wird das es ein löschen Befehl war... deswegen übernimmt der RouteController die zulezt gesetzte Route und löscht dann den aktiven Wegpunkt und dann später kommt die neue Route vom Server wieder und ersetzt die Route zu aktiven Wegpunkt ... das Problem ist jetzt den gibt es nicht mehr!.^^^

Das heißt du musst noch prüfen ob der Wegpunkt noch existiert wenn du die entsprechende Route ersetzt

Ich kann leider im View die erkenn Reihenfolge nicht verhindern ..Außer ich mach ne Runnable rein und lass ihn ne Zeit schlafen und warte ob ein Fling kommt XD notfalls mach ich des aber iwi wäre das unschön

floscher commented 11 years ago

Außer ich mach ne Runnable rein und lass ihn ne Zeit schlafen und warte ob ein Fling kommt XD

Wenn Du so etwas einbaust, mach bitte lieber folgendes:

Thread t = new Thread();
t.start();
t.join();

statt

Thread t = new Thread();
t.start();
while(t.isAlive()) {
    Thread.sleep(500);
}

Das bewirkt das selbe wie so eine sleep-Schleife, ist aber im Gegensatz dazu kein "active waiting", wie es Herr Batz auch schon ein paar Mal angeprangert hat. Da powert sich der Prozessor nur sinnlos aus.

lootwitch commented 11 years ago

ich hba das leider noch nie benutzt^^ außerdem sind des nur 500 - 1000 ms die ich einstelle ... mach du des ich schau mir des dann später an (ich hab schon was gemacht du findest es im WaypointView zwischen den Wörter PATCH

lootwitch commented 11 years ago

@themk der RouteController scheint den verschobenen aktiven Wegpunkt nicht zu normalisieren

floscher commented 11 years ago

OK, in dem Fall habe ich dich falsch verstanden. Du wartest ja nicht in einer Schleife und auch nicht auf das Beenden eines Thread (hatte sich für mich so angehört).

So etwas, wie ich oben geschrieben habe, habe ich aber schonmal irgendwo gesehen, deshalb meine Bedenken dazu.

themk commented 11 years ago

Gelesen, werde sobald wie möglich testen. (bin aber gerade noch am Verschieben von Bumerangpunkten usw.)

lootwitch commented 11 years ago

ja funst so sollte morgen auch noch funsen kümmer dich um deine anderen Brände

themk commented 11 years ago

@Profreak Das Problem mit dem Ersetzen der neuen Route durch die vom Server (wie du es oben beschrieben hast) hab ich nicht. Hast du das noch? Wenn nicht, dann können wir doch closen oder?

lootwitch commented 11 years ago

@floscher danke für den code @themk existiert ist aber durch das runnable unschön gelöst also vergiss den issue einfach

themk commented 11 years ago

Können wir dann closen?

lootwitch commented 11 years ago

nein für später