Changeset 7543

Timestamp:

Oct 14, 2010, 11:58:56 PM (14 years ago)

Author:

jo

Message:

Bug description. Couldn't find a solution though.

File:

• code/branches/lastmanstanding/src/orxonox/gametypes/LastManStanding.cc (modified) (2 diffs)

code/branches/lastmanstanding/src/orxonox/gametypes/LastManStanding.cc

@@ -190 +190 @@
             return 0;
     }
-
+    /*BUG-Description:
+    *There are two ways for a player to be killed: Either receiving damage, or through the following function.
+    *The function works fine - until the last call, when a player looses his last live. Than the kill part
+    *(it->second.state_ = PlayerState::Dead;) somehow isn't executed:
+    *The player isn't killed (he doesn't leave play). Although the corresponding code is reached.
+    *
+    *How to reproduce this bug: Start a new Lastmanstanding-Match. Immdiately add 8 bots(before actually entering the level).
+    *Just fly around and wait. Don't shoot, since only passive behaviour triggers the killPlayer-Function.
+    */
     void LastManStanding::killPlayer(PlayerInfo* player)
     {
         if(!player)
             return;
-        std::map<PlayerInfo*, Player>::iterator it = this->players_.find(player);//!!!!!!!!!!!
+        std::map<PlayerInfo*, Player>::iterator it = this->players_.find(player);
         if (it != this->players_.end())
         {
-            it->second.state_ = PlayerState::Dead;//-------------killpart
-            it->second.killed_++;
-
-            playerLives_[player]=playerLives_[player]-1;//-----------datapart
-            if (playerLives_[player]<=0)//if player lost all lives
+            if (playerLives_[player]<=1)//if player lost all lives
             {
                 this->playersAlive--;
@@ -208 +212 @@
                 COUT(0) << message << std::endl;
                 Host::Broadcast(message);
-            }
-            this->timeToAct_[player]=timeRemaining+3.0f;//reset timer
+                playerLives_[player]=playerLives_[player]-1;//-----------datapart
+                it->second.killed_++;
+                it->second.state_ = PlayerState::Dead;//-------------killpart
+            }
+            else
+            {
+                playerLives_[player]=playerLives_[player]-1;//-----------datapart
+                it->second.killed_++;
+                it->second.state_ = PlayerState::Dead;//-------------killpart
+
+                this->timeToAct_[player]=timeRemaining+3.0f;//reset timer
+            }
         }
     }