Changeset 7121

Timestamp:

Jun 7, 2010, 1:06:35 AM (14 years ago)

Author:

youngk

Message:

Updated the MoodManager validation process to be more generic, however I ran into a little trouble. Please review the changes and consult my notes.

Location:

code/branches/presentation3/src/orxonox

Files:

• MoodManager.cc (modified) (3 diffs)
• MoodManager.h (modified) (3 diffs)

code/branches/presentation3/src/orxonox/MoodManager.cc

@@ -32 +32 @@
 #include "core/CoreIncludes.h"
 #include "core/ScopedSingletonManager.h"
+#include "core/Resource.h"
 
 namespace orxonox
@@ -37 +38 @@
     ManageScopedSingleton(MoodManager, ScopeID::Root, false);
 
+    // Note: I'm (Kevin Young) not entirely sure whether that's good code style:
+    const std::string MoodManager::defaultMood_ = "default";
+
     MoodManager::MoodManager()
     {
         RegisterRootObject(MoodManager);
         this->setConfigValues();
+        
+        // Checking for the existence of the folder for the default mood
+        /* Note: Currently Resource::exists(path) will always return false when the path field points to a folder.
+           MoodManager::checkMoodValidity() performs the same check. Please help.
+        */
+        const std::string& path = "ambient/" + MoodManager::defaultMood_ + '/';
+        if (!Resource::exists(path))
+        {
+            // TODO: Non-fatal error handling (non-critical resource missing)
+            COUT(2) << "Mood Warning: Folder for default mood (" << MoodManager::defaultMood_ << ") does not exist!" << std::endl;
+        }
     }
 
     void MoodManager::setConfigValues()
     {
-        SetConfigValue(mood_, "default")
+        SetConfigValue(mood_, MoodManager::defaultMood_)
             .description("Sets the mood for the current level.")
             .callback(this, &MoodManager::checkMoodValidity);
     }
 
-    /** Sets the mood
-    @note
-        TODO: Inform dependent classes of mood change
-    */
+    /** Set a new mood
+     */
     void MoodManager::setMood(const std::string& mood)
     {
@@ -61 +74 @@
     void MoodManager::checkMoodValidity()
     {
-        // TODO: Insert new moods here & make this generic
-        if (mood_ != "default" && mood_ != "dnb")
+        //  Generic mood validation
+        const std::string& path = "ambient/" + mood_ + '/';
+        if (!Resource::exists(path))
         {
+            COUT(3) << "Mood " << mood_ << " does not exist. Will not change." << std::endl;
             ResetConfigValue(mood_);
         }

code/branches/presentation3/src/orxonox/MoodManager.h

@@ -38 +38 @@
 namespace orxonox
 {
+    /*
+    @brief
+        The MoodListener class is aware of a change in themes and directs that info to dependent classes.
+    */
     class _OrxonoxExport MoodListener : virtual public OrxonoxClass
     {
@@ -55 +59 @@
     };
 
+    /*
+    @brief
+        The MoodManager class serves to allow for different musical themes in the game.
+    */
     class _OrxonoxExport MoodManager : public Singleton<MoodManager>, public OrxonoxClass
     {
@@ -74 +82 @@
             // config values
             std::string mood_;
+            static const std::string defaultMood_;
 
             static MoodManager* singletonPtr_s;