Changeset 6731

Timestamp:

Apr 15, 2010, 2:47:39 PM (15 years ago)

Author:

dafrick

Message:

PickupCollection is broken for now because to make it work drastic changes would have to be made. I wil fix it at a later date, though.

Location:

code/trunk

Files:

• data/levels/pickup.oxw (modified) (1 diff)
• src/modules/pickup/PickupCollection.cc (modified) (1 diff)
• src/modules/pickup/PickupCollection.h (modified) (1 diff)
• src/orxonox/interfaces/Pickupable.cc (modified) (2 diffs)
• src/orxonox/interfaces/Pickupable.h (modified) (1 diff)

code/trunk/data/levels/pickup.oxw

@@ -114 +114 @@
     <!-- Pickup Collection pickups -->
     
-    <PickupSpawner position="0,-25,-100" triggerDistance="10" respawnTime="30" maxSpawnedItems="10">
+    <!--PickupSpawner position="0,-25,-100" triggerDistance="10" respawnTime="30" maxSpawnedItems="10">
         <pickup>
             <PickupCollection template=triplehealthspeedinvisibilitypickup />
         </pickup>
-    </PickupSpawner>
-    
-    <PickupSpawner position="0,-50,-100" triggerDistance="10" respawnTime="30" maxSpawnedItems="10">
-        <pickup>
-            <HealthPickup activationType=immediate durationType=continuous healthRate=10 health=100 />
-        </pickup>
-    </PickupSpawner>
+    </PickupSpawner-->
     
   </Scene>

code/trunk/src/modules/pickup/PickupCollection.cc

@@ -176 +176 @@
         Returns true if the PickupCarrier identified by the input PickupIdentififer it is a target of this PickupCollection, false if not.
     */
-    bool PickupCollection::isTarget(Identifier* identifier) const
+    bool PickupCollection::isTarget(PickupCarrier* carrier) const
     {
         for(std::vector<WeakPtr<Pickupable> >::const_iterator it = this->pickups_.begin(); it != this->pickups_.end(); it++)
         {
-            if(!(*it).get()->isTarget(identifier))
+            if(!carrier->isTarget((*it).get()))
                 return false;
         }

code/trunk/src/modules/pickup/PickupCollection.h

@@ -67 +67 @@
             virtual void clone(OrxonoxClass*& item); //!< Creates a duplicate of the input OrxonoxClass.
             
-            virtual bool isTarget(Identifier* identifier) const; //!< Get whether a given class, represented by the input Identifier, is a target of this PickupCollection.
+            virtual bool isTarget(PickupCarrier* carrier) const; //!< Get whether a given class, represented by the input Identifier, is a target of this PickupCollection.
             
             virtual const PickupIdentifier* getPickupIdentifier(void); //!< Get the PickupIdentifier of this PickupCollection.

code/trunk/src/orxonox/interfaces/Pickupable.cc

@@ -104 +104 @@
     bool Pickupable::isTarget(const PickupCarrier* carrier) const
     {
+        if(carrier == NULL)
+            return false;
         return this->isTarget(carrier->getIdentifier());
     }
@@ -109 +111 @@
     /**
     @brief
-        Get whether a given class, represented by the input Identifier, is a target of this Pickupable.
-    @param target
-        The Identifier of which it has to be determinde whether it is a target of this Pickupable.
-    @return
-        Returns true if the given Identifier is a target.
-    */
-    bool Pickupable::isTarget(Identifier* target) const
+        Get whether the given Identififer is a target of this Pickupable.
+    @param identifier
+        The PickupCarrier of which it has to be determinde whether it is a target of this Pickupable.
+    @return
+        Returns true if the given PickupCarrier is a target.
+    */
+    bool Pickupable::isTarget(const Identifier* identifier) const
     {
         //! Iterate through all targets of this Pickupable.
         for(std::list<Identifier*>::const_iterator it = this->targets_.begin(); it != this->targets_.end(); it++)
         {
-            if(target->isA(*it))
+            if(identifier->isA(*it))
                 return true;
         }

code/trunk/src/orxonox/interfaces/Pickupable.h

@@ -98 +98 @@
             bool dropped(void); //!< Sets the Pickupable to not picked up or dropped.
             
-            bool isTarget(const PickupCarrier* carrier) const; //!< Get whether the given PickupCarrier is a target of this pickup.
-            virtual bool isTarget(Identifier* identifier) const; //!< Get whether a given class, represented by the input Identifier, is a target of this Pickupable.
+            virtual bool isTarget(const PickupCarrier* carrier) const; //!< Get whether the given PickupCarrier is a target of this pickup.
+            bool isTarget(const Identifier* identifier) const; //!< Get whether a given class, represented by the input Identifier, is a target of this Pickupable.
             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.