Changeset 204edcf in network-game

Timestamp:

Jan 20, 2014, 7:10:55 PM (11 years ago)

Author:

dportnoy <dmp1488@…>

Branches:

master

Children:

0678d60

Parents:

5b92307

Message:

Server uses setters and getters to access id and targetPlayer attributes of the Player class

File:

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

server/server.cpp

@@ -171 +171 @@
             if (p->currentGame != NULL) {
                map<unsigned int, Player*> playersInGame = p->currentGame->getPlayers();
-               if (p->updateTarget(playersInGame[p->targetPlayer]))
+               if (p->updateTarget(playersInGame))
                {
                   serverMsg.type = MSG_TYPE_PLAYER;
@@ -333 +333 @@
          {
             updateUnusedPlayerId(unusedPlayerId, mapPlayers);
-            p->id = unusedPlayerId;
-            cout << "new player id: " << p->id << endl;
+            p->setId(unusedPlayerId);
+            cout << "new player id: " << p->getId() << endl;
             p->setAddr(from);
             p->currentGame = NULL;
@@ -351 +351 @@
 
                cout << "sending info about " << it->second->name  << endl;
-               cout << "sending id " << it->second->id  << endl;
+               cout << "sending id " << it->second->getId()  << endl;
                msgProcessor.sendMessage(&serverMsg, &from);
             }
@@ -403 +403 @@
          {
             // broadcast to all players before deleting p from the map
+            unsigned int playerId = p->getId();
             serverMsg.type = MSG_TYPE_LOGOUT;
-            memcpy(serverMsg.buffer, &p->id, 4);
+            memcpy(serverMsg.buffer, &playerId, 4);
 
             msgProcessor.broadcastMessage(serverMsg, mapPlayers);
 
-            if (p->id < unusedPlayerId)
-               unusedPlayerId = p->id;
-
-            mapPlayers.erase(p->id);
+            if (p->getId() < unusedPlayerId)
+               unusedPlayerId = p->getId();
+
+            mapPlayers.erase(p->getId());
             delete p;
 
@@ -559 +560 @@
 
          Player* p = mapPlayers[id];
-         p->targetPlayer = targetId;
+         p->setTargetPlayer(targetId);
          p->isChasing = true;
 
@@ -627 +628 @@
             Player* p = findPlayerByAddr(mapPlayers, from);
 
-            if (players.find(p->id) != players.end()) {
+            if (players.find(p->getId()) != players.end()) {
                cout << "Player " << p->name << " trying to join a game he's already in" << endl;
                serverMsg.type = MSG_TYPE_JOIN_GAME_FAILURE;
@@ -667 +668 @@
 
             p->currentGame = NULL;
-            g->removePlayer(p->id);
-
+            g->removePlayer(p->getId());
+
+            unsigned int playerId = p->getId();
             serverMsg.type = MSG_TYPE_LEAVE_GAME;
-            memcpy(serverMsg.buffer, &p->id, 4);
+            memcpy(serverMsg.buffer, &playerId, 4);
             strcpy(serverMsg.buffer+4, g->getName().c_str());
             msgProcessor.broadcastMessage(serverMsg, g->getPlayers());
@@ -755 +757 @@
 
             cout << "sending info about " << it->second->name  << endl;
-            cout << "sending id " << it->second->id  << endl;
+            cout << "sending id " << it->second->getId()  << endl;
             msgProcessor.sendMessage(&serverMsg, &from);
          }