added loopTimer and eventManager to mediator and removed the singletons

13 files changed, 54 insertions, 240 deletions

diff --git a/src/crepe/api/CMakeLists.txt b/src/crepe/api/CMakeLists.txt
index 7da9dca..60d9dc5 100644
--- a/src/crepe/api/CMakeLists.txt
+++ b/src/crepe/api/CMakeLists.txt
@@ -13,10 +13,7 @@ target_sources(crepe PUBLIC
 	Metadata.cpp
 	Camera.cpp
 	Animator.cpp
-	IKeyListener.cpp
-	IMouseListener.cpp
 	LoopManager.cpp
-	LoopTimer.cpp
 	Asset.cpp
 	EventHandler.cpp
 	Script.cpp
@@ -45,9 +42,6 @@ target_sources(crepe PUBLIC FILE_SET HEADERS FILES
 	EventHandler.h
 	EventHandler.hpp
 	Event.h
-	IKeyListener.h
-	IMouseListener.h
 	LoopManager.h
-	LoopTimer.h
 	Asset.h
 )
diff --git a/src/crepe/api/IKeyListener.cpp b/src/crepe/api/IKeyListener.cpp
deleted file mode 100644
index 8642655..0000000
--- a/src/crepe/api/IKeyListener.cpp
+++ /dev/null
@@ -1,19 +0,0 @@
-#include "IKeyListener.h"
-
-using namespace crepe;
-
-// Constructor with specified channel
-IKeyListener::IKeyListener(event_channel_t channel)
-	: event_manager(EventManager::get_instance()) {
-	this->press_id = event_manager.subscribe<KeyPressEvent>(
-		[this](const KeyPressEvent & event) { return this->on_key_pressed(event); }, channel);
-	this->release_id = event_manager.subscribe<KeyReleaseEvent>(
-		[this](const KeyReleaseEvent & event) { return this->on_key_released(event); },
-		channel);
-}
-
-// Destructor, unsubscribe events
-IKeyListener::~IKeyListener() {
-	event_manager.unsubscribe(this->press_id);
-	event_manager.unsubscribe(this->release_id);
-}
diff --git a/src/crepe/api/IKeyListener.h b/src/crepe/api/IKeyListener.h
deleted file mode 100644
index 6ded107..0000000
--- a/src/crepe/api/IKeyListener.h
+++ /dev/null
@@ -1,50 +0,0 @@
-#pragma once
-
-#include "../manager/EventManager.h"
-
-#include "Event.h"
-#include "EventHandler.h"
-
-namespace crepe {
-
-/**
- * \class IKeyListener
- * \brief Interface for keyboard event handling in the application.
- */
-class IKeyListener {
-public:
-	/**
-     * \brief Constructs an IKeyListener with a specified channel.
-     * \param channel The channel ID for event handling.
-     */
-	IKeyListener(event_channel_t channel = EventManager::CHANNEL_ALL);
-	virtual ~IKeyListener();
-	IKeyListener(const IKeyListener &) = delete;
-	IKeyListener & operator=(const IKeyListener &) = delete;
-	IKeyListener & operator=(IKeyListener &&) = delete;
-	IKeyListener(IKeyListener &&) = delete;
-
-	/**
-     * \brief Pure virtual function to handle key press events.
-     * \param event The key press event to handle.
-     * \return True if the event was handled, false otherwise.
-     */
-	virtual bool on_key_pressed(const KeyPressEvent & event) = 0;
-
-	/**
-     * \brief Pure virtual function to handle key release events.
-     * \param event The key release event to handle.
-     * \return True if the event was handled, false otherwise.
-     */
-	virtual bool on_key_released(const KeyReleaseEvent & event) = 0;
-
-private:
-	//! Key press event id
-	subscription_t press_id = -1;
-	//! Key release event id
-	subscription_t release_id = -1;
-	//! EventManager reference
-	EventManager & event_manager;
-};
-
-} // namespace crepe
diff --git a/src/crepe/api/IMouseListener.cpp b/src/crepe/api/IMouseListener.cpp
deleted file mode 100644
index 989aeb3..0000000
--- a/src/crepe/api/IMouseListener.cpp
+++ /dev/null
@@ -1,29 +0,0 @@
-#include "IMouseListener.h"
-
-using namespace crepe;
-
-IMouseListener::IMouseListener(event_channel_t channel)
-	: event_manager(EventManager::get_instance()) {
-	this->click_id = event_manager.subscribe<MouseClickEvent>(
-		[this](const MouseClickEvent & event) { return this->on_mouse_clicked(event); },
-		channel);
-
-	this->press_id = event_manager.subscribe<MousePressEvent>(
-		[this](const MousePressEvent & event) { return this->on_mouse_pressed(event); },
-		channel);
-
-	this->release_id = event_manager.subscribe<MouseReleaseEvent>(
-		[this](const MouseReleaseEvent & event) { return this->on_mouse_released(event); },
-		channel);
-
-	this->move_id = event_manager.subscribe<MouseMoveEvent>(
-		[this](const MouseMoveEvent & event) { return this->on_mouse_moved(event); }, channel);
-}
-
-IMouseListener::~IMouseListener() {
-	// Unsubscribe event handlers
-	event_manager.unsubscribe(this->click_id);
-	event_manager.unsubscribe(this->press_id);
-	event_manager.unsubscribe(this->release_id);
-	event_manager.unsubscribe(this->move_id);
-}
diff --git a/src/crepe/api/IMouseListener.h b/src/crepe/api/IMouseListener.h
deleted file mode 100644
index 9e4fdf7..0000000
--- a/src/crepe/api/IMouseListener.h
+++ /dev/null
@@ -1,73 +0,0 @@
-#pragma once
-
-#include "../manager/EventManager.h"
-
-#include "Event.h"
-#include "EventHandler.h"
-
-namespace crepe {
-
-/**
- * \class IMouseListener
- * \brief Interface for mouse event handling in the application.
- */
-class IMouseListener {
-public:
-	/**
-     * \brief Constructs an IMouseListener with a specified channel.
-     * \param channel The channel ID for event handling.
-     */
-	IMouseListener(event_channel_t channel = EventManager::CHANNEL_ALL);
-	virtual ~IMouseListener();
-	IMouseListener & operator=(const IMouseListener &) = delete;
-	IMouseListener(const IMouseListener &) = delete;
-	IMouseListener & operator=(const IMouseListener &&) = delete;
-	IMouseListener(IMouseListener &&) = delete;
-
-	/**
-     * \brief Move assignment operator (deleted).
-     */
-	IMouseListener & operator=(IMouseListener &&) = delete;
-
-	/**
-     * \brief Handles a mouse click event.
-     * \param event The mouse click event to handle.
-     * \return True if the event was handled, false otherwise.
-     */
-	virtual bool on_mouse_clicked(const MouseClickEvent & event) = 0;
-
-	/**
-     * \brief Handles a mouse press event.
-     * \param event The mouse press event to handle.
-     * \return True if the event was handled, false otherwise.
-     */
-	virtual bool on_mouse_pressed(const MousePressEvent & event) = 0;
-
-	/**
-     * \brief Handles a mouse release event.
-     * \param event The mouse release event to handle.
-     * \return True if the event was handled, false otherwise.
-     */
-	virtual bool on_mouse_released(const MouseReleaseEvent & event) = 0;
-
-	/**
-     * \brief Handles a mouse move event.
-     * \param event The mouse move event to handle.
-     * \return True if the event was handled, false otherwise.
-     */
-	virtual bool on_mouse_moved(const MouseMoveEvent & event) = 0;
-
-private:
-	//! Mouse click event id
-	subscription_t click_id = -1;
-	//! Mouse press event id
-	subscription_t press_id = -1;
-	//! Mouse release event id
-	subscription_t release_id = -1;
-	//! Mouse move event id
-	subscription_t move_id = -1;
-	//! EventManager reference
-	EventManager & event_manager;
-};
-
-} //namespace crepe
diff --git a/src/crepe/api/LoopManager.cpp b/src/crepe/api/LoopManager.cpp
index 454afe8..69cbfaf 100644
--- a/src/crepe/api/LoopManager.cpp
+++ b/src/crepe/api/LoopManager.cpp
@@ -1,3 +1,4 @@
+#include <iostream>
 #include "../facade/SDLContext.h"
 
 #include "../manager/EventManager.h"
@@ -20,10 +21,8 @@ LoopManager::LoopManager() {
 	this->load_system<PhysicsSystem>();
 	this->load_system<RenderSystem>();
 	this->load_system<ScriptSystem>();
-	EventManager::get_instance().subscribe<ShutDownEvent>(
+	this->event_manager.subscribe<ShutDownEvent>(
 		[this](const ShutDownEvent & event) { return this->on_shutdown(event); });
-	this->loop_timer = make_unique<LoopTimer>();
-	this->mediator.loop_timer = *loop_timer;
 }
 
 void LoopManager::process_input() { 
@@ -38,27 +37,28 @@ void LoopManager::start() {
 void LoopManager::fixed_update() {}
 
 void LoopManager::loop() {
-	this->loop_timer->start();
+	
 
 	while (game_running) {
-		this->loop_timer->update();
+		this->loop_timer.update();
 
-		while (this->loop_timer->get_lag() >= this->loop_timer->get_fixed_delta_time()) {
+		while (this->loop_timer.get_lag() >= this->loop_timer.get_fixed_delta_time()) {
 			this->process_input();
+			event_manager.dispatch_events();
 			this->fixed_update();
-			this->loop_timer->advance_fixed_update();
+			this->loop_timer.advance_fixed_update();
 		}
 
 		this->update();
 		this->render();
-		this->loop_timer->enforce_frame_rate();
+		this->loop_timer.enforce_frame_rate();
 	}
 }
 
 void LoopManager::setup() {
 
 	this->game_running = true;
-	this->loop_timer->start();
+	this->loop_timer.start();
 }
 
 void LoopManager::render() {
diff --git a/src/crepe/api/LoopManager.h b/src/crepe/api/LoopManager.h
index 2161dff..00f5409 100644
--- a/src/crepe/api/LoopManager.h
+++ b/src/crepe/api/LoopManager.h
@@ -5,10 +5,11 @@
 #include "../facade/SDLContext.h"
 #include "../manager/ComponentManager.h"
 #include "../manager/SceneManager.h"
+#include "../manager/EventManager.h"
+#include "../manager/LoopTimerManager.h"
 #include "../system/System.h"
 
 #include "api/Event.h"
-#include "api/LoopTimer.h"
 
 namespace crepe {
 /**
@@ -96,8 +97,10 @@ private:
 
 	//! SDL context \todo no more singletons!
 	SDLContext & sdl_context = SDLContext::get_instance();
-	//! loop timer instance
-	std::unique_ptr<LoopTimer> loop_timer;
+	//! LoopTimer instance
+	LoopTimerManager loop_timer{mediator};
+	//! EventManager instance
+	EventManager event_manager{mediator};
 
 private:
 	/**
diff --git a/src/crepe/manager/CMakeLists.txt b/src/crepe/manager/CMakeLists.txt
index 517b8a2..29d6df0 100644
--- a/src/crepe/manager/CMakeLists.txt
+++ b/src/crepe/manager/CMakeLists.txt
@@ -4,6 +4,7 @@ target_sources(crepe PUBLIC
 	Manager.cpp
 	SaveManager.cpp
 	SceneManager.cpp
+	LoopTimerManager.cpp
 )
 
 target_sources(crepe PUBLIC FILE_SET HEADERS FILES
@@ -16,5 +17,6 @@ target_sources(crepe PUBLIC FILE_SET HEADERS FILES
 	SaveManager.h
 	SceneManager.h
 	SceneManager.hpp
+	LoopTimerManager.h
 )
 
diff --git a/src/crepe/manager/EventManager.cpp b/src/crepe/manager/EventManager.cpp
index 20f0dd3..9b0fa95 100644
--- a/src/crepe/manager/EventManager.cpp
+++ b/src/crepe/manager/EventManager.cpp
@@ -3,11 +3,9 @@
 using namespace crepe;
 using namespace std;
 
-EventManager & EventManager::get_instance() {
-	static EventManager instance;
-	return instance;
+EventManager::EventManager(Mediator & mediator) : Manager(mediator){
+	this->mediator.event_manager = *this;
 }
-
 void EventManager::dispatch_events() {
 	for (auto & event : this->events_queue) {
 		this->handle_event(event.type, event.channel, *event.event.get());
diff --git a/src/crepe/manager/EventManager.h b/src/crepe/manager/EventManager.h
index d634f54..5f8b107 100644
--- a/src/crepe/manager/EventManager.h
+++ b/src/crepe/manager/EventManager.h
@@ -8,6 +8,8 @@
 #include "../api/Event.h"
 #include "../api/EventHandler.h"
 
+#include "Manager.h"
+
 namespace crepe {
 
 //! Event listener unique ID
@@ -22,27 +24,16 @@ typedef size_t subscription_t;
 typedef size_t event_channel_t;
 
 /**
- * \class EventManager
  * \brief Manages event subscriptions, triggers, and queues, enabling decoupled event handling.
  * 
  * The `EventManager` acts as a centralized event system. It allows for registering callbacks
  * for specific event types, triggering events synchronously, queueing events for later
  * processing, and managing subscriptions via unique identifiers.
  */
-class EventManager {
+class EventManager : public Manager {
 public:
 	static constexpr const event_channel_t CHANNEL_ALL = -1;
-
-	/**
-	 * \brief Get the singleton instance of the EventManager.
-	 * 
-	 * This method returns the unique instance of the EventManager, creating it if it
-	 * doesn't already exist. Ensures only one instance is active in the program.
-	 * 
-	 * \return Reference to the singleton instance of the EventManager.
-	 */
-	static EventManager & get_instance();
-
+	EventManager(Mediator & mediator);
 	/**
 	 * \brief Subscribe to a specific event type.
 	 * 
@@ -107,12 +98,7 @@ public:
 	void clear();
 
 private:
-	/**
-	 * \brief Default constructor for the EventManager.
-	 * 
-	 * Constructor is private to enforce the singleton pattern.
-	 */
-	EventManager() = default;
+
 
 	/**
 	 * \struct QueueEntry
diff --git a/src/crepe/api/LoopTimer.cpp b/src/crepe/manager/LoopTimerManager.cpp
index 8fb7ce8..8156c6d 100644
--- a/src/crepe/api/LoopTimer.cpp
+++ b/src/crepe/manager/LoopTimerManager.cpp
@@ -4,13 +4,16 @@
 #include "../facade/SDLContext.h"
 #include "../util/Log.h"
 
-#include "LoopTimer.h"
+#include "LoopTimerManager.h"
 
 using namespace crepe;
 
-LoopTimer::LoopTimer() { dbg_trace(); }
+LoopTimerManager::LoopTimerManager(Mediator & mediator) : Manager(mediator) { 
+	this->mediator.loop_timer = *this;
+	dbg_trace(); 
+	}
 
-void LoopTimer::start() {
+void LoopTimerManager::start() {
 	this->last_frame_time = std::chrono::steady_clock::now();
 
 	this->elapsed_time = std::chrono::milliseconds(0);
@@ -20,7 +23,7 @@ void LoopTimer::start() {
 	this->delta_time = std::chrono::milliseconds(0);
 }
 
-void LoopTimer::update() {
+void LoopTimerManager::update() {
 	auto current_frame_time = std::chrono::steady_clock::now();
 	// Convert to duration in seconds for delta time
 	this->delta_time = std::chrono::duration_cast<std::chrono::duration<double>>(
@@ -31,31 +34,30 @@ void LoopTimer::update() {
 	}
 	this->actual_fps = 1.0 / this->delta_time.count();
 
-	this->delta_time *= this->game_scale;
 	this->elapsed_time += this->delta_time;
 	this->last_frame_time = current_frame_time;
 }
 
-double LoopTimer::get_delta_time() const { return this->delta_time.count(); }
+double LoopTimerManager::get_delta_time() const { return this->delta_time.count() * this->game_scale; }
 
-double LoopTimer::get_current_time() const { return this->elapsed_time.count(); }
+double LoopTimerManager::get_current_time() const { return this->elapsed_time.count(); }
 
-void LoopTimer::advance_fixed_update() { this->elapsed_fixed_time += this->fixed_delta_time; }
+void LoopTimerManager::advance_fixed_update() { this->elapsed_fixed_time += this->fixed_delta_time; }
 
-double LoopTimer::get_fixed_delta_time() const { return this->fixed_delta_time.count(); }
+double LoopTimerManager::get_fixed_delta_time() const { return this->fixed_delta_time.count(); }
 
-void LoopTimer::set_target_fps(int fps) {
+void LoopTimerManager::set_target_fps(int fps) {
 	this->target_fps = fps;
 	// target time per frame in seconds
 	this->frame_target_time = std::chrono::duration<double>(1.0) / this->target_fps;
 }
 
-int LoopTimer::get_fps() const { return this->actual_fps; }
+int LoopTimerManager::get_fps() const { return this->actual_fps; }
 
-void LoopTimer::set_game_scale(double value) { this->game_scale = value; }
+void LoopTimerManager::set_time_scale(double value) { this->game_scale = value; }
 
-double LoopTimer::get_game_scale() const { return this->game_scale; }
-void LoopTimer::enforce_frame_rate() {
+double LoopTimerManager::get_time_scale() const { return this->game_scale; }
+void LoopTimerManager::enforce_frame_rate() {
 	auto current_frame_time = std::chrono::steady_clock::now();
 	auto frame_duration = current_frame_time - this->last_frame_time;
 
@@ -70,6 +72,6 @@ void LoopTimer::enforce_frame_rate() {
 	}
 }
 
-double LoopTimer::get_lag() const {
+double LoopTimerManager::get_lag() const {
 	return (this->elapsed_time - this->elapsed_fixed_time).count();
 }
diff --git a/src/crepe/api/LoopTimer.h b/src/crepe/manager/LoopTimerManager.h
index c4294d7..dba0f66 100644
--- a/src/crepe/api/LoopTimer.h
+++ b/src/crepe/manager/LoopTimerManager.h
@@ -1,12 +1,12 @@
 #pragma once
 
 #include <chrono>
-
+#include "../manager/Manager.h"
 namespace crepe {
 
-class LoopTimer {
+class LoopTimerManager : public Manager {
 public:
-	LoopTimer();
+	LoopTimerManager(Mediator & mediator);
 	/**
 	 * \brief Get the current delta time for the current frame.
 	 *
@@ -39,19 +39,19 @@ public:
 	int get_fps() const;
 
 	/**
-	 * \brief Get the current game scale.
+	 * \brief Get the current time scale.
 	 *
-	 * \return The current game scale, where 0 = paused, 1 = normal speed, and values > 1 speed
+	 * \return The current time scale, where 0 = paused, 1 = normal speed, and values > 1 speed
 	 * up the game.
 	 */
-	double get_game_scale() const;
+	double get_time_scale() const;
 
 	/**
-	 * \brief Set the game scale.
+	 * \brief Set the time scale.
 	 *
-	 * \param game_scale The desired game scale (0 = pause, 1 = normal speed, > 1 = speed up).
+	 * \param game_scale The desired time scale (0 = pause, 1 = normal speed, > 1 = speed up).
 	 */
-	void set_game_scale(double game_scale);
+	void set_time_scale(double game_scale);
 
 private:
 	friend class LoopManager;
diff --git a/src/crepe/manager/Mediator.h b/src/crepe/manager/Mediator.h
index cd96614..c72af8e 100644
--- a/src/crepe/manager/Mediator.h
+++ b/src/crepe/manager/Mediator.h
@@ -3,14 +3,14 @@
 #include "../util/OptionalRef.h"
 
 // TODO: remove these singletons:
-#include "EventManager.h"
 #include "SaveManager.h"
 
 namespace crepe {
 
 class ComponentManager;
 class SceneManager;
-class LoopTimer;
+class LoopTimerManager;
+class EventManager;
 /**
  * Struct to pass references to classes that would otherwise need to be singletons down to
  * other classes within the engine hierarchy. Made to prevent constant changes to subclasses to
@@ -27,8 +27,8 @@ struct Mediator {
 	OptionalRef<ComponentManager> component_manager;
 	OptionalRef<SceneManager> scene_manager;
 	OptionalRef<SaveManager> save_manager = SaveManager::get_instance();
-	OptionalRef<EventManager> event_manager = EventManager::get_instance();
-	OptionalRef<LoopTimer> loop_timer;
+	OptionalRef<EventManager> event_manager;
+	OptionalRef<LoopTimerManager> loop_timer;
 };
 
 } // namespace crepe