Changeset 7756 in orxonox.OLD for trunk/src/lib

Timestamp:

May 21, 2006, 6:42:09 PM (19 years ago)

Author:

bensch

Message:

orxonox/trunk: Multiple Event Subscription Method
@patrick, do you approve of this, or do you think, that it is too much??

This is something i proposed some time ago, and it is nice, because one can overload the Commandos

Location:

trunk/src/lib

Files:

• event/event_handler.cc (modified) (14 diffs)
• event/event_handler.h (modified) (3 diffs)
• graphics/text_engine/multi_line_text.cc (modified) (1 diff)

trunk/src/lib/event/event_handler.cc

@@ -26 +26 @@
 #include "class_list.h"
 
-using namespace std;
-
+#include <algorithm>
 
 /**
@@ -43 +42 @@
 
   /* now initialize them all to zero */
-  this->flush(ES_ALL);
   this->withUNICODE(false);
   this->grabEvents(false);
 
   this->state = ES_GAME;
-  this->keyMapper = NULL;
   this->eventsGrabbed = false;
 }
@@ -69 +66 @@
     for(int j = 0; j < EV_NUMBER; ++j)
     {
-      if( this->listeners[i][j] != NULL)
+      if(!this->listeners[i][j].empty())
       {
         PRINTF(2)("forgot to unsubscribe an EventListener!\n");// %s!\n", this->listeners[i][j]->getName());
@@ -75 +72 @@
     }
   }
-  delete this->keyMapper;
-
   SDL_QuitSubSystem(SDL_INIT_JOYSTICK);
 
@@ -90 +85 @@
 void EventHandler::init()
 {
-  if (this->keyMapper == NULL)
-  {
-    this->keyMapper = new KeyMapper();
-    this->keyMapper->loadKeyBindings();
-  }
+  this->keyMapper.loadKeyBindings();
 }
 
@@ -138 +129 @@
 
 /**
- *  subscribe to an event
+ * @brief subscribe to an event
  * @param el: the event listener that wants to subscribe itself, the listener that will be called when the evetn occures
  * @param state: for which the listener wants to receive events
@@ -153 +144 @@
   {
     for(unsigned int i = 0; i < ES_NUMBER; i++)
-      if( likely(this->listeners[i][eventType] == NULL))
-        this->listeners[i][eventType] = el;
-      else
-        PRINTF(2)("%s of class %s tried to subscribe to event %i @ state %i but this event has already been subscribed\n", el->getName(), el->getClassName(), eventType, state);
-  }
-  else
-    if( likely(this->listeners[state][eventType] == NULL))
-    {
-      this->listeners[state][eventType] = el;
-    }
-    else
-      PRINTF(2)("% of class %s tried to subscribe to event %i @ state %i but this event has already been subscribed\n", el->getName(), el->getClassName(), eventType, state);
-}
-
-
-/**
- *  unsubscribe from the EventHandler
+    {
+      if( !likely(this->listeners[i][eventType].empty()))
+      {
+        PRINTF(2)("'%s' of class '%s' tried to subscribe to event %i @ state %i but this event has already been subscribed\n", el->getName(), el->getClassName(), eventType, state);
+      }
+      this->listeners[i][eventType].push_back(el);
+    }
+  }
+  else
+  {
+    if( likely(!this->listeners[state][eventType].empty()))
+    {
+      PRINTF(2)("%s of class %s tried to subscribe to event %i @ state %i but this event has already been subscribed\n", el->getName(), el->getClassName(), eventType, state);
+    }
+    this->listeners[state][eventType].push_back(el);
+  }
+}
+
+
+/**
+ * @brief unsubscribe from the EventHandler
  * @param state: the stat in which it has been subscribed
  * @param eventType: the event, that shall be unsubscribed
@@ -181 +176 @@
   if (state == ES_ALL)
     for (unsigned int i = 0; i < ES_NUMBER; i++)
-      this->listeners[i][eventType] = NULL;
-  else
-    this->listeners[state][eventType] = NULL;
+      this->listeners[i][eventType].clear();
+  else
+    this->listeners[state][eventType].clear();
 }
 
@@ -203 +198 @@
       for(unsigned int j = 0; j < EV_NUMBER; j++)
       {
-        if( this->listeners[i][j] == el )
-          this->listeners[i][j] = NULL;
+        std::vector<EventListener*>::iterator deller = std::find (this->listeners[i][j].begin(), this->listeners[i][j].end(), el);
+        if( deller != this->listeners[i][j].end())
+          this->listeners[i][j].erase(deller);
       }
     }
@@ -212 +208 @@
     for(int j = 0; j < EV_NUMBER; j++)
     {
-      if( this->listeners[state][j] == el )
-        this->listeners[state][j] = NULL;
-    }
-  }
-}
+      std::vector<EventListener*>::iterator deller =  std::find (this->listeners[state][j].begin(), this->listeners[state][j].end(), el);
+      if( deller != this->listeners[state][j].end())
+        this->listeners[state][j].erase(deller);
+    }
+  }
+}
+
+bool EventHandler::isSubscribed(elState state, int eventType)
+{
+  return(listeners[state][eventType].empty()) ? false : true;
+};
+
 
 
@@ -231 +234 @@
       for(int j = 0; j < EV_NUMBER; ++j)
       {
-        this->listeners[i][j] = NULL;
+        this->listeners[i][j].clear();
       }
     }
@@ -239 +242 @@
     for(int j = 0; j < EV_NUMBER; ++j)
     {
-      this->listeners[state][j] = NULL;
+      this->listeners[state][j].clear();
     }
   }
@@ -344 +347 @@
     PRINT(4)("=  Got Event nr %i, for state %i", ev.type, this->state);
 
-    listener = this->listeners[this->state][ev.type];
-    if( listener != NULL)
+
+    for (unsigned int i = 0; i < this->listeners[this->state][ev.type].size(); i++)
     {
       PRINT(4)("=  Event dispatcher msg: This event has been consumed\n");
       PRINT(4)("=======================================================\n");
-      listener->process(ev);
-    }
-    else
-    {
-      PRINT(4)("=  Event dispatcher msg: This event has NOT been consumed\n");
-      PRINT(4)("=======================================================\n");
-    }
+      listeners[this->state][ev.type][i]->process(ev);
+    }
+    /*    else
+        {
+          PRINT(4)("=  Event dispatcher msg: This event has NOT been consumed\n");
+          PRINT(4)("=======================================================\n");
+        }*/
   }
 }
@@ -393 +396 @@
   for(int i = 0; i < ES_NUMBER; ++i)
     for(int j = 0; j < EV_NUMBER; ++j)
-      if( this->listeners[i][j] != NULL)
-        PRINT(0)("Event %d of State %d subscribed to %s (%p)\n", j, i, this->listeners[i][j]->getName(), this->listeners[i][j]);
+      for (unsigned int evl = 0; evl < this->listeners[i][j].size(); evl++)
+        PRINT(0)("Event %d of State %d subscribed to %s (%p)\n", j, i, this->listeners[i][j][evl]->getName(), this->listeners[i][j][evl]);
   PRINT(0)("============================EH=\n");
 }

trunk/src/lib/event/event_handler.h

@@ -12 +12 @@
 #include "event_def.h"
 #include <stack>
+#include <vector>
 
 // FORWARD DECLARATION
@@ -38 +39 @@
   void flush(elState state);
   /** @returns true, if the @param state has @param eventType subscribed?? */
-  inline bool isSubscribed(elState state, int eventType) { return(listeners[state][eventType] == NULL)?false:true; };
+  bool isSubscribed(elState state, int eventType);
 
 
@@ -54 +55 @@
 
  private:
-  static EventHandler*       singletonRef;                    //!< the singleton reference
+  static EventHandler*         singletonRef;                    //!< the singleton reference
 
-  EventListener*             listeners[ES_NUMBER][EV_NUMBER]; //!< a list of registered listeners.
-  elState                    state;                           //!< the state of the event handlder.
-  std::stack<short>          stateStack;                      //!< a stack for the States we are in.
-  KeyMapper*                 keyMapper;                       //!< reference to the key mapper.
+  std::vector<EventListener*>  listeners[ES_NUMBER][EV_NUMBER]; //!< a list of registered listeners.
+  elState                      state;                           //!< the state of the event handlder.
+  std::stack<short>            stateStack;                      //!< a stack for the States we are in.
+  KeyMapper                    keyMapper;                       //!< reference to the key mapper.
 
-  bool                       bUNICODE;                        //!< If unicode should be enabled.
-  bool                       eventsGrabbed;                   //!< If the events should be grabbed
+  bool                         bUNICODE;                        //!< If unicode should be enabled.
+  bool                         eventsGrabbed;                   //!< If the events should be grabbed
 };
 

trunk/src/lib/graphics/text_engine/multi_line_text.cc

@@ -136 +136 @@
     if (width > maxWidth || this->getText()[i] == '\n')
     {
-      this->lineEnds.push_back(i);
-      width = 0.0f;
+      if (likely(i > 0))
+      {
+        this->lineEnds.push_back( i -1 );
+        width = this->getFont()->getGlyphArray()[this->getText()[i-1]]->advance;
+      }
+      else
+        width = 0.0f;
     }
+
     // Advance the Text.
     if(this->getFont()->getGlyphArray()[this->getText()[i]] != NULL)