Changeset df74597 in network-game

Timestamp:

Dec 25, 2013, 4:44:20 PM (11 years ago)

Author:

dportnoy <dmp1488@…>

Branches:

master

Children:

257de75

Parents:

e0fd377

Message:

Server-side logic for handling a game victory is fixed

File:

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

server/server.cpp

@@ -216 +216 @@
          for (itGames = mapGames.begin(); itGames != mapGames.end();) { 
             if (handleGameEvents(itGames->second, mapPlayers, msgProcessor)) {
+               delete itGames->second;
                mapGames.erase(itGames++);
-               delete itGames->second;
             }else
                itGames++;
@@ -831 +831 @@
    NETWORK_MSG serverMsg;
    map<unsigned int, Player*>::iterator it;
+   bool gameFinished = false;
 
    for (it = game->getPlayers().begin(); it != game->getPlayers().end(); it++)
    {
-      if (handlePlayerEvents(it->second, game, msgProcessor)) {
-                // send a GAME_INFO message with 0 players to force clients to delete the game
-         int numPlayers = 0;
-         serverMsg.type = MSG_TYPE_GAME_INFO;
-         memcpy(serverMsg.buffer, &numPlayers, 4);
-         broadcastMessage(msgProcessor, serverMsg, mapPlayers);
-
-         return true;
-      }else
-         return false;
-   }
+      gameFinished = gameFinished ||
+         handlePlayerEvents(it->second, game, msgProcessor);
+   }
+
+   // set each player's current game to null
+   if (gameFinished) {
+      // send a GAME_INFO message with 0 players to force clients to delete the game
+      int numPlayers = 0;
+      serverMsg.type = MSG_TYPE_GAME_INFO;
+      memcpy(serverMsg.buffer, &numPlayers, 4);
+      broadcastMessage(msgProcessor, serverMsg, mapPlayers);
+
+      for (it = game->getPlayers().begin(); it != game->getPlayers().end(); it++)
+      {
+         it->second->currentGame = NULL;
+      }
+   }
+
+   return gameFinished;
 }
 
@@ -1097 +1106 @@
       broadcastMessage(msgProcessor, serverMsg, playersInGame);
    }
+
+   return gameFinished;
 }