fix: bug fixes related to client-server communication, lobby and room management, and game state handling.

client/src/game/world/ClientWorld.cpp

@@ -431,7 +431,8 @@ namespace World
 
     void ClientWorld::reset()
     {
-        _registry = Ecs::Registry{};
+        _registry.clear();
+        _registry = Ecs::Registry();
         _entityMap.clear();
         _entityLastSeen.clear();
         _scoresByPlayerId.clear();

client/src/thread/ClientRuntime.cpp

@@ -357,7 +357,7 @@ namespace Thread
         _eventBus->on<Engine::JoinRoomRequested>([this](const Engine::JoinRoomRequested &e) {
             const auto req = nextReqId();
             _tcpClient->sendPacket(*_tcpPacketFactory.makeJoinRoom(req, e.roomId));
-            if (const auto pkt = _tcpPacketFactory.makeRoomInfo(11))
+            if (const auto pkt = _tcpPacketFactory.makeRoomInfo(nextReqId()))
                 _tcpClient->sendPacket(*pkt);
         });
 
@@ -385,6 +385,9 @@ namespace Thread
         });
 
         _eventBus->on<Engine::StartGameRequested>([this](const Engine::StartGameRequested &) {
+            if (!_stateManager->is<Engine::LobbyState>())
+                return;
+            _world->reset();
             if (const auto pkt = _tcpPacketFactory.makeStartGame(nextReqId()))
                 _tcpClient->sendPacket(*pkt);
         });
@@ -393,18 +396,6 @@ namespace Thread
             if (const auto pkt = _tcpPacketFactory.makeLeaveRoom(nextReqId()))
                 _tcpClient->sendPacket(*pkt);
             _pendingHome.store(true, std::memory_order_release);
-        });
-
-        _eventBus->on<Engine::UpdateRoomRequested>([this](const Engine::UpdateRoomRequested &) {
-            const auto req = nextReqId();
-            if (const auto pkt = _tcpPacketFactory.makeRoomInfo(req))
-                _tcpClient->sendPacket(*pkt);
-        });
-
-        _eventBus->on<Engine::LeaveRoomRequested>([this](const Engine::LeaveRoomRequested &) {
-            const auto req = nextReqId();
-            if (const auto pkt = _tcpPacketFactory.makeLeaveRoom(req))
-                (void) _tcpClient->sendPacket(*pkt);
             _world->reset();
             {
                 std::scoped_lock lock(_frameMutex);
@@ -415,6 +406,15 @@ namespace Thread
             }
         });
 
+        _eventBus->on<Engine::UpdateRoomRequested>([this](const Engine::UpdateRoomRequested &) {
+            if (!_stateManager->is<Engine::LobbyState>() || _pendingHome)
+                return;
+
+            const auto req = nextReqId();
+            if (const auto pkt = _tcpPacketFactory.makeRoomInfo(req))
+                _tcpClient->sendPacket(*pkt);
+        });
+
         _eventBus->on<Engine::SendingMessage>([this](const Engine::SendingMessage &e) {
             const auto req = nextReqId();
             if (const auto pkt = _tcpPacketFactory.makeRoomMessage(req, e.message))
@@ -489,6 +489,8 @@ namespace Thread
         });
 
         _tcpPacketRouter->sink()->onRoomUpdatedSubscribe([&](uint32_t, const RoomData &room) {
+            if (_stateManager->is<Engine::GameState>())
+                return;
             _roomManager->setCurrentData(room);
             if (!_stateManager->is<Engine::LobbyState>())
                 _pendingJoinRoom.store(true, std::memory_order_release);
@@ -573,4 +575,4 @@ namespace Thread
             (void) _udpClient->sendPacket(*pingPkt);
     }
 
-} // namespace Thread
+} // namespace Thread

client/src/ui/lobby/Lobby.hpp

@@ -195,8 +195,8 @@ namespace Engine
 
         bool _needUpdate = false; ///> Indicates if the lobby needs to be updated.
 
-        std::chrono::steady_clock::time_point _lastRefresh{};                 ///> Last refresh time point.
-        static constexpr auto refreshPeriod = std::chrono::milliseconds(500); ///> Refresh period.
+        std::chrono::steady_clock::time_point _lastRefresh{};                  ///> Last refresh time point.
+        static constexpr auto refreshPeriod = std::chrono::milliseconds(1000); ///> Refresh period.
 
         size_t _lobbyCapacity = 4; ///> Maximum capacity of the lobby.
 

client/src/ui/rooms/RoomMenu.cpp

@@ -418,6 +418,7 @@ namespace Engine
             if (_root.join->onClickReleased(mx, my, [&] {
                     _page = Page::List;
                     _listRooms = true;
+                    _createRoom = false;
                     _list.scroll = 0.f;
                     _list.lastRefresh = {};
                     updateListRooms();

server/src/game/gameServer/GameServer.cpp

@@ -113,8 +113,10 @@ namespace
                 const sockaddr_in *addr = sessionsL->getUdpAddress(event.sessionId);
                 if (!addr)
                     return;
-                if (const auto pkt = factoryL->createAcceptPacket(*addr, event.netPlayerId))
-                    (void) serverL->sendPacket(*pkt);
+                if (const auto pkt = factoryL->createAcceptPacket(*addr, event.netPlayerId)) {
+                    for (int i = 0; i < 3; i++)
+                        (void) serverL->sendPacket(*pkt);
+                }
             });
     }