spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Zitat von »cojo2015«

Das "std::make_..." steht unter den #include`s, also wird das doch nicht jedes mal erstellt

Nee, in dem Fall nicht! Ich dachte du erstellst den in irgendeiner Methode.

Gibt es in Playstate ebenfalls eine Player Instanz?

Hab ich mir schon fast gedacht.

Du solltest dringend an deinem Design arbeiten!

Zitat von »cojo2015«

Ja. Auf die habe ich in einem Beitrag schon geschrieben, dass ich auf diese zugegriffen habe, aber es gab keine Änderung.

Kannst du das Projekt mal irgendwo hochladen, sonst geht das wieder Seitenweise soweiter

Also...

In PlayState wird der Player nicht erstellt (Ist der in Asteroid.cpp gelandet wegen debuggen?), husch husch zurück damit in PlayState!

Dann empfehle ich dir, den Konstruktor einer Klasse zu nutzen und auf Init Methoden zu verzichten.
Da könntest du dann nämlich dem Asteroiden mitgeben um welchen x Wert er spawnen soll.


Bei deinem Aktuellem Design musst du den X Wert des Players entweder beim erstellen des Asteroiden speichern (in der Asteroiden Klasse) was aus meiner Sicht blödsinn ist (bei anderem/besserem Design).
Oder der InitMethode und dann der loadAsteroid den X Wert des Players mitgeben.



Noch paar tips zu Design.

• Konstruktor nutzen statt Init-Methode
  
• Klassenmember private halten (Datenkapselung)
  
• ChangeState Methode dringend überarbeiten, du erstellts immer einen ganz neuen State anstatt wie die Methode vermuten lassen würde ihn zu wechseln.
  
• gibt noch andere Sachen aber bin zu Faul das jetzt alles nieder zu schreiben
  

Zitat von »cojo2015«

Wenn es für dich kein Problem ist, kannst du mir noch ein paar Tipps zum Codedesign geben?

Na wenn du schon so nett Fragst, sicherlich.

• Du hast zwar Konstanten wie z.B. const float m_fMaxX = 935.f; nutzt sie aber gar nicht (macht den Code besser lesbar)
  
• Stichwort Const Correctness
  
• Wenn du in Klasse A, Klasse B hast und in Klasse B, Klasse C hast ist es in den meisten Fällen wohl ein Zeichen für schlechts Design wenn Klasse C wissen muss das es Klasse A gibt (gilt auch für Klasse B, die brauch auch nichts wissen über A).
  

Gibt bestimmt noch paar Sachen die man beachten könnte, aber mehr ist mir jetzt nicht aufgefallen bei durchschauen von deinem Code.