Changeset e487381 in network-game

Timestamp:

May 25, 2013, 6:04:08 PM (11 years ago)

Author:

dportnoy <dmp1488@…>

Branches:

master

Children:

7511a2b

Parents:

626e5b0

Message:

The server removes objects from its map when they are picked up by players and processes DROP_FLAG messages

Files:

• common/WorldMap.cpp (modified) (1 diff)
• common/WorldMap.h (modified) (1 diff)
• server/server.cpp (modified) (6 diffs)

common/WorldMap.cpp

@@ -65 +65 @@
 }
 
-vector<WorldMap::Object> WorldMap::getObjects() {
-   return *vctObjects;
+vector<WorldMap::Object>* WorldMap::getObjects() {
+   return vctObjects;
 }
 vector<WorldMap::Object> WorldMap::getObjects(int x, int y) {

common/WorldMap.h

@@ -61 +61 @@
    void setStructure(int x, int y, StructureType type);
 
-   vector<Object> getObjects();
+   vector<Object>* getObjects();
    vector<Object> getObjects(int x, int y);
 

server/server.cpp

@@ -161 +161 @@
                }
 
-               vector<WorldMap::Object> vctObjects = gameMap->getObjects();
+               vector<WorldMap::Object>* vctObjects = gameMap->getObjects();
                vector<WorldMap::Object>::iterator itObjects;
 
-               for (itObjects = vctObjects.begin(); itObjects != vctObjects.end(); itObjects++) {
+               for (itObjects = vctObjects->begin(); itObjects != vctObjects->end();) {
                   POSITION pos = itObjects->pos;
                   if (posDistance(it->second.pos, pos.toFloat()) < 10) {
@@ -180 +180 @@
                      }
 
-                     // send a MSG_TYPE_REMOVE_OBJECT message
                      serverMsg.type = MSG_TYPE_REMOVE_OBJECT;
                      memcpy(serverMsg.buffer, &itObjects->id, 4);
@@ -190 +189 @@
                            error("sendMessage");
                      }
+
+                     // remove the object form the server-side map
+                     cout << "size before: " << gameMap->getObjects()->size() << endl;
+                     itObjects = vctObjects->erase(itObjects);
+                     cout << "size after: " << gameMap->getObjects()->size() << endl;
                   }
+                  else
+                     itObjects++;
                }
 
@@ -323 +329 @@
             // (currently just the flags)
             serverMsg.type = MSG_TYPE_OBJECT;
-            vector<WorldMap::Object> vctObjects = gameMap->getObjects();
+            vector<WorldMap::Object>* vctObjects = gameMap->getObjects();
             vector<WorldMap::Object>::iterator itObjects;
             cout << "sending items" << endl;
-            for (itObjects = vctObjects.begin(); itObjects != vctObjects.end(); itObjects++) {
+            for (itObjects = vctObjects->begin(); itObjects != vctObjects->end(); itObjects++) {
                itObjects->serialize(serverMsg.buffer);
                cout << "sending item id " << itObjects->id  << endl;
@@ -408 +414 @@
       case MSG_TYPE_PLAYER_MOVE:
       {
-         istringstream iss;
-         iss.str(clientMsg.buffer);
-
          cout << "PLAYER_MOVE" << endl;
 
@@ -451 +454 @@
          break;
       }
+      case MSG_TYPE_DROP_FLAG:
+      {
+         // may want to check the id matches the sender, just like for PLAYER_NOVE
+         cout << "DROP_FLAG" << endl;
+
+         int id;
+
+         memcpy(&id, clientMsg.buffer, 4);
+         cout << "id: " << id << endl;
+
+         WorldMap::ObjectType flagType = WorldMap::OBJECT_NONE;
+         if (mapPlayers[id].hasBlueFlag)
+            flagType = WorldMap::OBJECT_BLUE_FLAG;
+         else if (mapPlayers[id].hasRedFlag)
+            flagType = WorldMap::OBJECT_RED_FLAG;
+
+         gameMap->addObject(flagType, mapPlayers[id].pos.x, mapPlayers[id].pos.y);
+
+         // need to send the OBJECT message too
+         serverMsg.type = MSG_TYPE_OBJECT;
+         gameMap->getObjects()->back().serialize(serverMsg.buffer);
+
+         map<unsigned int, Player>::iterator it;
+         for (it = mapPlayers.begin(); it != mapPlayers.end(); it++)
+         {
+            if ( sendMessage(&serverMsg, sock, &(it->second.addr)) < 0 )
+               error("sendMessage");
+         }
+
+         mapPlayers[id].hasBlueFlag = false;
+         mapPlayers[id].hasRedFlag = false;
+
+         serverMsg.type = MSG_TYPE_PLAYER;
+         mapPlayers[id].serialize(serverMsg.buffer);
+
+         map<unsigned int, Player>::iterator it2;
+         broadcastResponse = true;
+
+         break;
+      }
       default:
       {