Changeset 360c0f1 in network-game for server

Timestamp:

Dec 28, 2013, 2:18:05 PM (11 years ago)

Author:

dportnoy <dmp1488@…>

Branches:

master

Children:

8aed9c0

Parents:

3ff2bd7

Message:

Server no longer sends OBJECT messages on player login or runs the drop flag logic on player logout

File:

• server/server.cpp (modified) (3 diffs)

server/server.cpp

@@ -392 +392 @@
             }
 
-            // tell the new player about all map objects
-            // (currently just the flags)
-            serverMsg.type = MSG_TYPE_OBJECT;
-            vector<WorldMap::Object>* vctObjects = gameMap->getObjects();
-            vector<WorldMap::Object>::iterator itObjects;
-            cout << "sending items" << endl;
-            for (itObjects = vctObjects->begin(); itObjects != vctObjects->end(); itObjects++) {
-               itObjects->serialize(serverMsg.buffer);
-               cout << "sending item id " << itObjects->id  << endl;
-               msgProcessor.sendMessage(&serverMsg, &from);
-            }
-
             // send info about existing games to new player
             map<string, Game*>::iterator itGames;
@@ -451 +439 @@
          else
          {
-            if (!p->isDead) {
-               WorldMap::ObjectType flagType = WorldMap::OBJECT_NONE;
-               if (p->hasBlueFlag)
-                  flagType = WorldMap::OBJECT_BLUE_FLAG;
-               else if (p->hasRedFlag)
-                  flagType = WorldMap::OBJECT_RED_FLAG;
-
-               if (flagType != WorldMap::OBJECT_NONE) {
-                  addObjectToMap(flagType, p->pos.x, p->pos.y, gameMap, mapPlayers, msgProcessor);
-               }
-            }
-
             // broadcast to all players before deleting p from the map
             serverMsg.type = MSG_TYPE_LOGOUT;
@@ -714 +690 @@
             p->currentGame = NULL;
 
+            // check if the player that's leaving needs to receive the
+            // LEAVE_GAME message and run thie logic below to make the player
+            // drop any flag he's carrying
+
+            /*
+            if (!p->isDead) {
+               WorldMap::ObjectType flagType = WorldMap::OBJECT_NONE;
+               if (p->hasBlueFlag)
+                  flagType = WorldMap::OBJECT_BLUE_FLAG;
+               else if (p->hasRedFlag)
+                  flagType = WorldMap::OBJECT_RED_FLAG;
+
+               if (flagType != WorldMap::OBJECT_NONE) {
+                  addObjectToMap(flagType, p->pos.x, p->pos.y, gameMap, mapPlayers, msgProcessor);
+               }
+            }
+            */
+
             serverMsg.type = MSG_TYPE_LEAVE_GAME;
             memcpy(serverMsg.buffer, &p->id, 4);