Changeset 7866 in orxonox.OLD

Timestamp:

May 26, 2006, 1:11:10 PM (18 years ago)

Author:

bensch

Message:

Events are subscribed at the EventListener, and not the EventHandler

Location:

branches/gui/src

Files:

• lib/event/event_def.h (modified) (1 diff)
• lib/event/event_handler.cc (modified) (1 diff)
• lib/event/event_handler.h (modified) (1 diff)
• lib/event/event_listener.cc (modified) (3 diffs)
• lib/event/event_listener.h (modified) (2 diffs)
• lib/gui/gl_gui/glgui_handler.cc (modified) (1 diff)
• lib/gui/gl_gui/glgui_widget.h (modified) (2 diffs)
• lib/shell/shell.cc (modified) (1 diff)
• lib/shell/shell_input.cc (modified) (2 diffs)
• story_entities/movie_loader.cc (modified) (1 diff)
• story_entities/simple_game_menu.cc (modified) (4 diffs)
• world_entities/camera.cc (modified) (1 diff)
• world_entities/playable.cc (modified) (5 diffs)
• world_entities/player.cc (modified) (1 diff)
• world_entities/weapons/crosshair.cc (modified) (1 diff)

branches/gui/src/lib/event/event_def.h

@@ -12 +12 @@
 #include "SDL/SDL_keysym.h"
 #endif
-#include "stdincl.h"
 
 

branches/gui/src/lib/event/event_handler.cc

@@ -171 +171 @@
    unsubscribe(EventListener* el, elState state) function
 */
-void EventHandler::unsubscribe(elState state, int eventType)
+void EventHandler::unsubscribe(EventListener* el, elState state, int eventType)
 {
   PRINTF(4)("Unsubscribing event type nr: %i\n", eventType);
   if (state == ES_ALL)
     for (unsigned int i = 0; i < ES_NUMBER; i++)
-      this->listeners[i][eventType].clear();
-  else
-    this->listeners[state][eventType].clear();
-}
-
-
-/**
- * unsubscribe all events from a specific listener
+    {
+      std::vector<EventListener*>::iterator listener =
+         std::find(this->listeners[i][eventType].begin(),
+                   this->listeners[i][eventType].end(),
+                   el);
+      if (listener != this->listeners[i][eventType].end())
+        this->listeners[i][eventType].erase(listener);
+    }
+  else
+  {
+    std::vector<EventListener*>::iterator listener =
+        std::find(this->listeners[state][eventType].begin(),
+                  this->listeners[state][eventType].end(),
+                  el);
+    if (listener != this->listeners[state][eventType].end())
+      this->listeners[state][eventType].erase(listener);
+  }
+}
+
+
+/**
+ * @brief unsubscribe all events from a specific listener
  * @param el: the listener that wants to unsubscribe itself
  * @param state: the state in which the events shall be unsubscribed
-
-*/
+ */
 void EventHandler::unsubscribe(EventListener* el, elState state)
 {

branches/gui/src/lib/event/event_handler.h

@@ -35 +35 @@
 
   void subscribe(EventListener* el, elState state, int eventType);
-  void unsubscribe(elState state, int eventType);
+  void unsubscribe(EventListener* el, elState state, int eventType);
   void unsubscribe(EventListener* el, elState state = ES_ALL);
   void flush(elState state);

branches/gui/src/lib/event/event_listener.cc

@@ -23 +23 @@
 
 /**
- *  standard constructor
-*/
+ * @brief standard constructor
+ */
 EventListener::EventListener ()
 {
@@ -32 +32 @@
 
 /**
- *  standard deconstructor
-*/
+ * @brief standard deconstructor
+ *
+ * Unsubscribes all Subscribed Events, of this EventListener.
+ */
 EventListener::~EventListener ()
 {
@@ -39 +41 @@
   EventHandler::getInstance()->unsubscribe(this, ES_ALL);
 }
+
+bool EventListener::isEventSubscribed(elState state, int eventType) const
+{
+  return EventHandler::getInstance()->isSubscribed(state, eventType);
+}
+
+
+/**
+ * @brief Subscribes an Events to this EventListener.
+ * @param state the state to subscribe to.
+ * @param eventType the Type of Event to subscribe.
+ */
+void EventListener::subscribeEvent(elState state, int eventType)
+{
+  EventHandler::getInstance()->subscribe(this, state, eventType);
+}
+
+
+/**
+ * @brief Unubscribes an Event from this EventListener.
+ * @param state the state to unsubscribe from.
+ * @param eventType the Type of Event to unsubscribe.
+ */
+void EventListener::unsubscribeEvent(elState state, int eventType)
+{
+  EventHandler::getInstance()->unsubscribe(this, state, eventType);
+}
+
+void EventListener::unsubscribeEvents(elState state)
+{
+  EventHandler::getInstance()->unsubscribe(this, state);
+}
+

branches/gui/src/lib/event/event_listener.h

@@ -9 +9 @@
 #include "base_object.h"
 #include "event.h"
+#include "event_def.h"
 
 //! A class for event listener
@@ -17 +18 @@
   virtual ~EventListener();
 
+  bool isEventSubscribed(elState state, int eventType) const;
+
+  void subscribeEvent(elState state, int eventType);
+  void unsubscribeEvent(elState state, int eventType);
+  void unsubscribeEvents(elState state = ES_ALL);
+
   /**
-   * abstract function that processes events from the handler
+   * @brief abstract function that processes events from the handler
    * @param event: the event
    */
   virtual void process(const Event &event) = 0;
+
 };
 

branches/gui/src/lib/gui/gl_gui/glgui_handler.cc

@@ -31 +31 @@
     this->setClassID(CL_GLGUI_HANDLER, "GLGuiHandler");
     this->setName("GLGuiHandler");
+
+    //this->subscribeEvent()
 
   }

branches/gui/src/lib/gui/gl_gui/glgui_widget.h

@@ -56 +56 @@
     bool focusOverWidget(float x, float y);
 
-    // if something was clickt on the GUI-widget.
+
+    virtual void update() {};
+    virtual void draw() const;
+
+    Material& backMaterial() { return this->backMat; };
+    const Material& backMaterial() const { return this->backMat; };
+
+    Material& frontMaterial() { return this->frontMat; };
+    const Material& frontMaterial() const { return this->frontMat; };
+
+  protected:
+        // if something was clickt on the GUI-widget.
     virtual void clicked(const Event& event) {};
     virtual void released(const Event& event) {};
@@ -65 +76 @@
     virtual void destroyed() {};
 
-    virtual void update() {};
-    virtual void draw() const;
 
-    Material& backMaterial() { return this->backMat; };
-    Material& frontMaterial() { return this->frontMat; };
-
-  protected:
     inline void startDraw() const { glPushMatrix(); glTranslatef(this->getAbsCoor2D().x, this->getAbsCoor2D().y, 0); };
     inline void endDraw() const { glPopMatrix(); };

branches/gui/src/lib/shell/shell.cc

@@ -65 +65 @@
 
     // EVENT-Handler subscription of '`' to all States.
-    EventHandler::getInstance()->subscribe(this, ES_ALL, SDLK_BACKQUOTE);
-    EventHandler::getInstance()->subscribe(this, ES_ALL, SDLK_F12);
-    EventHandler::getInstance()->subscribe(this, ES_SHELL, SDLK_PAGEUP);
-    EventHandler::getInstance()->subscribe(this, ES_SHELL, SDLK_PAGEDOWN);
-    EventHandler::getInstance()->subscribe(this, ES_SHELL, EV_VIDEO_RESIZE);
+    this->subscribeEvent(ES_ALL, SDLK_BACKQUOTE);
+    this->subscribeEvent(ES_ALL, SDLK_F12);
+    this->subscribeEvent(ES_SHELL, SDLK_PAGEUP);
+    this->subscribeEvent(ES_SHELL, SDLK_PAGEDOWN);
+    this->subscribeEvent(ES_SHELL, EV_VIDEO_RESIZE);
 
     // BUFFER

branches/gui/src/lib/shell/shell_input.cc

@@ -20 +20 @@
 #include "shell_command.h"
 #include "shell_command_class.h"
-#include "event_handler.h"
 
 #include "debug.h"
@@ -50 +49 @@
     this->setRepeatDelay(.3, .05);
 
-    // subscribe all keyboard commands to ES_SEHLL
-    EventHandler* evh = EventHandler::getInstance();
+    // subscribe all keyboard commands to ES_SHELL
     for (int i = 1; i < SDLK_LAST; i++)
     {
-      if (!evh->isSubscribed(ES_SHELL, i))
-        evh->subscribe(this, ES_SHELL, i);
+      if (!this->isEventSubscribed(ES_SHELL, i))
+        this->subscribeEvent(ES_SHELL, i);
     }
     // unsubscribe unused TODO improve.
-    evh->unsubscribe(ES_SHELL, SDLK_BACKQUOTE);
-    evh->unsubscribe(ES_SHELL, SDLK_F12);
-    evh->unsubscribe(ES_SHELL, SDLK_PAGEUP);
-    evh->unsubscribe(ES_SHELL, SDLK_PAGEDOWN);
+    this->unsubscribeEvent(ES_SHELL, SDLK_BACKQUOTE);
+    this->unsubscribeEvent(ES_SHELL, SDLK_F12);
+    this->unsubscribeEvent(ES_SHELL, SDLK_PAGEUP);
+    this->unsubscribeEvent(ES_SHELL, SDLK_PAGEDOWN);
 
   }

branches/gui/src/story_entities/movie_loader.cc

@@ -70 +70 @@
 ErrorMessage MovieLoader::unloadData()
 {
-  EventHandler::getInstance()->unsubscribe(this, ES_GAME);
+  this->unsubscribeEvents(ES_GAME);
 }
 

branches/gui/src/story_entities/simple_game_menu.cc

@@ -18 +18 @@
 
 #include "simple_game_menu.h"
+
+#include "event_handler.h"
 
 #include "state.h"
@@ -32 +34 @@
 #include "camera.h"
 
-#include "event_handler.h"
 #include "graphics_engine.h"
 #include "object_manager.h"
@@ -112 +113 @@
   GameWorld::init();
 
-  EventHandler::getInstance()->subscribe(this, ES_MENU, SDLK_UP);
-  EventHandler::getInstance()->subscribe(this, ES_MENU, SDLK_DOWN);
-  EventHandler::getInstance()->subscribe(this, ES_MENU, SDLK_RETURN);
-  EventHandler::getInstance()->subscribe(this, ES_MENU, SDLK_SPACE);
-  EventHandler::getInstance()->subscribe(this, ES_MENU, SDLK_ESCAPE);
+  this->subscribeEvent(ES_MENU, SDLK_UP);
+  this->subscribeEvent(ES_MENU, SDLK_DOWN);
+  this->subscribeEvent(ES_MENU, SDLK_RETURN);
+  this->subscribeEvent(ES_MENU, SDLK_SPACE);
+  this->subscribeEvent(ES_MENU, SDLK_ESCAPE);
 
   this->dataTank->localCamera->setRelCoor(this->cameraVector);
@@ -253 +254 @@
 ErrorMessage SimpleGameMenu::unloadData()
 {
-  EventHandler::getInstance()->unsubscribe(this, ES_MENU);
+  this->unsubscribeEvents(ES_MENU);
 
   std::vector<MenuLayer>::iterator mit;

branches/gui/src/world_entities/camera.cc

@@ -31 +31 @@
   this->target = new CameraTarget();
 
-  EventHandler::getInstance()->subscribe(this, ES_GAME, KeyMapper::PEV_VIEW0);
-  EventHandler::getInstance()->subscribe(this, ES_GAME, KeyMapper::PEV_VIEW1);
-  EventHandler::getInstance()->subscribe(this, ES_GAME, KeyMapper::PEV_VIEW2);
-  EventHandler::getInstance()->subscribe(this, ES_GAME, KeyMapper::PEV_VIEW3);
-  EventHandler::getInstance()->subscribe(this, ES_GAME, KeyMapper::PEV_VIEW4);
-  EventHandler::getInstance()->subscribe(this, ES_GAME, KeyMapper::PEV_VIEW5);
+  this->subscribeEvent(ES_GAME, KeyMapper::PEV_VIEW0);
+  this->subscribeEvent(ES_GAME, KeyMapper::PEV_VIEW1);
+  this->subscribeEvent(ES_GAME, KeyMapper::PEV_VIEW2);
+  this->subscribeEvent(ES_GAME, KeyMapper::PEV_VIEW3);
+  this->subscribeEvent(ES_GAME, KeyMapper::PEV_VIEW4);
+  this->subscribeEvent(ES_GAME, KeyMapper::PEV_VIEW5);
 
   this->setFovy(90);

branches/gui/src/world_entities/playable.cc

@@ -16 +16 @@
 
 #include "playable.h"
-#include "event_handler.h"
+
+#include "key_mapper.h"
 
 #include "player.h"
@@ -225 +226 @@
 
     // unsubscibe all events.
-    EventHandler* evh = EventHandler::getInstance();
     std::vector<int>::iterator ev;
     for (ev = this->events.begin(); ev != events.end(); ev++)
-      evh->unsubscribe( ES_GAME, (*ev));
+      player->unsubscribeEvent(ES_GAME, (*ev));
 
     // leave the entity
@@ -251 +251 @@
 
     /*EventHandler*/
-    EventHandler* evh = EventHandler::getInstance();
     std::vector<int>::iterator ev;
     for (ev = this->events.begin(); ev != events.end(); ev++)
-      evh->subscribe(player, ES_GAME, (*ev));
+      player->subscribeEvent(ES_GAME, (*ev));
 
     this->enter();
@@ -440 +439 @@
 
   if (this->currentPlayer != NULL)
-    EventHandler::getInstance()->subscribe(this->currentPlayer, ES_GAME, eventType);
+    this->currentPlayer->subscribeEvent(ES_GAME, eventType);
 }
 
@@ -453 +452 @@
 
   if (this->currentPlayer != NULL)
-    EventHandler::getInstance()->unsubscribe(ES_GAME, eventType);
+    this->currentPlayer->unsubscribeEvent(ES_GAME, eventType);
 }
 

branches/gui/src/world_entities/player.cc

@@ -40 +40 @@
   this->hud.show();
 
-  EventHandler::getInstance()->subscribe(this, ES_GAME, KeyMapper::PEV_CHANGE_SHIP);
+  this->subscribeEvent(ES_GAME, KeyMapper::PEV_CHANGE_SHIP);
 }
 

branches/gui/src/world_entities/weapons/crosshair.cc

@@ -65 +65 @@
   this->material = new Material;
 
-  //EventHandler::getInstance()->subscribe(this, ES_GAME, EV_MOUSE_MOTION);
+  //this->subscribeEvent(ES_GAME, EV_MOUSE_MOTION);
 
   // center the mouse on the screen, and also hide the cursors