Changeset 11716 for code/trunk/src/orxonox

Timestamp:

Jan 7, 2018, 9:48:58 PM (6 years ago)

Author:

landauf

Message:

[Highscore_HS16] moved logic to compare old an new score to Highscore; fixed formatting in Gametypes

Location:

code/trunk/src/orxonox

Files:

• Highscore.cc (modified) (1 diff)
• Highscore.h (modified) (1 diff)

code/trunk/src/orxonox/Highscore.cc

@@ -50 +50 @@
 
     /**
-     * @brief stores a new highscore in the orxonox.ini file
+     * @brief stores a new highscore in the orxonox.ini file if the new score is better than the highest score so far.
      */
-    void Highscore::storeHighscore(const std::string& level, int points, PlayerInfo* player){
-        ModifyConfigValue(highscores_, add, player->getName() + "./." + level + "./." + multi_cast<std::string>(points));
+    void Highscore::storeScore(const std::string& level, int points, PlayerInfo* player){
+        if (points > this->getHighestScoreOfGame(level)) {
+            ModifyConfigValue(highscores_, add, player->getName() + "./." + level + "./." + multi_cast<std::string>(points));
+        }
     }
 }

code/trunk/src/orxonox/Highscore.h

@@ -18 +18 @@
         void setConfigValues(); // Inherited function
 
-        void storeHighscore(const std::string& level, int points, PlayerInfo* player);
+        void storeScore(const std::string& level, int points, PlayerInfo* player);
         int getHighestScoreOfGame(const std::string& game);