spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

1) Typisches Problem bei schlechtem Design und von EierLegendenWollMilchSau-Funktionen. Der Spieler muss das Fenster nicht kennen, er muss das Sprite nicht kennen, er muss das Event nicht kennen und schon gar nicht einen Spieler übergeben bekommen. Überleg Dir rein fachlich, was "Player::Move" eigentlich tun sollte und was er dazu wissen muss, damit er die Aktion durchführen kann. Im Normalfall ja maximal die Map und die Richtung. Seine aktuelle Position wird er ja hoffentlich ohnehin kennen.

2) std::list<Bomb>. Nix Array mit tausend Dimensionen. Die Sachen "Typ", und "Koordinate" sollten Attribute von "Bomb" sein.

3) Einfach abfragen reicht. Ist sicher besser als irgendwelche anderen Varianten.

4) Lies ein Buch über Strukturen und Klassen. Dringend.

5) c_spieler::SpielerMove? Erstens finde ich Englisch-Deutsch-Mix ganz schlimm. Zweitens finde ich Präfixe ganz schlimm. Drittens finde ich kleine Klassen-Namen ganz schlimm. Und viertens ist "SpielerMove" doppelt gemoppelt, da "Move" ja bereits eine Methode von "Spieler" ist.

im Grunde hat BlueCobold schon alles geschrieben

zu 1.)
wie schon geschrieben ist dein Design wohl nicht das beste
der Spieler sollte nach Möglichkeit nur sich selbst kennen und max. wissen, in welcher Umgebung er sich befindet (auf welche er aber keinen "Vollzugriff" benötigt)
zum Wissen über sich selbst würde gehören, welcher Spieler ihn kontrolliert, was für ein Aussehen er hat und (wenn man so will) auch die Position, an der er sich befindet
da für die Bewegung aber genaueres Wissen über die Umgebung erforderlich ist, kann der Spieler selbst nicht prüfen, wie weit er sich in eine bestimmte Richtung bewegen kann
das heißt, die Umgebung muss entweder gefragt werden, wie weit sich der Spieler in die Richtung bewegen kann oder die Umgebung muss sich um die Bewegung des Spielers kümmern
dann wären die Parameter nicht mehr erforderlich, da die Umgebung bereits alles weiß, was sie wissen muss (man müsste nur noch den Spieler übergeben, der bewegt werden soll und die Richtung, in die er laufen soll)

zu 2.)
ich bin mir nicht sicher, was das heißen soll, was du als Datentyp/Variable geschrieben hast (es könnte daran liegen, dass ich nicht mit C++ arbeite), aber ich gehe mal davon aus, es sollte ungefähr sowas heißen:
vector<vector<map<Type, Bomb>>> bombs (ich weiß nicht, wie entsprechende Datentypen in C++ heißen)
um an eine einzelne Bombe zu kommen, müsste man bombs[x][y][type] verwenden, wobei x und y die Koordinate darstellen und type den Typ der Bombe (vom Typ Type)
wie BlueCobold schon geschrieben hat, wäre das nicht der richtige Ansatz, da der Typ (und ggf. auch die Koordinate) Eigenschaften der Bombe sind und somit nicht als Indizes dienen sollten
du könntest darüber nachdenken, was durch die von dir verwendete Datenstruktur möglich wäre:
an jeder X-Koordinate und jeder Y-Koordinate kann von jedem Typ eine Bombe vorhanden sein
allerdings dürfte es wohl nicht das Ziel sein, dass an einer Koordinate mehrere Bomben liegen, weshalb das wohl nicht das gewünschte ist

zu 3.)
warum willst du dein Array um eine Dimension erweitern?
wann eine Bombe gelegt wurde ist wiedereinmal eine Eigenschaft der Bombe


zu 4.)
zusätzlich evtl. noch Objektorientiertes Design

zu 5.)
ich kann nur dem zustimmen, was BlueCobold geschrieben hat

Der Spieler braucht das Fenster nicht zu kennen. Der Spieler ist ein abstraktes Modell und hat mit der Repräsentation Deines Programms als Fenster nichts zu tun.

alternativ könntest du auch erst die Karte fragen, ob die neue Position gültig ist und ihn dann verschieben
allerdings hast du immer das Problem, dass der Spieler sich (abhängig von der Geschwindigkeit) nicht bis an die Wand ran bewegen kann, sondern ggf. mit geringem Abstand zu dieser stehen bleibt

besser wäre es, wenn du deine Bewegen-Funktion so wie du sie hast/hattest in die Map-Klasse packst, die ganz Kollisionsprüfung dann auch dort durchführst (was ohne Parameter möglich ist) und die Position des Spielers aktualisierst
die Methode muss dazu folgende Dinge wissen:
welcher Spieler?
aktuelle Position?
welche Bewegungsrichtung?
welche Geschwindigkeit?
nicht alle diese Dinge müssen als Parameter übergeben werden
es ließe sich auch so lösen, dass nur noch der Spieler als Parameter übergeben werden muss


und als Randbemerkung noch:
die Position des Spielers in der Umgebung (Map) ist keine Eigenschaft des Spielers und keine Eigenschaft der Map, sondern eine Eigenschaft der Beziehung zwischen Spieler und Map (ein Spieler, der sich nicht in einer Umgebung befindet, benötigt keine Position und eine Map ohne Spieler keine Spielerpositionen)
an der Stelle macht man sich das Leben meist einfacher, indem man dem Spieler die Position gibt


spieler zeichnen:
das sollte gänzlich unabhängig von der Bewegung passieren
schreibe dir zusätzlich eine Draw()-Methode, die das Zeichnen erledigt

Zitat von »Flutschi«

Hmm ich geb das Fenster halt weiter damit ich es Updaten kann, ich versteh sonst nicht wie ich den Spieler sonst in das Fenster drawen soll? Ich definier ja das Sprite erst in der Spieler klasse, somit ist es in der eigentlichen Run() funktion nicht bekannt, soll ich jetzt das Sprite von der Spielerklasse zurück an die Run klasse geben oder das Fenster von der Run in die Spielerklasse?

Die Methode "Run" sagt ihrem Namen nach nicht, dass sie ein Sprite irgendwohin zeichnet. Sonst hieße sie ja RunAndRender. Wie ich schon sagte, spalte das Design auf. Die Methode macht zu viel auf einmal. Das sollte sie nicht.

@Sacaldur: Sehe ich leicht anders. Ich finde, dass die Position eines Spielers durchaus eine seiner Eigenschaften ist, auch wenn nicht so konkret wie Mütze oder Hose. Sie ist abstrakter. Natürlich kann man sie in eine externe Relation umwandeln, die zwischen Map und Spieler besteht. Wäre ebenso logisch, für mich aber ein unüblicher Ansatz. Das Problem, was ich vor allem dabei sehen würde ist: Gehört z.B. Geschwindigkeit und Richtung auch dazu? Oder gehört dazu auch die aktuelle Animation, also welcher Fuß oder welcher Arm gerade wo ist? Würdest Du die komplette Animation und Bewegung der Figur auslagern? Wenn ja, was bleibt vom Konstrukt "Spieler" eigentlich noch übrig außer einem reinen Daten-Container?

Zitat von »Flutschi«

Soll ich jetzt eifach mal den Spieler fortbewegen, und nachdem er gewandert ist, die Karte fragen ob ich da überhaupt stehen darf? und wenn nicht dann wieder zurücksetzen?

Mach doch einfach mal folgenden Versuch. Du stellst dich in ca. 3m Entfernung vor einer Wand hin, dann nimmst du Anlauf und rennst los.

3m ... 2m ... 1m ... 0m ... BUMM

Wer ruft Aua? Die Wand oder du?