refactor: singleton pattern

modules/engine/include/engine/core/application.hpp

@@ -1,16 +1,13 @@
 #pragma once
 
 #include <engine/base/base.hpp>
-#include <engine/debug/instrumentor.hpp>
 #include <engine/input/input.hpp>
 #include <engine/layer/layer_stack.hpp>
-#include <engine/utils/resource_manager.hpp>
 
 namespace Light {
 
 class Window;
 class Event;
-class Instrumentor;
 
 class Application
 {
@@ -35,15 +32,7 @@ private:
 
 	void log_debug_data();
 
-	static Application *s_context;
-
-	Scope<Instrumentor> m_instrumentor;
-
 	Scope<LayerStack> m_layer_stack;
-
-	Scope<Input> m_input;
-
-	Scope<ResourceManager> m_resource_manager;
 };
 
 extern Light::Scope<Application> create_application();

modules/engine/include/engine/debug/instrumentor.hpp

@@ -16,30 +16,32 @@ struct ScopeProfileResult
 class Instrumentor
 {
 public:
-	static Scope<Instrumentor> create();
+	static auto instance() -> Instrumentor &
+	{
+		static auto instance = Instrumentor {};
+		return instance;
+	}
 
 	static void begin_session(const std::string &outputPath)
 	{
-		s_context->begin_session_impl(outputPath);
+		instance().begin_session_impl(outputPath);
 	}
 	static void end_session()
 	{
-		s_context->end_session_impl();
+		instance().end_session_impl();
 	}
 
 	static void submit_scope_profile(const ScopeProfileResult &profileResult)
 	{
-		s_context->submit_scope_profile_impl(profileResult);
+		instance().submit_scope_profile_impl(profileResult);
 	}
 
 private:
-	static Instrumentor *s_context;
-
 	std::ofstream m_output_file_stream;
 
 	unsigned int m_current_session_count { 0u };
 
-	Instrumentor();
+	Instrumentor() = default;
 
 	void begin_session_impl(const std::string &outputPath);
 

modules/engine/include/engine/input/input.hpp

@@ -11,29 +11,35 @@ class Event;
 class Input
 {
 public:
-	static auto create() -> Scope<Input>;
+	static auto instance() -> Input &
+	{
+		static auto instance = Input {};
+		return instance;
+	}
 
 	static void receive_user_interface_events(bool receive, bool toggle = false)
 	{
-		s_context->receive_user_interface_events_impl(receive, toggle);
+		instance().receive_user_interface_events_impl(receive, toggle);
 	}
+
 	static void receive_game_events(bool receive, bool toggle = false)
 	{
-		s_context->receieve_game_events_impl(receive, toggle);
+		instance().receieve_game_events_impl(receive, toggle);
 	}
 
 	static auto get_keyboard_key(int code) -> bool
 	{
-		return s_context->m_keyboad_keys[code];
+		return instance().m_keyboad_keys[code];
 	}
+
 	static auto get_mouse_button(int code) -> bool
 	{
-		return s_context->m_mouse_buttons[code];
+		return instance().m_mouse_buttons[code];
 	}
 
 	static auto get_mouse_position(int /*code*/) -> const glm::vec2 &
 	{
-		return s_context->m_mouse_position;
+		return instance().m_mouse_position;
 	}
 
 	void on_event(const Event &inputEvent);
@@ -49,7 +55,13 @@ public:
 	}
 
 private:
-	static Input *s_context;
+	Input();
+
+	void receive_user_interface_events_impl(bool receive, bool toggle = false);
+
+	void receieve_game_events_impl(bool receive, bool toggle = false);
+
+	void restart_input_state();
 
 	std::array<bool, 348> m_keyboad_keys {};
 
@@ -64,14 +76,6 @@ private:
 	bool m_user_interface_events { true };
 
 	bool m_game_events { true };
-
-	Input();
-
-	void receive_user_interface_events_impl(bool receive, bool toggle = false);
-
-	void receieve_game_events_impl(bool receive, bool toggle = false);
-
-	void restart_input_state();
 };
 
 } // namespace Light

modules/engine/include/engine/layer/layer_stack.hpp

@@ -10,7 +10,11 @@ class Event;
 class LayerStack /* singleton */
 {
 public:
-	static auto create() -> Scope<LayerStack>;
+	static auto instance() -> LayerStack &
+	{
+		static auto instance = LayerStack {};
+		return instance;
+	}
 
 	~LayerStack();
 
@@ -18,17 +22,17 @@ public:
 	template<typename t, typename... Args>
 	static void emplace_layer(Args &&...args)
 	{
-		s_context->attach_layer_impl(new t((args)...));
+		instance().attach_layer_impl(new t((args)...));
 	}
 
 	static void attach_layer(Layer *layer)
 	{
-		s_context->attach_layer_impl(layer);
+		instance().attach_layer_impl(layer);
 	}
 
 	static void detach_layer(Layer *layer)
 	{
-		s_context->detach_layer_impl(layer);
+		instance().detach_layer_impl(layer);
 	}
 
 	auto is_empty() -> bool
@@ -57,15 +61,13 @@ public:
 	}
 
 private:
-	static LayerStack *s_context;
-
 	std::vector<Layer *> m_layers;
 
 	std::vector<Layer *>::iterator m_begin;
 
 	std::vector<Layer *>::iterator m_end;
 
-	LayerStack();
+	LayerStack() = default;
 
 	void attach_layer_impl(Layer *layer);
 

modules/engine/include/engine/utils/resource_manager.hpp

@@ -11,7 +11,11 @@ class SharedContext;
 class ResourceManager
 {
 public:
-	static auto create() -> Scope<ResourceManager>;
+	static auto instance() -> ResourceManager &
+	{
+		static auto instance = ResourceManager {};
+		return instance;
+	}
 
 	static void load_shader(
 	    const std::string &name,
@@ -19,7 +23,7 @@ public:
 	    const std::string &pixelPath
 	)
 	{
-		s_context->load_shader_impl(name, vertexPath, pixelPath);
+		instance().load_shader_impl(name, vertexPath, pixelPath);
 	}
 
 	static void load_texture(
@@ -28,32 +32,26 @@ public:
 	    unsigned int desiredComponents = 4u
 	)
 	{
-		s_context->load_texture_impl(name, path, desiredComponents);
+		instance().load_texture_impl(name, path, desiredComponents);
 	}
 
 	static void release_texture(const std::string &name)
 	{
-		s_context->release_texture_impl(name);
+		instance().release_texture_impl(name);
 	}
 
 	static auto get_shader(const std::string &name) -> Ref<Shader>
 	{
-		return s_context->m_shaders[name];
+		return instance().m_shaders[name];
 	}
 
 	static auto get_texture(const std::string &name) -> Ref<Texture>
 	{
-		return s_context->m_textures[name];
+		return instance().m_textures[name];
 	}
 
 private:
-	static ResourceManager *s_context;
+	ResourceManager() = default;
-
-	std::unordered_map<std::string, Ref<Shader>> m_shaders;
-
-	std::unordered_map<std::string, Ref<Texture>> m_textures;
-
-	ResourceManager();
 
 	void load_shader_impl(
 	    const std::string &name,
@@ -68,6 +66,10 @@ private:
 	);
 
 	void release_texture_impl(const std::string &name);
+
+	std::unordered_map<std::string, Ref<Shader>> m_shaders;
+
+	std::unordered_map<std::string, Ref<Texture>> m_textures;
 };
 
 } // namespace Light

modules/engine/src/core/application.cpp

@@ -12,34 +12,23 @@
 
 namespace Light {
 
-Application *Application::s_context = nullptr;
+Application::Application(): m_window(nullptr)
-
-Application::Application()
-    : m_instrumentor(nullptr)
-    , m_layer_stack(nullptr)
-    , m_input(nullptr)
-    , m_window(nullptr)
 {
-	lt_assert(!s_context, "Repeated singleton construction");
+	static auto constructed = false;
-	s_context = this;
+	lt_assert(!constructed, "Application constructed twice");
+	constructed = true;
 
 	log_debug_data();
 
-	m_instrumentor = Instrumentor::create();
 	Light::Instrumentor::begin_session("Logs/ProfileResults_Startup.json");
 
-	m_layer_stack = LayerStack::create();
-	m_input = Input::create();
-
-	m_resource_manager = ResourceManager::create();
-
 	m_window = Window::create([this](auto &&PH1) { on_event(std::forward<decltype(PH1)>(PH1)); });
 }
 
 Application::~Application()
 {
 	log_trc("Application::~Application()");
-	Light::Instrumentor::end_session();
+	Instrumentor::end_session();
 }
 
 void Application::game_loop()
@@ -54,8 +43,8 @@ void Application::game_loop()
 	// reveal window
 	m_window->set_visibility(true);
 
-	Light::Instrumentor::end_session();
+	Instrumentor::end_session();
-	Light::Instrumentor::begin_session("Logs/ProfileResults_GameLoop.json");
+	Instrumentor::begin_session("Logs/ProfileResults_GameLoop.json");
 
 	/* game loop */
 	auto delta_timer = DeltaTimer {};
@@ -65,7 +54,8 @@ void Application::game_loop()
 			// update layers
 			lt_profile_scope("game_loop::update");
 
-			for (auto & it : *m_layer_stack) {
+			for (auto &it : *m_layer_stack)
+			{
 				it->on_update(delta_timer.get_delta_time());
 			}
 		}
@@ -75,7 +65,8 @@ void Application::game_loop()
 			lt_profile_scope("game_loop::Render");
 			m_window->get_graphics_context()->get_renderer()->begin_frame();
 
-			for (auto & it : *m_layer_stack) {
+			for (auto &it : *m_layer_stack)
+			{
 				it->on_render();
 			}
 
@@ -87,7 +78,8 @@ void Application::game_loop()
 			lt_profile_scope("game_loop::UserInterface");
 			m_window->get_graphics_context()->get_user_interface()->begin();
 
-			for (auto & it : *m_layer_stack) {
+			for (auto &it : *m_layer_stack)
+			{
 				it->on_user_interface_update();
 			}
 
@@ -104,13 +96,13 @@ void Application::game_loop()
 		delta_timer.update();
 	}
 
-	Light::Instrumentor::end_session();
+	Instrumentor::end_session();
-	Light::Instrumentor::begin_session("Logs/ProfileResults_Termination.json");
+	Instrumentor::begin_session("Logs/ProfileResults_Termination.json");
 }
 
 void Application::quit()
 {
-	s_context->m_window->close();
+	/** TODO: fix quitting procedure */
 }
 
 void Application::on_event(const Event &event)
@@ -120,7 +112,8 @@ void Application::on_event(const Event &event)
 	{
 		m_window->on_event(event);
 
-		if (event.get_event_type() == EventType::WindowResized) {
+		if (event.get_event_type() == EventType::WindowResized)
+		{
 			m_window->get_graphics_context()->get_renderer()->on_window_resize(
 			    (const WindowResizedEvent &)event
 			);
@@ -130,9 +123,9 @@ void Application::on_event(const Event &event)
 	// input
 	if (event.has_category(InputEventCategory))
 	{
-		m_input->on_event(event);
+		Input::instance().on_event(event);
 
-		if (!m_input->is_receiving_game_events())
+		if (!Input::instance().is_receiving_game_events())
 		{
 			return;
 		}
@@ -149,13 +142,10 @@ void Application::on_event(const Event &event)
 
 void Application::log_debug_data()
 {
-	// #todo: log more information
-	log_inf("________________________________________");
 	log_inf("Platform::");
 	log_inf("       Platform name: {}", constants::platform_name);
 	log_inf("       Platform identifier: {}", std::to_underlying(constants::platform));
 	log_inf("       CWD: {}", std::filesystem::current_path().generic_string());
-	log_inf("________________________________________");
 }
 
 } // namespace Light

modules/engine/src/debug/instrumentor.cpp

@@ -2,23 +2,6 @@
 
 namespace Light {
 
-Instrumentor *Instrumentor::s_context = nullptr;
-
-auto Instrumentor::create() -> Scope<Instrumentor>
-{
-	return make_scope<Instrumentor>(new Instrumentor);
-}
-
-Instrumentor::Instrumentor() 
-{
-	// #todo: maintenance
-	lt_assert(
-	    !s_context,
-	    "An instance of 'Instrumentor' already exists, do not construct this class!"
-	);
-	s_context = this;
-}
-
 void Instrumentor::begin_session_impl(const std::string &outputPath)
 {
 	std::filesystem::create_directory(outputPath.substr(0, outputPath.find_last_of('/') + 1));
@@ -29,7 +12,8 @@ void Instrumentor::begin_session_impl(const std::string &outputPath)
 
 void Instrumentor::end_session_impl()
 {
-	if (m_current_session_count == 0u) {
+	if (m_current_session_count == 0u)
+	{
 		log_wrn("0 profiling for the ended session");
 	}
 
@@ -42,9 +26,12 @@ void Instrumentor::end_session_impl()
 
 void Instrumentor::submit_scope_profile_impl(const ScopeProfileResult &profileResult)
 {
-	if (m_current_session_count++ == 0u) {
+	if (m_current_session_count++ == 0u)
+	{
 		m_output_file_stream << "{";
-	} else {
+	}
+	else
+	{
 		m_output_file_stream << ",{";
 	}
 

modules/engine/src/input/input.cpp

@@ -8,25 +8,9 @@
 
 namespace Light {
 
-Input *Input::s_context = nullptr;
+Input::Input(): m_mouse_position {}, m_mouse_delta {}
-
-auto Input::create() -> Scope<Input>
-{
-	return make_scope(new Input);
-}
-
-Input::Input()
-    : 
-     m_mouse_position {}
-    , m_mouse_delta {}
 
 {
-	lt_assert(
-	    !s_context,
-	    "Input::Input: an instance of 'Input' already exists, do not construct this class!"
-	);
-	s_context = this;
-
 	restart_input_state();
 }
 
@@ -40,7 +24,8 @@ void Input::receieve_game_events_impl(bool receive, bool toggle /*= false*/)
 	auto prev = m_game_events;
 	m_game_events = toggle ? !m_user_interface_events : receive;
 
-	if (m_game_events != prev) {
+	if (m_game_events != prev)
+	{
 		restart_input_state();
 	}
 }
@@ -71,7 +56,8 @@ void Input::on_event(const Event &inputEvent)
 			m_mouse_position = event.get_position();
 		}
 
-		if (m_user_interface_events) {
+		if (m_user_interface_events)
+		{
 			io.MousePos = ImVec2(event.get_x(), event.get_y());
 		}
 
@@ -81,11 +67,13 @@ void Input::on_event(const Event &inputEvent)
 	{
 		const auto &event = dynamic_cast<const ButtonPressedEvent &>(inputEvent);
 
-		if (m_game_events) {
+		if (m_game_events)
+		{
 			m_mouse_buttons[event.get_button()] = true;
 		}
 
-		if (m_user_interface_events) {
+		if (m_user_interface_events)
+		{
 			io.MouseDown[event.get_button()] = true;
 		}
 
@@ -95,11 +83,13 @@ void Input::on_event(const Event &inputEvent)
 	{
 		const auto &event = dynamic_cast<const ButtonReleasedEvent &>(inputEvent);
 
-		if (m_game_events) {
+		if (m_game_events)
+		{
 			m_mouse_buttons[event.get_button()] = false;
 		}
 
-		if (m_user_interface_events) {
+		if (m_user_interface_events)
+		{
 			io.MouseDown[event.get_button()] = false;
 		}
 
@@ -109,11 +99,13 @@ void Input::on_event(const Event &inputEvent)
 	{
 		const auto &event = dynamic_cast<const WheelScrolledEvent &>(inputEvent);
 
-		if (m_game_events) {
+		if (m_game_events)
+		{
 			m_mouse_wheel_delta = event.get_offset();
 		}
 
-		if (m_user_interface_events) {
+		if (m_user_interface_events)
+		{
 			io.MouseWheel = event.get_offset();
 		}
 
@@ -124,7 +116,8 @@ void Input::on_event(const Event &inputEvent)
 	{
 		const auto &event = dynamic_cast<const KeyPressedEvent &>(inputEvent);
 
-		if (m_game_events) {
+		if (m_game_events)
+		{
 			m_keyboad_keys[event.get_key()] = true;
 		}
 
@@ -141,11 +134,13 @@ void Input::on_event(const Event &inputEvent)
 	{
 		const auto &event = dynamic_cast<const KeyReleasedEvent &>(inputEvent);
 
-		if (m_game_events) {
+		if (m_game_events)
+		{
 			m_keyboad_keys[event.get_key()] = false;
 		}
 
-		if (m_user_interface_events) {
+		if (m_user_interface_events)
+		{
 			io.KeysDown[event.get_key()] = false;
 		}
 

modules/engine/src/layer/layer_stack.cpp

@@ -7,25 +7,10 @@
 
 namespace Light {
 
-LayerStack *LayerStack::s_context = nullptr;
-
-auto LayerStack::create() -> Scope<LayerStack>
-{
-	return make_scope<LayerStack>(new LayerStack());
-}
-
-LayerStack::LayerStack() 
-{
-	lt_assert(
-	    !s_context,
-	    "An instance of 'LayerStack' already exists, do not construct this class!"
-	) s_context
-	    = this;
-}
-
 LayerStack::~LayerStack()
 {
-	for (auto *layer : m_layers) {
+	for (auto *layer : m_layers)
+	{
 		delete layer;
 	}
 }

modules/engine/src/utils/resource_manager.cpp

@@ -6,19 +6,6 @@
 
 namespace Light {
 
-ResourceManager *ResourceManager::s_context = nullptr;
-
-auto ResourceManager::create() -> Scope<ResourceManager>
-{
-	return make_scope(new ResourceManager());
-}
-
-ResourceManager::ResourceManager() 
-{
-	lt_assert(!s_context, "Repeated singleton construction");
-	s_context = this;
-}
-
 void ResourceManager::load_shader_impl(
     const std::string &name,
     const std::string &vertexPath,
@@ -26,7 +13,6 @@ void ResourceManager::load_shader_impl(
 )
 {
 	// check
-	lt_assert(s_context, "Uninitliazed singleton");
 	lt_assert(!vertexPath.empty(), "Empty 'vertexPath'");
 	lt_assert(!pixelPath.empty(), "Empty 'pixelPath'");
 
@@ -54,8 +40,6 @@ void ResourceManager::load_texture_impl(
     unsigned int desiredComponents /* = 4u */
 )
 {
-	lt_assert(s_context, "Uninitliazed singleton");
-
 	// load file
 	auto imgFile = FileManager::read_image_file(path, desiredComponents);
 

modules/engine/src/utils/serializer.cpp

@@ -19,7 +19,8 @@ struct convert<glm::vec3>
 
 	static auto decode(const Node &node, glm::vec3 &rhs) -> bool
 	{
-		if (!node.IsSequence() || node.size() != 3) {
+		if (!node.IsSequence() || node.size() != 3)
+		{
 			return false;
 		}
 
@@ -45,7 +46,8 @@ struct convert<glm::vec4>
 
 	static auto decode(const Node &node, glm::vec4 &rhs) -> bool
 	{
-		if (!node.IsSequence() || node.size() != 4) {
+		if (!node.IsSequence() || node.size() != 4)
+		{
 			return false;
 		}
 
@@ -60,14 +62,14 @@ struct convert<glm::vec4>
 
 namespace Light {
 
-static auto operator<<(YAML::Emitter &out, const glm::vec3 &v) -> YAML::Emitter &
+auto operator<<(YAML::Emitter &out, const glm::vec3 &v) -> YAML::Emitter &
 {
 	out << YAML::Flow;
 	out << YAML::BeginSeq << v.x << v.y << v.z << YAML::EndSeq;
 	return out;
 }
 
-static auto operator<<(YAML::Emitter &out, const glm::vec4 &v) -> YAML::Emitter &
+auto operator<<(YAML::Emitter &out, const glm::vec4 &v) -> YAML::Emitter &
 {
 	out << YAML::Flow;
 	out << YAML::BeginSeq << v.x << v.y << v.z << v.w << YAML::EndSeq;