Added some basic code to RoomManager

server/room_data.hpp

@@ -27,11 +27,11 @@
 
 struct RoomData {
 	enum class RoomType {
-		OVERWORLD,
+		OVERWORLD = 0,
-		RUINS,
+		RUINS = 1,
-		TOWERS,
+		TOWERS = 2,
-		FORESTS,
+		FORESTS = 3,
-		CAVES,
+		CAVE = 4,
 	};
 
 	//members

server/room_manager.cpp

@@ -21,34 +21,52 @@
 */
 #include "room_manager.hpp"
 
+#include <stdexcept>
+
 //-------------------------
 //public access methods
 //-------------------------
 
-//TODO
+RoomData* RoomManager::CreateRoom(int uid) {
+	//don't overwrite existing rooms
+	std::map<int, RoomData*>::iterator it = roomMap.find(uid);
+	if (it != roomMap.end()) {
+		throw(std::runtime_error("Cannot overwrite an existing room"));
+	}
+	roomMap[uid] = new RoomData();
+	//TODO: create room in the API
+	if (luaState) {
+		roomMap[uid]->pager.SetLuaState(luaState);
+	}
+	return roomMap[uid];
+}
 
-//-------------------------
+RoomData* RoomManager::UnloadRoom(int uid) {
-//accessors and mutators
+	//TODO: unload room in the API
-//-------------------------
+	delete roomMap[uid];
+	roomMap.erase(uid);
+}
 
 RoomData* RoomManager::GetRoom(int uid) {
-	std::map<int, RoomData>::iterator it = roomMap.find(uid);
+	RoomData* ptr = FindRoom(uid);
+	if (ptr) return ptr;
+	ptr = CreateRoom(uid);
+	return ptr;
+}
 
+RoomData* RoomManager::FindRoom(int uid) {
+	std::map<int, RoomData*>::iterator it = roomMap.find(uid);
 	if (it == roomMap.end()) {
 		return nullptr;
 	}
-
+	return it->second;
-	return &it->second;
 }
 
-std::map<int, RoomData>* RoomManager::GetContainer() {
+RoomData* RoomManager::PushRoom(int uid, RoomData* room) {
-	return &roomMap;
+	//unload existing rooms with this index
-}
+	std::map<int, RoomData*>::iterator it = roomMap.find(uid);
-
+	if (it != roomMap.end()) {
-lua_State* RoomManager::SetLuaState(lua_State* L) {
+		UnloadRoom(uid);
-	return luaState = L;
+	}
-}
+	roomMap[uid] = room;
-
-lua_State* RoomManager::GetLuaState() {
-	return luaState;
 }

server/room_manager.hpp

@@ -36,17 +36,21 @@ public:
 	~RoomManager() = default;
 
 	//public access methods
-	//TODO: Fill this out
+	RoomData* CreateRoom(int uid);
+	RoomData* UnloadRoom(int uid);
+
+	RoomData* GetRoom(int uid);
+	RoomData* FindRoom(int uid);
+	RoomData* PushRoom(int uid, RoomData*);
 
 	//accessors and mutators
-	RoomData* GetRoom(int uid);
+	std::map<int, RoomData*>* GetContainer() { return &roomMap; }
-	std::map<int, RoomData>* GetContainer();
 
-	lua_State* SetLuaState(lua_State*);
+	lua_State* SetLuaState(lua_State* L) { return luaState = L; }
-	lua_State* GetLuaState();
+	lua_State* GetLuaState() { return luaState; }
 
 private:
-	std::map<int, RoomData> roomMap;
+	std::map<int, RoomData*> roomMap;
 	lua_State* luaState = nullptr;
 };
 

server/server_application.cpp

@@ -316,7 +316,7 @@ void ServerApplication::HandleRegionRequest(RegionPacket* const argPacket) {
 	newPacket.region = roomMgr.GetRoom(argPacket->roomIndex)->pager.GetRegion(argPacket->x, argPacket->y);
 
 	//send the content
-	network.SendTo(&argPacket->srcAddress, static_cast<SerialPacket*>(argPacket));
+	network.SendTo(&argPacket->srcAddress, static_cast<SerialPacket*>(&newPacket));
 }
 
 //-------------------------

todo.txt

@@ -1,5 +1,5 @@
+TODO: encapsulate the data structures
 TODO: Get the rooms working
-TODO: update the map API to handle multiple rooms
 
 TODO: Rejection packets
 TODO: Authentication