From 55e561fd1471d2d1581d2ccf2cba849f3174468f Mon Sep 17 00:00:00 2001
From: CatChow0 <thetrueshibadoggo@gmail.com>
Date: Sat, 6 Sep 2025 14:18:28 +0200
Subject: [PATCH] Minor - Implements component serialization - V12.9.0

Adds serialization and deserialization functionality to the ECS component system.

This allows components to be saved and loaded, enabling scene persistence.
The IdentityComponent is updated to support serialization/deserialization.
The scene saving logic in scene_manager is updated to serialize components instead of hardcoded values.
---
 .../.idea/workspace.xml                       |  21 +-
 enginecustom/imgui.ini                        |  10 +-
 enginecustom/src/inc/system/ecs/component.h   |  15 +-
 .../ecs/components/identity_component.h       |  46 ++++
 enginecustom/src/inc/system/ecs/entity.h      |   8 +
 enginecustom/src/src/system/scene_manager.cpp | 201 ++++++++++--------
 6 files changed, 190 insertions(+), 111 deletions(-)

diff --git a/.idea/.idea.KhaoticEngineReborn/.idea/workspace.xml b/.idea/.idea.KhaoticEngineReborn/.idea/workspace.xml
index 8745a33..6cf7b6f 100644
--- a/.idea/.idea.KhaoticEngineReborn/.idea/workspace.xml
+++ b/.idea/.idea.KhaoticEngineReborn/.idea/workspace.xml
@@ -5,16 +5,11 @@
   </component>
   <component name="ChangeListManager">
     <list default="true" id="e81d6e08-efc7-40a0-909d-ec4943d948e9" name="Changes" comment="">
-      <change beforePath="$PROJECT_DIR$/enginecustom/src/inc/system/Logger.h" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/src/inc/system/Logger.h" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/enginecustom/src/inc/system/Skybox.h" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/src/inc/system/Skybox.h" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/enginecustom/src/inc/system/camera_class.h" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/src/inc/system/camera_class.h" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/enginecustom/src/inc/system/d_3d_class.h" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/src/inc/system/d_3d_class.h" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/enginecustom/src/inc/system/display_plane_class.h" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/src/inc/system/display_plane_class.h" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/enginecustom/src/inc/system/fps_limiter.h" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/src/inc/system/fps_limiter.h" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/enginecustom/src/inc/system/frustum.h" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/src/inc/system/frustum.h" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/enginecustom/src/inc/system/frustumclass.h" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/src/inc/system/frustumclass.h" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/enginecustom/src/inc/system/imguiManager.h" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/src/inc/system/imguiManager.h" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/enginecustom/src/inc/system/scene_manager.h" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/src/inc/system/scene_manager.h" afterDir="false" />
+      <change beforePath="$PROJECT_DIR$/.idea/.idea.KhaoticEngineReborn/.idea/workspace.xml" beforeDir="false" afterPath="$PROJECT_DIR$/.idea/.idea.KhaoticEngineReborn/.idea/workspace.xml" afterDir="false" />
+      <change beforePath="$PROJECT_DIR$/enginecustom/imgui.ini" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/imgui.ini" afterDir="false" />
+      <change beforePath="$PROJECT_DIR$/enginecustom/src/inc/system/ecs/component.h" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/src/inc/system/ecs/component.h" afterDir="false" />
+      <change beforePath="$PROJECT_DIR$/enginecustom/src/inc/system/ecs/components/identity_component.h" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/src/inc/system/ecs/components/identity_component.h" afterDir="false" />
+      <change beforePath="$PROJECT_DIR$/enginecustom/src/inc/system/ecs/entity.h" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/src/inc/system/ecs/entity.h" afterDir="false" />
       <change beforePath="$PROJECT_DIR$/enginecustom/src/src/system/scene_manager.cpp" beforeDir="false" afterPath="$PROJECT_DIR$/enginecustom/src/src/system/scene_manager.cpp" afterDir="false" />
     </list>
     <option name="SHOW_DIALOG" value="false" />
@@ -81,7 +76,7 @@
     &quot;node.js.selected.package.eslint&quot;: &quot;(autodetect)&quot;,
     &quot;node.js.selected.package.tslint&quot;: &quot;(autodetect)&quot;,
     &quot;nodejs_package_manager_path&quot;: &quot;npm&quot;,
-    &quot;settings.editor.selected.configurable&quot;: &quot;preferences.environmentSetup&quot;,
+    &quot;settings.editor.selected.configurable&quot;: &quot;MonitoringCountersPageId&quot;,
     &quot;vue.rearranger.settings.migration&quot;: &quot;true&quot;
   }
 }</component>
@@ -242,6 +237,10 @@
       <workItem from="1753700140931" duration="918000" />
       <workItem from="1753701507863" duration="7153000" />
       <workItem from="1753713925469" duration="1739000" />
+      <workItem from="1754572996727" duration="224000" />
+      <workItem from="1755617169013" duration="222000" />
+      <workItem from="1757101936080" duration="185000" />
+      <workItem from="1757158576637" duration="2501000" />
     </task>
     <task id="LOCAL-00001" summary="Minor update - viewport window tweak">
       <option name="closed" value="true" />
diff --git a/enginecustom/imgui.ini b/enginecustom/imgui.ini
index fed8312..6dbab2a 100644
--- a/enginecustom/imgui.ini
+++ b/enginecustom/imgui.ini
@@ -22,13 +22,13 @@ DockId=0x00000009,0
 
 [Window][Objects]
 Pos=0,19
-Size=234,609
+Size=234,842
 Collapsed=0
 DockId=0x0000000B,0
 
 [Window][Terrain]
 Pos=236,19
-Size=266,609
+Size=266,842
 Collapsed=0
 DockId=0x00000007,0
 
@@ -45,15 +45,15 @@ Collapsed=0
 DockId=0x0000000B,1
 
 [Window][Engine Settings]
-Pos=1267,462
-Size=317,166
+Pos=1267,631
+Size=317,230
 Collapsed=0
 DockId=0x00000006,0
 
 [Docking][Data]
 DockSpace           ID=0xCCBD8CF7 Window=0x3DA2F1DE Pos=0,19 Size=1584,842 Split=Y
   DockNode          ID=0x00000003 Parent=0xCCBD8CF7 SizeRef=1584,609 Split=X
-    DockNode        ID=0x0000000B Parent=0x00000003 SizeRef=234,842 Selected=0x321620B2
+    DockNode        ID=0x0000000B Parent=0x00000003 SizeRef=234,842 Selected=0x031DC75C
     DockNode        ID=0x0000000C Parent=0x00000003 SizeRef=1348,842 Split=X
       DockNode      ID=0x00000007 Parent=0x0000000C SizeRef=266,842 Selected=0x393905AB
       DockNode      ID=0x00000008 Parent=0x0000000C SizeRef=1316,842 Split=X
diff --git a/enginecustom/src/inc/system/ecs/component.h b/enginecustom/src/inc/system/ecs/component.h
index 8732856..911cd68 100644
--- a/enginecustom/src/inc/system/ecs/component.h
+++ b/enginecustom/src/inc/system/ecs/component.h
@@ -33,8 +33,19 @@ public:
      */
     virtual void Update(float deltaTime) {}
 
-    // virtual std::string Serialize() {}
-    // virtual void Deserialize(const std::string& data) {}
+    /**
+     * Serialize the component to a string (for debugging or saving).
+     * @return A string representation of the component.
+     */
+    virtual std::string Serialize() const { return ""; }
+
+    /**
+     * Deserialize the component from a string (for loading).
+     * @param data The string data to deserialize from.
+     * @return True if deserialization was successful, otherwise false.
+     */
+    virtual bool Deserialize(const std::string& data) { return false;}
+    
 };
 
 /**
diff --git a/enginecustom/src/inc/system/ecs/components/identity_component.h b/enginecustom/src/inc/system/ecs/components/identity_component.h
index c35f161..4bfc595 100644
--- a/enginecustom/src/inc/system/ecs/components/identity_component.h
+++ b/enginecustom/src/inc/system/ecs/components/identity_component.h
@@ -92,6 +92,52 @@ public:
         return ObjectType::Unknown;
     }
 
+    /**
+     * Serialize the component to a string.
+     * Format: "id:name:type"
+     * @return A string representation of the component.
+     */
+    std::string Serialize() const override
+    {
+        std::stringstream ss;
+        ss << m_id << ":" << m_name << ":" << ObjectTypeToString(m_type);
+        return ss.str();
+    }
+
+    /**
+     * Deserialize the component from a string.
+     * Expected format: "id:name:type"
+     * @param data The string data to deserialize from.
+     * @return True if deserialization was successful, otherwise false.
+     */
+    bool Deserialize(const std::string& data) override
+    {
+        std::stringstream ss(data);
+        std::string segment;
+        std::vector<std::string> segments;
+
+        while (std::getline(ss, segment, ':'))
+        {
+            segments.push_back(segment);
+        }
+
+        if (segments.empty()) return false;
+        if (segments.size() != 3) return false;
+
+        try
+        {
+            m_id = std::stoi(segments[0]);
+            m_name = segments[1];
+            m_type = StringToObjectType(segments[2]);
+            return true;
+        } catch (const std::exception&)
+        {
+            return false;
+        }
+        
+    }
+
+
 private:
     int m_id; // ID unique de l'objet
     std::string m_name; // Nom de l'objet
diff --git a/enginecustom/src/inc/system/ecs/entity.h b/enginecustom/src/inc/system/ecs/entity.h
index 860757b..c32663e 100644
--- a/enginecustom/src/inc/system/ecs/entity.h
+++ b/enginecustom/src/inc/system/ecs/entity.h
@@ -124,6 +124,14 @@ public:
         }
     }
 
+    /**
+     * Get all components of the entity.
+     * @return A constant reference to the map of components.
+     */
+    const std::unordered_map<ComponentTypeID, ComponentPtr>& GetAllComponents() const {
+        return m_Components;
+    }
+
 private:
 
     /**
diff --git a/enginecustom/src/src/system/scene_manager.cpp b/enginecustom/src/src/system/scene_manager.cpp
index 86dffe1..2356246 100644
--- a/enginecustom/src/src/system/scene_manager.cpp
+++ b/enginecustom/src/src/system/scene_manager.cpp
@@ -310,106 +310,121 @@ bool scene_manager::save_scene() {
         return false;
     }
 
-    for (const auto& object : entity_) {
-        XMFLOAT3 position, scale, rotation;
-        int id = 0;
-        int mass = 0;
-        float boundingRadius = 0;
-        std::string name = "NONE";
-        std::string shaderType = "NONE";
-        std::string objectType = "NONE";
-        std::wstring model_path = L"";
-        bool physics_enabled = false;
+    outFile << entity_.size() << std::endl;
 
-        auto transform = object->GetComponent<ecs::TransformComponent>();
-        if (transform) {
-            // convert XMVECTOR to XMFLOAT3
-            XMStoreFloat3(&position, transform->GetPosition());
-            XMStoreFloat3(&rotation, transform->GetRotation());
-            XMStoreFloat3(&scale, transform->GetScale());
-        }
+    for (const auto& entity : entity_)
+    {
+        const auto& components = entity->GetAllComponents();
 
-        
-        auto identity = object->GetComponent<ecs::IdentityComponent>();
-        if (identity) {
+        outFile << components.size() << std::endl;
 
-            id = identity->GetId();
-            name = identity->GetName();
-            objectType = identity->ObjectTypeToString(identity->GetType());
-            
-        }
-
-        auto model_path_component = object->GetComponent<ecs::ModelPathComponent>();
-        if (model_path_component) {
-            
-            model_path = model_path_component->GetPath();
-        }
-
-        auto shader = object->GetComponent<ecs::ShaderComponent>();
-        if (shader)
+        for (const auto& component : components)
         {
-            shaderType = shader->ShaderTypeToString(shader->GetActiveShader());
+            outFile << typeid(*component).name() << std::endl;
+            outFile << component->Serialize() << std::endl;
         }
-
-        auto physics = object->GetComponent<ecs::PhysicsComponent>();
-        if (physics) {
-            physics_enabled = physics->IsPhysicsEnabled();
-            mass = physics->GetMass();
-            boundingRadius = physics->GetBoundingRadius();
-        }
-        
-        // Écrire les données de base de l'objet
-        outFile << id << " "
-                << name << " "
-                << position.x << " " << position.y << " " << position.z << " "
-                << rotation.x << " " << rotation.y << " " << rotation.z << " "
-                << scale.x << " " << scale.y << " " << scale.z << " "
-                << convert_w_string_to_string(model_path) << " "
-                << shaderType << " "
-                << boundingRadius << " "
-                << objectType << " "
-                << mass << " "
-                << physics_enabled;
-        
-        // Sauvegarder les chemins des textures_
-        // Format: nombre de textures_ diffuses, puis les chemins
-        // Même chose pour les autres types de textures_
-
-
-        auto render = object->GetComponent<ecs::RenderComponent>();
-        if (render)
-        {
-            const auto& model = render->GetModel();
-
-            const auto& textureContainer = model->GetTextureContainer();
-
-            const auto& diffusePaths = textureContainer.GetPaths(TextureType::Diffuse);
-            outFile << " " << diffusePaths.size();
-            for (const auto& path : diffusePaths) {
-                outFile << " " << convert_w_string_to_string(path);
-            }
-
-            const auto& normalPaths = textureContainer.GetPaths(TextureType::Normal);
-            outFile << " " << normalPaths.size();
-            for (const auto& path : normalPaths) {
-                outFile << " " << convert_w_string_to_string(path);
-            }
-            const auto& specularPaths = textureContainer.GetPaths(TextureType::Specular);
-            outFile << " " << specularPaths.size();
-            for (const auto& path : specularPaths) {
-                outFile << " " << convert_w_string_to_string(path);
-            }
-            const auto& alphaPaths = textureContainer.GetPaths(TextureType::Alpha);
-            outFile << " " << alphaPaths.size();
-            for (const auto& path : alphaPaths) {
-                outFile << " " << convert_w_string_to_string(path);
-            }
-        }
-        
-        
-        outFile << std::endl;
     }
 
+    // for (const auto& object : entity_) {
+    //     XMFLOAT3 position, scale, rotation;
+    //     int id = 0;
+    //     int mass = 0;
+    //     float boundingRadius = 0;
+    //     std::string name = "NONE";
+    //     std::string shaderType = "NONE";
+    //     std::string objectType = "NONE";
+    //     std::wstring model_path = L"";
+    //     bool physics_enabled = false;
+    //
+    //     auto transform = object->GetComponent<ecs::TransformComponent>();
+    //     if (transform) {
+    //         // convert XMVECTOR to XMFLOAT3
+    //         XMStoreFloat3(&position, transform->GetPosition());
+    //         XMStoreFloat3(&rotation, transform->GetRotation());
+    //         XMStoreFloat3(&scale, transform->GetScale());
+    //     }
+    //
+    //     
+    //     auto identity = object->GetComponent<ecs::IdentityComponent>();
+    //     if (identity) {
+    //
+    //         id = identity->GetId();
+    //         name = identity->GetName();
+    //         objectType = identity->ObjectTypeToString(identity->GetType());
+    //         
+    //     }
+    //
+    //     auto model_path_component = object->GetComponent<ecs::ModelPathComponent>();
+    //     if (model_path_component) {
+    //         
+    //         model_path = model_path_component->GetPath();
+    //     }
+    //
+    //     auto shader = object->GetComponent<ecs::ShaderComponent>();
+    //     if (shader)
+    //     {
+    //         shaderType = shader->ShaderTypeToString(shader->GetActiveShader());
+    //     }
+    //
+    //     auto physics = object->GetComponent<ecs::PhysicsComponent>();
+    //     if (physics) {
+    //         physics_enabled = physics->IsPhysicsEnabled();
+    //         mass = physics->GetMass();
+    //         boundingRadius = physics->GetBoundingRadius();
+    //     }
+    //     
+    //     // Écrire les données de base de l'objet
+    //     outFile << id << " "
+    //             << name << " "
+    //             << position.x << " " << position.y << " " << position.z << " "
+    //             << rotation.x << " " << rotation.y << " " << rotation.z << " "
+    //             << scale.x << " " << scale.y << " " << scale.z << " "
+    //             << convert_w_string_to_string(model_path) << " "
+    //             << shaderType << " "
+    //             << boundingRadius << " "
+    //             << objectType << " "
+    //             << mass << " "
+    //             << physics_enabled;
+    //     
+    //     // Sauvegarder les chemins des textures_
+    //     // Format: nombre de textures_ diffuses, puis les chemins
+    //     // Même chose pour les autres types de textures_
+    //
+    //
+    //     auto render = object->GetComponent<ecs::RenderComponent>();
+    //     if (render)
+    //     {
+    //         const auto& model = render->GetModel();
+    //
+    //         const auto& textureContainer = model->GetTextureContainer();
+    //
+    //         const auto& diffusePaths = textureContainer.GetPaths(TextureType::Diffuse);
+    //         outFile << " " << diffusePaths.size();
+    //         for (const auto& path : diffusePaths) {
+    //             outFile << " " << convert_w_string_to_string(path);
+    //         }
+    //
+    //         const auto& normalPaths = textureContainer.GetPaths(TextureType::Normal);
+    //         outFile << " " << normalPaths.size();
+    //         for (const auto& path : normalPaths) {
+    //             outFile << " " << convert_w_string_to_string(path);
+    //         }
+    //         const auto& specularPaths = textureContainer.GetPaths(TextureType::Specular);
+    //         outFile << " " << specularPaths.size();
+    //         for (const auto& path : specularPaths) {
+    //             outFile << " " << convert_w_string_to_string(path);
+    //         }
+    //         const auto& alphaPaths = textureContainer.GetPaths(TextureType::Alpha);
+    //         outFile << " " << alphaPaths.size();
+    //         for (const auto& path : alphaPaths) {
+    //             outFile << " " << convert_w_string_to_string(path);
+    //         }
+    //     }
+    //     
+    //     
+    //     outFile << std::endl;
+    // }
+
     outFile.close();
     Logger::Get().Log("Scene saved successfully to " + scene_path_, __FILE__, __LINE__, Logger::LogLevel::Info);
     return true;