Better player save & load

src/world/Level.cpp

@@ -12,11 +12,6 @@
 #include "../items/Inventory.h"
 #include "../items/Inventories.h"
 
-
-const float DEF_PLAYER_Y = 100.0f;
-const float DEF_PLAYER_SPEED = 4.0f;
-const int DEF_PLAYER_INVENTORY_SIZE = 40;
-
 Level::Level(World* world, const Content* content, EngineSettings& settings)
 	  : world(world),
 	    content(content),
@@ -25,7 +20,7 @@ Level::Level(World* world, const Content* content, EngineSettings& settings)
         events(std::make_unique<LevelEvents>()),
 		settings(settings) 
 {
-	auto inv = std::make_shared<Inventory>(0, DEF_PLAYER_INVENTORY_SIZE);
+	auto inv = std::make_shared<Inventory>(world->getNextInventoryId(), DEF_PLAYER_INVENTORY_SIZE);
 	auto player = spawnObject<Player>(glm::vec3(0, DEF_PLAYER_Y, 0), DEF_PLAYER_SPEED, inv);
 
     uint matrixSize = (settings.chunks.loadDistance + settings.chunks.padding) * 2;

src/world/Level.h

@@ -7,6 +7,10 @@
 #include "../interfaces/Object.h"
 #include <vector>
 
+const float DEF_PLAYER_Y = 100.0f;
+const float DEF_PLAYER_SPEED = 4.0f;
+const int DEF_PLAYER_INVENTORY_SIZE = 40;
+
 class Content;
 class World;
 class Player;

src/world/World.cpp

@@ -64,11 +64,16 @@ void World::write(Level* level) {
     }
 
     wfile->write(this, content);
-    for (auto object : level->objects) {
-        if (std::shared_ptr<Player> player = std::dynamic_pointer_cast<Player>(object)) {
-            wfile->writePlayer(player);
+	auto playerFile = dynamic::Map();
+    {
+        auto& players = playerFile.putList("players");
+        for (auto object : level->objects) {
+            if (std::shared_ptr<Player> player = std::dynamic_pointer_cast<Player>(object)) {
+                players.put(player->serialize().release());
+            }
         }
     }
+    files::write_json(wfile->getPlayerFile(), &playerFile);
 }
 
 Level* World::create(std::string name, 
@@ -98,10 +103,25 @@ Level* World::load(fs::path directory,
     }
 
     auto level = new Level(world.get(), content, settings);
-    for (auto object : level->objects) {
-        if (std::shared_ptr<Player> player = std::dynamic_pointer_cast<Player>(object)) {
-            wfile->readPlayer(player);
-            level->inventories->store(player->getInventory());
+    {
+        fs::path file = wfile->getPlayerFile();
+        if (!fs::is_regular_file(file)) {
+            std::cerr << "warning: player.json does not exists" << std::endl;
+        } else {
+            auto playerFile = files::read_json(file);
+            if (playerFile->has("players")) {
+                level->objects.clear();
+                auto players = playerFile->list("players");
+                for (size_t i = 0; i < players->size(); i++) {
+                    auto player = level->spawnObject<Player>(glm::vec3(0, DEF_PLAYER_Y, 0), DEF_PLAYER_SPEED, level->inventories->create(DEF_PLAYER_INVENTORY_SIZE));
+                    player->deserialize(players->map(i));
+                    level->inventories->store(player->getInventory());
+                }
+            } else {
+	            auto player = level->getObject<Player>(0);
+                player->deserialize(playerFile.get());
+                level->inventories->store(player->getInventory());
+            }
         }
     }
     (void)world.release();

src/world/World.h

@@ -36,7 +36,7 @@ class World : Serializable {
     const Content* const content;
     std::vector<ContentPack> packs;
 
-    int64_t nextInventoryId = 1;
+    int64_t nextInventoryId = 0;
 public:
     std::unique_ptr<WorldFiles> wfile;