Problem mit zirkulärer Abhängigkeit

  • C++

Es gibt 3 Antworten in diesem Thema. Der letzte Beitrag () ist von ichduersie.

    Problem mit zirkulärer Abhängigkeit

    Servus,

    ich hab eine Architekturfrage in C++. Ich arbeite an einer Game Engine und möchte die State Machine überarbeiten. Bisher ist es so, dass neue Szenen manuell in eine Enum eingetragen werden müssen und beim Szenenwechsel einfach die changeScene(...) aus dem Manager in der Szene durch ein gebindetes std::function-Objekt aufgerufen wird. Ich möchte das ganze dahingehend umgestalten, dass zum einen die Szenen dynamisch hinzugefügt werden können und dass der Szenenwechsel über ihren Namen / Tag funktioniert. Weitergehend möchte ich gerne ohne das Binding arbeiten, ich persönlich finde das echt nicht schön.

    Ich stoße dabei aber auf das Problem, dass ich ohne Binding eine zirkuläre Abhängigkeit aufbaue (siehe vereinfachtes Klassendiagramm). Im Internet werde ich leider nicht besonders schlau, denn die Wege, diese zirkuläre Abhängigkeit zu umgehen (bspw. mit einer anonymen Klasse und friend-Funktionen) finde ich genauso unschön. Da kann ich dann ja auch direkt funktional programmieren. Und andere Beispiele für eine State Machine ohne Enum finde ich nicht, aber es muss ja einen Weg geben.

    Von daher hoffe ich, dass mir hier jemand helfen kann.
    Liebe Grüße

    Spoiler anzeigen

    C-Quellcode

    1. class Scene {
    2. public:
    3. virtual void load() = 0;
    4. virtual void unload() = 0;
    5. virtual void render(...) = 0;
    6. virtual void update(const std::chrono::high_resolution_clock::duration& deltaTime) = 0;
    8. public:
    9. enum class SceneType : int8_t {
    10. INVALID = -1,
    11. MAIN = 0,
    12. GAME = 1,
    13. OPTIONS = 2,
    14. CREDITS = 3
    15. };
    17. std::function<void(SceneType)> changeScene;
    18. std::function<void()> requestQuit;
    19. };

    C-Quellcode

    1. void Manager::updateScene(const std::chrono::high_resolution_clock::duration& deltaTime) {
    2. switch (currentScene) {
    3. case scenes::Scene::SceneType::MAIN: mainScene.update(deltaTime); break;
    4. case scenes::Scene::SceneType::GAME: gameScene.update(deltaTime); break;
    5. case scenes::Scene::SceneType::OPTIONS: optionsScene.update(deltaTime); break;
    6. case scenes::Scene::SceneType::CREDITS: creditsScene.update(deltaTime); break;
    7. }
    8. }
    10. void Manager::changeScene(scenes::Scene::SceneType scene) {
    11. // Game state machine
    13. // We consciously don't check to see if a scene is trying to reload itself.
    14. // This way a scene can reset itself.
    16. // Unload the current scene
    17. if (currentScene != scenes::Scene::SceneType::INVALID) {
    18. switch (currentScene) {
    19. case scenes::Scene::SceneType::MAIN: mainScene.unload(); break;
    20. case scenes::Scene::SceneType::GAME: gameScene.unload(); break;
    21. case scenes::Scene::SceneType::OPTIONS: optionsScene.unload(); break;
    22. case scenes::Scene::SceneType::CREDITS: creditsScene.unload(); break;
    23. }
    24. }
    26. // Load the new scene
    27. switch (scene) {
    28. case scenes::Scene::SceneType::MAIN:
    29. mainScene.changeScene = std::bind(&Manager::changeScene, this, std::placeholders::_1);
    30. mainScene.requestQuit = std::bind(&Game::exit, game);
    31. mainScene.load();
    32. break;
    34. case scenes::Scene::SceneType::GAME:
    35. gameScene.changeScene = std::bind(&Manager::changeScene, this, std::placeholders::_1);
    36. gameScene.requestQuit = std::bind(&Game::exit, game);
    37. gameScene.load();
    38. break;
    40. case scenes::Scene::SceneType::OPTIONS:
    41. optionsScene.changeScene = std::bind(&Manager::changeScene, this, std::placeholders::_1);
    42. optionsScene.requestQuit = std::bind(&Game::exit, game);
    43. optionsScene.load();
    44. break;
    46. case scenes::Scene::SceneType::CREDITS:
    47. creditsScene.changeScene = std::bind(&Manager::changeScene, this, std::placeholders::_1);
    48. creditsScene.requestQuit = std::bind(&Game::exit, game);
    49. creditsScene.load();
    50. break;
    51. }
    53. currentScene = scene;
    54. }


    // Edit: Die eigentliche State Machine ergänzt. Ist schon echt hart widerlich gemacht <X
    Bilder

    • cd.png

      9,98 kB, 546×368, 87 mal angesehen

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von „ichduersie“ ()

    Ok, das ist mir jetzt echt unangenehm. Ist sitze no joke seit mehreren Wochen an diesem Problem und komme just nachdem ich hier geschrieben habe auf die Lösung :S
    Ist eigentlich echt easy, hab ein einfaches Demoprojekt angehängt, sodass das jeder nachvollziehen kann, wenn ersie möchte. Der Witz war einfach, die Scene für den Manager im Header anonym zu deklarieren und den Header der Scene erst im Source des Managers und nicht im Header reinzunehmen. Ich hoffe, dass solche Krämpfe mit den neuen Modulen nicht mehr auftreten.

    Natürlich werde ich in der Engine keine statischen Klassen sondern Singletons einsetzen. Ist mMn deutlich schöner.
    Dateien

    • Project2.zip

      (4,95 kB, 71 mal heruntergeladen, zuletzt: )

    Mir ist klar, dass du diesen Beitrag schon als gelöst markiert hast, allerdings habe ich da noch eine Frage bezüglich deines dynamischen Szenen Problems (falls du dies noch nicht gelöst hast):
    Wieso verwendest du dafür nicht einfach einen Vector-container der auf dynamisch allozierte Szenen Objekte zurückgreift?
    Oder villeicht auch eine unordered_map wenn du auf die Szenen via ID zugreifen möchtest!
    So kannst du dir dann auch die ganzen switch-statements sparen und mit einem reinen null check überprüfen.

    ~ Elanda
    ----------------------------------------------------------------------------------------------------------------------

    Hier könnte meine Signatur stehen, aber die ist mir abfußen gekommen.

    ----------------------------------------------------------------------------------------------------------------------

    Genau das hab ich vor. Ich will das switch schmeißen und die Szenen mit ihrer ID in eine map packen. Ich kann das switch zur Laufzeit schließlich nicht verändern, aber es sollen zur Laufzeit Szenen dynamisch hinzugefügt werden können.

    Aber trotzdem danke für deine Anmerkung :)

Facebook
Telefonspende