Schleife fährt sich fest

[TheNonsenseGuy]

Hallo,
ich bin relativ neu und spiele gerade etwas mit dem Figuren-Set von BlueJ herum. Ich versuche einen Kreis auf der 300x300 Leinwand "herumrollen" zu lassen und ihn an den Wänden abprallen zu lassen. Um das zu erreichen, habe ich String-Variable "richtung" ins Leben gerufen (dabei ist "ur" "unten rechts","ol" "oben links", usw.) Rollen tut er jedoch nur bis zur ganz rechten Seite, dort bleibt er stehen, obwohl die Variable "richtung" wie geplant "ul" einnimmt. Weiterrechnen tut das Programm auch, passieren tut trotzdem nichts. Es hängt also irgendwo fest. Ich weiß beim besten Willen allerdings nicht wo. Ich hoffe ihr könnt mir helfen...

 public void BouncenGo(){
        bouncen = true;
        richtung="ur";
        while(bouncen=true)
        {
            while(richtung=="ur")
            {
             if(xPosition<270)
             {
                if(yPosition<270)
                {
                    horizontalBewegen(1);
                    vertikalBewegen(1);
                }
                else
                {
                 richtung="or";
                }
            }
            else
            {
            richtung="ul";
            }
        }
       
            while(richtung=="ul")
            {
             if(xPosition<30)
             {
                if(yPosition<270)
                {
                    horizontalBewegen(-1);
                    vertikalBewegen(1);
                }
                else
                {
                 richtung="ol";
                }
            }
            else
            {
            richtung="ur";
            }
        }
   
          while(richtung=="ol")
            {
             if(xPosition<30)
             {
                if(yPosition<30)
                {
                    horizontalBewegen(-1);
                    vertikalBewegen(-1);
                }
                else
                {
                 richtung="ul";
                }
            }
            else
            {
            richtung="or";
            }
           
        }
          while(richtung=="or")
            {
             if(xPosition<270)
             {
                if(yPosition<30)
                {
                    horizontalBewegen(1);
                    vertikalBewegen(-1);
                }
                else
                {
                 richtung="ur";
                }
            }
            else
            {
            richtung="ol";
            }
           
        }
    }
}

 

[VfL_Freak]

Moin,
wo werden denn die Werte vom 'xPosition' und 'yPosition' verändert ???
VG Klaus

 

[tk22k]

Vielleicht könnte man vorher eine besser Übersicht mit einer switch case Verzweigung einbringen

 

[Flown]

Ohne den Code durchgesehen zu haben, Strings vergleicht man mit equals und nicht mit ==

 

[Joose]

Außerdem setzt du in der Bedingung der äußerten while-Schleife den Wert bouncen auf true.

 

[X5-599]

@Flown @Joose Ist zwar alles richtig hat aber beides nichts mit seinem Problem hier zu tun.

Ich gehe mal davon aus, dass am Start xPosition und yPosition 0 sind. In der ersten while Schleife für "ur" werden beide solange erhöht bis sie 270 sind. Dann wird die Richtung auf "ul" gesetzt.
In der while Schleife für "ul" trifft die erste if Abfrage "if(xPosition<30)" schon nicht mehr zu und ändert somit wieder die Richtung auf "ur". Und schon sind wir wieder in der ersten Schleife, die die Richtung sofort auf "ul" ändert, da die if Bedingung "if(xPosition<270)" nicht wahr ist. Und das geht dann ohne Veränderung immer so weiter.

Das sind natürlich alles nur vermutungen, da ja nicht bekannt ist was genau die Methoden "horizontalBewegen/vertikalBewegen" machen. Ich habe zB angenommen, dass sie die x/y Variablen lediglich inkrementieren/dekrementieren...

 

[Flown]

@Flown @Joose Ist zwar alles richtig hat aber beides nichts mit seinem Problem hier zu tun.

Ich hab ja auch ein allgemeines Problem aufgeführt und, wenn das schon nicht passt, warum weiterlesen?

 

[TheNonsenseGuy]

Entschuldigt bitte die fehlenden Infos:
1. Danke für den Tipp mit dem Vergleichen von String-Texten. Problem tritt trotzdem weiterhin auf.
2. Der Ball Bewegt sich nicht direkt in die Ecke und bleibt dort hängen. Seine Ursprungskoordinaten sind weiter rechts. Zumindest darauf habe ich geachtet.
3. So sieht der Code der "bewegen"-Methoden aus:

    public void horizontalBewegen(int entfernung) {
        loeschen();
        xPosition += entfernung;
        zeichnen();
    }
    public void vertikalBewegen(int entfernung) {
        loeschen();
        yPosition += entfernung;
        zeichnen();
    }

Mit "loeschen" wird der Kreis von der Leinwand entfernt und mit "zeichnen" wieder mit gegebenen Werten gezeichnet.

 

[Thallius]

Ich denke mal das Problem liegt ganz wo anders. Du änderst wahrscheinlich die Position des Balls im Maim Thread und deshalb zeigt er nur die Endposition an und es bewegt sich nichts.

Gruß

Claus

 

[TheNonsenseGuy]

Er bewegt sich ja. Bis zur rechten Seite. Dann ist Schluss. Obwohl "richtung" "ul" annimmt geht er nicht in die Schleife für "ul", sondern macht halt gar nichts mehr. Was auch merkwürdig ist: Wenn "richtung" von Anfang an auf "ul","or","ol" setze, verhält er sich genau so. Setze ich ihn, aber auf einen Wert, der nicht einer meiner Schleifen erwähnt ist, wie "test", dann bewegt er sich gar nicht. Außerdem: Wenn ich die "ur"-Schleife zur "ul"-Schleife mache und die "ul"-Schleife zur "ur"-Schleife mache, indem ich das einfach in der while-Bedingung ändere, dann wird nicht wie erwartet eine der Schleifen durchgearbeitet. Tatsächlich aber passiert gar nichts. Es ist, als würde das Programm nur die 1. Schleife sehen und die anderen garnicht erst abfragen....

 

[Thallius]

Vielleicht fängst du einfach mal damit an deine schleifen und in Anweisungen richtig einzurücken. Dann wird dir glaube ich der Fehler sehr schnell auffallen...

 

[TheNonsenseGuy]

Ich habe jetzt die letzten Stunden mit Try&Error verbracht. Es scheint an den Bedingungen der "if"s zu tun. Ich bin langsam echt am Ende.

 

[Thallius]

Nein es liegt daran das du falsch geklammert hast. Aber wenn du das nicht aufräumst wirst du auch weiter nur stundenlang sinnlos rumprobieren

 

[TheNonsenseGuy]

Hm merkwürdig. Als ich die Bedingungen der "if"s veränderte, so dass sie immer zutreffen, funktionierte das Ganze nämlich. (Hat halt bloß nicht an der Wand angehalten/die Richtung geändert, weil die End-Bedingungen fehlen)

 

[TheNonsenseGuy]

Mittlerweile hab ich das Problem irgendwie gelöst. Geklammert habe ich komplett richtig, bloß gab es ein Problem mit dem "<" und dem ">". Also eigentlich komplett banal.....
Trotzdem danke ich allen Helfern