Changeset 95ffe57 in network-game for client/Client/main.cpp

Timestamp:

Oct 4, 2013, 5:06:20 PM (11 years ago)

Author:

dportnoy <dmp1488@…>

Branches:

master

Children:

90eaad2

Parents:

e6c26b8

Message:

The server now has a map of Player pointers instead of Players and MSG_TYPE_PLAYER is sent to notify a player joining a game about the players already in the game

File:

• client/Client/main.cpp (modified) (2 diffs)

client/Client/main.cpp

@@ -674 +674 @@
                {
                   wndCurrent = wndLobby;
-                  
-                  Player p("", "");
-                  p.deserialize(msg.buffer);
-
-                  if (mapPlayers.find(p.id) != mapPlayers.end())
-                     delete mapPlayers[p.id];
-                  mapPlayers[p.id] = new Player(p);
-                  curPlayerId = p.id;
+                
+                  // this message should only be sent when a player first logs in so they know their id
+  
+                  Player* p = new Player("", "");
+                  p->deserialize(msg.buffer);
+
+                  if (mapPlayers.find(p->id) != mapPlayers.end())
+                     delete mapPlayers[p->id];
+                  mapPlayers[p->id] = p;
+                  curPlayerId = p->id;
 
                   cout << "Got a valid login response with the player" << endl;
                   cout << "Player id: " << curPlayerId << endl;
-                  cout << "Player health: " << p.health << endl;
+                  cout << "Player health: " << p->health << endl;
                   cout << "player map size: " << mapPlayers.size() << endl;
                }
@@ -708 +710 @@
                cout << "Received MSG_TYPE_PLAYER" << endl;
 
-               Player p("", "");
-               p.deserialize(msg.buffer);
-               p.timeLastUpdated = getCurrentMillis();
-               p.isChasing = false;
-               if (p.health <= 0)
-                  p.isDead = true;
+               Player* p = new Player("", "");
+               p->deserialize(msg.buffer);
+               p->timeLastUpdated = getCurrentMillis();
+               p->isChasing = false;
+               if (p->health <= 0)
+                  p->isDead = true;
                else
-                  p.isDead = false;
-
-               if (mapPlayers.find(p.id) != mapPlayers.end())
-                    delete mapPlayers[p.id];
-               mapPlayers[p.id] = new Player(p);
+                  p->isDead = false;
+
+               if (mapPlayers.find(p->id) != mapPlayers.end())
+                    delete mapPlayers[p->id];
+               mapPlayers[p->id] = p;
 
                break;