refactor: update enemy AI system to accept player entity directly

Author: AhmedSobhy01

src/game/systems/enemy-ai.hpp

@@ -12,7 +12,6 @@
 
 #include "../components/enemy.hpp"
 #include "../components/health.hpp"
-#include "../components/player.hpp"
 
 namespace gameplay {
 
@@ -44,18 +43,8 @@ namespace gameplay {
         }
 
     public:
-        void update(our::World* world, float deltaTime) {
-            if (!world || deltaTime <= 0.0f) return;
-
-            // get player position
-            our::Entity* playerEntity = nullptr;
-            for (our::Entity* entity : world->getEntities()) {
-                if (entity->getComponent<PlayerComponent>()) {
-                    playerEntity = entity;
-                    break;
-                }
-            }
-            if (!playerEntity) return;
+        void update(our::World* world, our::Entity* playerEntity, float deltaTime) {
+            if (!world || !playerEntity || deltaTime <= 0.0f) return;
 
             glm::vec3 playerPos = glm::vec3(playerEntity->getLocalToWorldMatrix() * glm::vec4(0, 0, 0, 1));
             float t = static_cast<float>(glfwGetTime());

src/states/play-state.hpp

@@ -1,6 +1,7 @@
 #pragma once
 
 #include <application.hpp>
+#include <components/player.hpp>
 #include <ecs/world.hpp>
 #include <systems/enemy-ai.hpp>
 #include <systems/enemy-spawner.hpp>
@@ -14,6 +15,7 @@ class Playstate : public our::State {
     our::ForwardRenderer renderer;
     our::FreeCameraControllerSystem cameraController;
     our::MovementSystem movementSystem;
+    our::Entity* playerEntity = nullptr;
     gameplay::EnemyAISystem enemyAI;
     gameplay::EnemySpawner enemySpawner;
 
@@ -50,6 +52,15 @@ class Playstate : public our::State {
         if (config.contains("world")) {
             world.deserialize(config["world"]);
         }
+
+        // Store a pointer to the player entity
+        for (our::Entity* entity : world.getEntities()) {
+            if (entity->getComponent<gameplay::PlayerComponent>()) {
+                playerEntity = entity;
+                break;
+            }
+        }
+
         // We initialize the camera controller system since it needs a pointer to the app
         cameraController.enter(getApp());
         // Then we initialize the renderer
@@ -64,7 +75,7 @@ class Playstate : public our::State {
         // Here, we just run a bunch of systems to control the world logic
         movementSystem.update(&world, (float)deltaTime);
         cameraController.update(&world, (float)deltaTime);
-        enemyAI.update(&world, (float)deltaTime);
+        enemyAI.update(&world, playerEntity, (float)deltaTime);
         enemySpawner.update(&world, (float)deltaTime);
         // And finally we use the renderer system to draw the scene
         renderer.render(&world);
@@ -83,6 +94,7 @@ class Playstate : public our::State {
         renderer.destroy();
         // On exit, we call exit for the camera controller system to make sure that the mouse is unlocked
         cameraController.exit();
+        playerEntity = nullptr;
         // Clear the world
         world.clear();
         // and we delete all the loaded assets to free memory on the RAM and the VRAM