Changeset 6490

Timestamp:

Mar 8, 2010, 9:39:47 AM (14 years ago)

Author:

dafrick

Message:

Added target to HealthPickup, which I had forgotten.

Location:

code/branches/pickup3/src

Files:

• modules/pickup/items/HealthPickup.cc (modified) (2 diffs)
• orxonox/interfaces/Pickupable.cc (modified) (2 diffs)
• orxonox/interfaces/Pickupable.h (modified) (1 diff)

code/branches/pickup3/src/modules/pickup/items/HealthPickup.cc

@@ -72 +72 @@
     */
     void HealthPickup::initialize(void)
-    {
-        RegisterObject(HealthPickup);
-        
+    {        
         this->health_ = 0;
         this->healthRate_ = 0;
@@ -80 +78 @@
         this->maxHealthSave_ = 0;
         this->maxHealthOverwrite_ = 0;
+        
+        this->addTarget(ClassIdentifier<Pawn>::getIdentifier());
     }
     

code/branches/pickup3/src/orxonox/interfaces/Pickupable.cc

@@ -104 +104 @@
     bool Pickupable::isTarget(const PickupCarrier* carrier) const
     {
-        Identifier* identifier = carrier->getIdentifier();
+        return this->isTarget(carrier->getIdentifier());
+    }
+    
+    /**
+    @brief
+        Get whether a given class, represented by the input Identifier, is a target of this pickup.
+    @param target
+        The Identifier of which it has to be determinde whether it is a target of this pickup.
+    @return
+        Returns true if the given Identifier is a target.
+    */
+    bool Pickupable::isTarget(Identifier* target) const
+    {
         //! Iterate through all targets of this Pickupable.
         for(std::list<Identifier*>::const_iterator it = this->targets_.begin(); it != this->targets_.end(); it++)
         {
-            if(identifier->isA(*it))
+            if(target->isA(*it))
                 return true;
         }
         return false;
     }
-    
+        
     /**
     @brief
@@ -124 +136 @@
     bool Pickupable::addTarget(PickupCarrier* target)
     {
+        return this->addTarget(target->getIdentifier());
+    }
+    
+    /**
+    @brief
+        Add a class, representetd by the input Identifier, as target of this pickup.
+    @param target
+        The Identifier to be added.
+    @return
+        Returns true if the target was added, false if not.
+    */
+    bool Pickupable::addTarget(Identifier* target)
+    {
         if(this->isTarget(target)) //!< If the input target is already present in the list of targets.
             return false;
         
-        COUT(4) << "Target (&" << target << ") added to Pickupable (&" << this << ")." << std::endl;
-        this->targets_.push_back(target->getIdentifier());
+        COUT(4) << "Target " << target->getName() << " added to Pickupable (&" << this << ")." << std::endl;
+        this->targets_.push_back(target);
         return true;
     }

code/branches/pickup3/src/orxonox/interfaces/Pickupable.h

@@ -83 +83 @@
             
             bool isTarget(const PickupCarrier* carrier) const; //!< Get whether the given PickupCarrier is a target of this pickup.
+            bool isTarget(Identifier* identifier) const; //!< Get whether a given class, represented by the input Identifier, is a target of this pickup.
             bool addTarget(PickupCarrier* target); //!< Add a PickupCarrier as target of this pickup.
+            bool addTarget(Identifier* identifier); //!< Add a class, representetd by the input Identifier, as target of this pickup.
             
             /**