spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Du kannst auch fixe Wert-Arrays erstellen und drüber loopen oder schöne Methoden mit Referenz-Parametern erstellen. Aber wirklich hübscher wird es wohl kaum werden, das ist einfach nur Fleißarbeit. Wobei die Referenz-Variante gar nicht so viel Fleiß wäre.

Btw, was soll denn "RectEnemyRectangle" eigentlich sein? Ich meine ist ja hübsch, dass Du Kommentare dran geschrieben hast, aber mit einer vernünftigen Methoden-Benennung wäre das gleich besser gewesen. Es empfiehlt sich da generell mit einem Verb zu beginnen. Mit Rectangles hat die Methode ja irgendwie auch nichts am Hut und mit einem anderen Enemy auch nicht. "CalculateMovement" oder "CalculateDirection" wäre wohl irgendwie aussagekräftiger gewesen.
Den Klassen-Namen in einer Methode oder Variable zu wiederholen ist auch unsinnig. Man weiß, dass es eine Eigenschaft der Klasse ist, in der sie steckt. So sind z.B. playerHitPoints in einer Player-Klasse Quatsch und "buttonColor" in einer Klasse "Button" ebenfalls. Genauso verhält es sich auch mit den Methoden. Man sieht doch sofort, was enemy.calculateMovement() tut. Es berechnet die Bewegung des Enemies. Aber was wäre denn bitte enemy.calculateEnemyMovement()? Berechnet der Gegner jetzt Bewegung anderer Gegner oder was genau tut das Ding überhaupt?

Welche vier Koordinaten meinst du jetzt? Naja normalerweise würde ich mir eine Bewegungsrichtung berechnen. Dann würde ich die Bewegung in diese Richtung berechnen und danach auf Kollision/Probleme prüfen. Dann kann man entsprechend reagieren und die Position zum Beispiel zurück setzen oder die Bewegungsrichtung ändern oder was auch immer man möchte. Und bevor man Code an vielen Stellen doppelt schreibt, schreibt man lieber eine Funktion und setzt diese an den jeweiligen Stellen ein.

Die direction-Variable darf überhaupt nicht statisch sein. Wenn sie das ist, ist das kein Wunder das sie den Wert dauernd wechselt, sobald mehrere Instanzen vorkommen. Innerhalb deiner switch-Anweisung setzt du nur eine der 4 Richtungen fest ohne jede if-Anweisung. Außerhalb prüfst du mit deinen if's ob die Grenzen überschritten wurden und legst ggf. die Richtung neu fest.

Du musst doch nur genau 4 Zustände beachten. Alle lassen sich mit einer einzigen Methode behandeln, wenn Du die Parameter entsprechend baust.

Und trotzdem lassen sich alle in einer einzigen Methode behandeln, die auch nur ein einziges IF beinhaltet.

Nö, meine ich nicht. Ist aber schon mal besser als die bisherige Lösung.