Tastatur.wurdeGedrueckt() macht nicht das, was ich will :/

[HerrBerger]

Hallo Communitiy

Ich hatte mir vorgenommen zum Einstieg in Java ein einfaches Dart-Spiel zu programmieren.
Zu Beginn des Spiels soll der Pfeil am linken Bildschirmran gerade herunterfallen. Wird die Tastatur einmal gedrückt, so bleibt er auf seiner aktuellen Höhe und fängt an sich zu drehen. Beim zweiten mal wird auch seine Ausrichtung fixiert.

Der betreffende Programmcode ist folgender:

[Java]public void Pfeilwurf()
{
Oberflaeche();
BST.bewegeBis(200,50);
//Fallen
for(int x = 10; x <= 500; x = x+1)
{
ZeichnePfeil(50,x);
UH.warte(50);
BST.hoch();
BST.bewegeBis(45,x-10);
BST.setzeFarbe(Farbe.WEISS);
BST.setzeFuellmuster(Muster.GEFUELLT);
BST.zeichneRechteck(40,30);
if(x == 500)
{
x = 10;
}
if(TST.wurdeGedrueckt())
{
BSTReset();
ZeichnePfeil(50,x);
TST.weiter();
break;
}
}
//Drehen
double vpd = BST.vPosition();
int vp = (int)vpd;
UH.warte(1000);
boolean beenden = false;
for(int x = 30; x >= -30; x--)
{
BST.dreheBis(x);
ZeichnePfeil(50,vp);
UH.warte(10);
BST.bewegeBis(45,vp-50);
BST.setzeFarbe(Farbe.WEISS);
BST.setzeFuellmuster(Muster.GEFUELLT);
BST.zeichneRechteck(40,100);
if(x == -30)
{
for(x = -30; x <= 30; x++)
{
BST.dreheBis(x);
ZeichnePfeil(50,vp);
UH.warte(10);
BST.bewegeBis(45,vp-50);
BST.setzeFarbe(Farbe.WEISS);
BST.setzeFuellmuster(Muster.GEFUELLT);
BST.zeichneRechteck(40,100);
if(TST.wurdeGedrueckt())
{
beenden = true;
TST.weiter();
break;
}
}
}
if(TST.wurdeGedrueckt())
{
beenden = true;
TST.weiter();
}
if(beenden = true)
{
BST.dreheBis(x);
ZeichnePfeil(50,vp);
break;
}
}
}
[/code]
Das Problem liegt meiner Meinung nach an der Funktion Tastatur.weiter(). Denn sobald die Tastatur das erste Mal gedrückt wurde hält der Pfeil an (was er ja auch soll), aber bleibt auch in seiner Ausrichtung fixiert (30° schräg nach oben).
Was habe ich falsch gemacht?

 

[Gucky]

Benutze bitte JAVA TAGS um deinen Code lesbarer zu machen.

Wir haben keine Ahnung, was die einzelnen Methoden (in Java gibt es keine Funktionen) tun, sind ein paar Hintergrundinformationen wünschenswert. Was macht Tastatur#weiter()?
Was tun die ganzen Methoden? Halten sie den aktuellen Thread so lange an, bis etwas passiert oder fragen sie nur ein Boolean ab und das war's?

Die Methode ist viel zu lang. Wenn du das Gesamtproblem in viele Kleine (Methoden) zerlegst, dann sollte die Lösung leichter zu erreichen sein.

Wie ich sowas spontan lösen würde:
Ein Thread ist für das Zeichnen des Pfeils zuständig und tut auch nur das. Ein Anderer ist für das Reagieren auf Ereignisse vom Benutzer. Wenn der Benutzer ein Ereignis ausgelöst hat, so wird der Zeichenthread darüber in Kenntnis gesetzt, dass er etwas Anderes zu zeichnen hat. Zum Beispiel wird eine Variable status einen hochgezählt und noch weitere Informationen gesetzt und so weiß der Zeichenthread, was er zeichnen soll.
Für das Hochundrunterfahren wäre status auf 0.
Dann löst der Benutzer ein Ereignis aus, status wird auf 1 gesetzt und die aktuelle Höhe wird gespeichert. Der Pfeil rotiert.
Der Benutzer löst noch ein Ereignis aus, status wird auf 2 gesetzt, der Winkel wird gespeichert und noch etwas passiert (Kraftanzeige pendelt o.Ä.).
Das ist natürlich noch nicht der Weisheit letzter Schluss aber so sollte es erst einmal funktionieren.

 

[HerrBerger]

Oh natürlich, tut mir leid...

Ihr kennt natürlich alle BlueJ und die SuM-Bibliotheken nicht,mit denen ich programmiere. Wie konnte ich das vergessen
Aber danke trotzdem für die Antwort!