Changeset 1857

Timestamp:

Sep 29, 2008, 9:25:27 AM (16 years ago)

Author:

rgrieder

Message:

Added comments to the TutorialShip

Location:

code/branches/orxonox_tutorial/src/orxonox/objects

Files:

• TutorialShip.cc (modified) (6 diffs)
• TutorialShip.h (modified) (1 diff)

code/branches/orxonox_tutorial/src/orxonox/objects/TutorialShip.cc

@@ -29 +29 @@
 // for precompiled header files. Has to be first!
 #include "OrxonoxStableHeaders.h"
-// always include this class's header file first so that it compiles on its own too.
+// always include this class's header file first so that
+// it compiles on its own too.
 #include "TutorialShip.h"
 
 // Additional includes
-#include <OgreSceneNode.h>
 #include "util/Convert.h"
 #include "util/Debug.h"
@@ -45 +45 @@
 namespace orxonox
 {
-    SetConsoleCommand(TutorialShip, fire, true).keybindMode(KeybindMode::OnHold);
+    // Specify a console command that can be used in
+    // the shell or as key binding.
+    SetConsoleCommand(TutorialShip, fire, true)
+        .keybindMode(KeybindMode::OnHold);
 
+    // Make sure we can create an object of this class by XML
     CreateFactory(TutorialShip);
 
+    // Constructor
     TutorialShip::TutorialShip()
     {
@@ -60 +65 @@
     }
 
-    bool TutorialShip::create()
-    {
-        return true;
-    }
-
+    // Destructor
     TutorialShip::~TutorialShip()
     {
     }
-    
+
+    // Sets the configurable member variables.
+    // They can be found later in orxonox.ini directly.
     void TutorialShip::setConfigValues()
     {
-        SetConfigValue(reloadTime_, 0.125).description("The reload time of the weapon in seconds");
+        SetConfigValue(reloadTime_, 0.125)
+            .description("The reload time of the weapon in seconds");
     }
-
-    //void TutorialShip::registerAllVariables()
-    //{
-    //}
     
-    /**
-        @brief XML loading and saving.
-        @param xmlelement The XML-element
-        @param loading Loading (true) or saving (false)
-        @return The XML-element
-    */
+    // Called when loading an object of this class with XML
+    // You don't have to know what exactly xmlelement is.
+    // And mode is not important yet (load/save).
     void TutorialShip::XMLPort(Element& xmlelement, XMLPort::Mode mode)
     {
-        XMLPortParam(TutorialShip, "specialEffects", setSpecialEffects, hasSpecialEffects, xmlelement, mode);
+        // Load our parameter "specialEffects". Case sensitive!
+        XMLPortParam(TutorialShip, "specialEffects", setSpecialEffects,
+            hasSpecialEffects, xmlelement, mode);
 
-        // Calls SpaceShip::XMLPort
+        // Calls SpaceShip::XMLPort so that the SpaceShip XML parameters
+        // are loaded too.
         SUPER(TutorialShip, XMLPort, xmlelement, mode);
     }
 
+    // XML save function. Also used by back end class SpaceShip
+    // to show or hide the special effects.
     bool TutorialShip::hasSpecialEffects()
     {
@@ -97 +99 @@
     }
 
+    // XML load function. Called by the XML macro above.
     void TutorialShip::setSpecialEffects(bool value)
     {
@@ -102 +105 @@
     }
 
-    /**
-        @brief Returns the weapon reload time. Used virtually by the base class.
-    */
+    // virtual function used by back end class SpaceShip.
     float TutorialShip::getReloadTime()
     {
@@ -110 +111 @@
     }
 
-    
+    // run time update method. Gets called every frame with the delta time that
+    // has passed since the last frame.
     void TutorialShip::tick(float dt)
     {
+        // Also call the tick() method of the base clas.
         SUPER(TutorialShip, tick, dt);
     }
 
+    // Fire a projectile. Delegated to the back end class SpaceShip.
+    // Function content is insignificant for the tutorial.
     void TutorialShip::fire()
     {

code/branches/orxonox_tutorial/src/orxonox/objects/TutorialShip.h

@@ -49 +49 @@
             ~TutorialShip();
 
-            bool create();
             //void registerAllVariables();
             void setConfigValues();