Changeset 2858 for code/branches/questsystem5/src/orxonox/overlays/notifications/NotificationQueue.cc

Timestamp:

Mar 26, 2009, 9:45:23 PM (15 years ago)

Author:

dafrick

Message:

Done some documenting, and resolved some (possible) issues. So the Notifications are about finished…

File:

• code/branches/questsystem5/src/orxonox/overlays/notifications/NotificationQueue.cc (modified) (6 diffs)

code/branches/questsystem5/src/orxonox/overlays/notifications/NotificationQueue.cc

@@ -27 +27 @@
  */
 
+/**
+    @file NotificationQueue.cc
+    @brief Implementation of the NotificationQueue class.
+*/
+
 #include "OrxonoxStableHeaders.h"
 #include "NotificationQueue.h"
@@ -33 +38 @@
 #include <OgreTextAreaOverlayElement.h>
 #include <list>
+#include <iostream>
+#include <sstream>
 
 #include "core/CoreIncludes.h"
@@ -155 +162 @@
         this->clear();
     
-        std::multimap<std::time_t,Notification*>* notifications = NotificationManager::getInstance().getNotifications(this, this->displayTime_);
-        
-        if(notifications == NULL)
+        std::multimap<std::time_t,Notification*>* notifications = new std::multimap<std::time_t,Notification*>;
+        if(!NotificationManager::getInstance().getNotifications(this, notifications, this->displayTime_)) //!< Get the Notifications sent in the interval form now to minus the display time.
+        {
+            COUT(1) << "NotificationQueue update failed due to undetermined cause." << std::endl;
             return;
-        
-        for(std::multimap<std::time_t,Notification*>::iterator it = notifications->begin(); it != notifications->end(); it++)
+        }
+        
+        if(notifications->empty())
+            return;
+        
+        for(std::multimap<std::time_t,Notification*>::iterator it = notifications->begin(); it != notifications->end(); it++) //!> Add all Notifications.
         {
             this->addNotification(it->second, it->first);
         }
+        
+        delete notifications;
         
         COUT(3) << "NotificationQueue updated." << std::endl;
@@ -343 +357 @@
     }
 
+    /**
+    @brief
+        Scrolls the NotificationQueue, meaning all NotificationOverlays are moved the input vector.
+    @param pos
+        The vector the NotificationQueue is scrolled.
+    */
     void NotificationQueue::scroll(const Vector2 pos)
     {
@@ -351 +371 @@
     }
 
-    void NotificationQueue::positionChanged()
+    /**
+    @brief
+        Aligns all the Notifications to the position of the NotificationQueue.
+    */
+    void NotificationQueue::positionChanged(void)
     {
         int counter = 0;
@@ -379 +403 @@
         std::string timeString = std::ctime(&time);
         timeString.erase(timeString.length()-1);
-        char buffer[64]; //TDO: Very un-nice.
-        std::sprintf(buffer,"%x",(unsigned long)notification); //TDO: Use other conversion to avoid 64bit problems.
-        std::string addressString = buffer;
+        std::ostringstream stream;
+        stream << (unsigned long)notification;
+        std::string addressString = stream.str() ;
         container->name = "NotificationOverlay(" + timeString + ")&" + addressString;