Changeset 5902 for code/branches/pickup/src/orxonox/pickup/DroppedItem.h

Timestamp:

Oct 7, 2009, 2:20:43 PM (15 years ago)

Author:

dafrick

Message:

Added comments and a lot of TODO's, many of which probably are obsolete by now, so don't mind them, they're just to help me remember, what I thought at the time I went through that particular code.

File:

• code/branches/pickup/src/orxonox/pickup/DroppedItem.h (modified) (4 diffs)

code/branches/pickup/src/orxonox/pickup/DroppedItem.h

@@ -49 +49 @@
         virtual ~DroppedItem();
 
+        //TODO: Comment.
+        //DroppedItem -> Item with no owner, alone in space?
+        //Would be much nicer if it would be triggered by a standard issue DistanceTrigger.
+        //Where is this created? I see no XMLPort.
+        //Where is the item for this created? What happens if more than one pawn triggers this?
+        //Add more than just one items, or even better create the ability to add a Collection.? Rename to ...?
+
         void tick(float dt);
         void trigger(Pawn* pawn);
@@ -55 +62 @@
         static DroppedItem* createDefaultDrop(BaseItem* item, Pawn* pawn, const ColourValue& flareColour = ColourValue(0.5f, 1.0f, 0.3f), float timeToLive = 0);
 
-        void createTimer();
-        void timerCallback();
+        void createTimer(); //TODO: Can this be made private, too?
+        void timerCallback(); //TODO: This should really be private.
 
         inline float getTriggerDistance() const
@@ -65 +72 @@
         inline BaseItem* getItem() const
             { return this->item_; }
+        //TODO: Needs to be public?
         inline void setItem(BaseItem* item)
             { this->item_ = item; }
@@ -70 +78 @@
         inline float getTimeToLive() const
             { return this->timeToLive_; }
+        //TODO: Needs to be public?
         inline void setTimeToLive(float time)
             { this->timeToLive_ = time; }