Changeset 5617

Timestamp:

Aug 11, 2009, 12:27:38 AM (15 years ago)

Author:

landauf

Message:

Reduced dependencies between orxonox and overlays: GUIOverlays send their owners to a map in the QuestManager instead of the QuestManager asking the GUIOverlays.

Location:

code/branches/libraries/src/orxonox

Files:

• objects/quest/QuestManager.cc (modified) (11 diffs)
• objects/quest/QuestManager.h (modified) (2 diffs)
• overlays/GUIOverlay.cc (modified) (2 diffs)
• overlays/GUIOverlay.h (modified) (1 diff)

code/branches/libraries/src/orxonox/objects/quest/QuestManager.cc

@@ -38 +38 @@
 
 #include "objects/infos/PlayerInfo.h"
-#include "objects/infos/PlayerInfo.h"
-#include "overlays/GUIOverlay.h"
 #include "Quest.h"
 #include "QuestHint.h"
@@ -203 +201 @@
     /**
     @brief
-        
+
     @param name
     @return
@@ -209 +207 @@
     QuestContainer* QuestManager::getQuestTree(std::string & name)
     {
-        GUIOverlay* gui = NULL;
-        for (ObjectList<GUIOverlay>::iterator it = ObjectList<GUIOverlay>::begin(); it != ObjectList<GUIOverlay>::end(); ++it)
-            if (it->getGUIName() == name)
-                gui = *it;
-
-        PlayerInfo* player;
-        if(gui == NULL)
-        {
-            COUT(1) << "Error: No GUIOverlay with the given name '" << name << "' present." << std::endl;
+        PlayerInfo* player = this->players_[name];
+        if(player == NULL)
+        {
+            COUT(1) << "Error: GUIOverlay with name '" << name << "' has no player." << std::endl;
             return NULL;
         }
-        BaseObject* obj = gui->getOwner();
-        if(obj == NULL)
-        {
-            COUT(1) << "Error: GUIOverlay has no owner. " << std::endl;
-            return NULL;
-        }
-        player = orxonox_cast<PlayerInfo*>(obj);
-    
+
         QuestContainer* root = NULL;
         QuestContainer* current = NULL;
-        
+
         std::list<Quest*>* rootQuests = new std::list<Quest*>();
         getRootQuests(player, *rootQuests);
-        
+
         for(std::list<Quest*>::iterator it = rootQuests->begin(); it != rootQuests->end(); it++)
         {
@@ -246 +232 @@
                 current->next = container;
             }
-            
+
             current = container;
 
@@ -260 +246 @@
     /**
     @brief
-        
+
     @param player
     @param list
@@ -279 +265 @@
     /**
     @brief
-        
+
     @param quest
     @param player
@@ -310 +296 @@
             COUT(1) << "An error occurred. A Quest of un-specified status wanted to be displayed." << std::endl;
         }
-        
+
         std::list<Quest*> quests = quest->getSubQuestList();
         QuestContainer* current = NULL;
@@ -329 +315 @@
                     current->next = subContainer;
                 }
-                
+
                 current = subContainer;
             }
@@ -336 +322 @@
             current->next = NULL;
         container->subQuests = first;
-        
+
         return container;
     }
@@ -342 +328 @@
     /**
     @brief
-        
+
     @param quest
     @param player
@@ -368 +354 @@
                     current->next = hint;
                 }
-                
+
                 current = hint;
             }

code/branches/libraries/src/orxonox/objects/quest/QuestManager.h

@@ -92 +92 @@
             QuestContainer* getQuestTree(std::string & name); // tolua_export
 
+            inline void setPlayer(const std::string& guiname, PlayerInfo* player)
+                { this->players_[guiname] = player; }
+            inline PlayerInfo* getPlayer(const std::string& guiname) const
+                { std::map<std::string, PlayerInfo*>::const_iterator it = this->players_.find(guiname); return (it != this->players_.end()) ? it->second : 0; }
+
         private:
             static QuestManager* singletonPtr_s;
@@ -97 +102 @@
             std::map<std::string, Quest*> questMap_; //!< All Quests registered by their id's.
             std::map<std::string, QuestHint*> hintMap_; //!< All QuestHints registered by their id's.
+            std::map<std::string, PlayerInfo*> players_; //!< Stores the player (owner) for each gui
 
             void getRootQuests(const PlayerInfo* player, std::list<Quest*> & list);

code/branches/libraries/src/orxonox/overlays/GUIOverlay.cc

@@ -36 +36 @@
 #include "core/GUIManager.h"
 #include "core/XMLPort.h"
+#include "objects/quest/QuestManager.h"
+#include "objects/infos/PlayerInfo.h"
 
 namespace orxonox
@@ -79 +81 @@
     }
 
+    void GUIOverlay::setGUIName(const std::string& name)
+    {
+        this->guiName_ = name;
+        QuestManager::getInstance().setPlayer(name, orxonox_cast<PlayerInfo*>(this->getOwner()));
+    }
 }

code/branches/libraries/src/orxonox/overlays/GUIOverlay.h

@@ -46 +46 @@
             virtual void XMLPort(Element& xmlElement, XMLPort::Mode mode);
 
-            inline void setGUIName(const std::string& name) { this->guiName_ = name; }
+            void setGUIName(const std::string& name);
             inline const std::string& getGUIName() const { return this->guiName_; }