Changeset 6675 for code/branches/pickup4/src/modules

Timestamp:

Apr 2, 2010, 5:35:14 PM (15 years ago)

Author:

dafrick

Message:

Lua bugs fixed.
Renamed name and description in PickupRepresentation to pickupName and pickupDescription to avoid problems.
More work on PickupInventory.

Location:

code/branches/pickup4/src/modules/pickup

Files:

• PickupManager.cc (modified) (4 diffs)
• PickupManager.h (modified) (1 diff)
• PickupRepresentation.cc (modified) (2 diffs)
• PickupRepresentation.h (modified) (3 diffs)

code/branches/pickup4/src/modules/pickup/PickupManager.cc

@@ -127 +127 @@
     }
     
-    unsigned int PickupManager::getNumCarrierChildren(PickupCarrier* carrier)
+    int PickupManager::getNumCarrierChildren(PickupCarrier* carrier)
     {
         if(carrier == NULL)
@@ -134 +134 @@
     }
             
-    PickupCarrier* PickupManager::getCarrierChild(unsigned int index, PickupCarrier* carrier)
+    PickupCarrier* PickupManager::getCarrierChild(int index, PickupCarrier* carrier)
     {
+        if(carrier == NULL)
+            return NULL;
         return carrier->getCarrierChild(index);
     }
     
-    PickupRepresentation* PickupManager::getPickupRepresentation(unsigned int index, PickupCarrier* carrier)
+    const std::string& PickupManager::getCarrierName(orxonox::PickupCarrier* carrier)
+    {
+        if(carrier == NULL)
+            return BLANKSTRING;
+        return carrier->getCarrierName();
+    }
+    
+    PickupRepresentation* PickupManager::getPickupRepresentation(int index, PickupCarrier* carrier)
     {
         Pickupable* pickup = carrier->getPickup(index);
@@ -149 +158 @@
     
 
-    unsigned int PickupManager::getNumPickups(PickupCarrier* carrier)
+    int PickupManager::getNumPickups(PickupCarrier* carrier)
     {
+        if(carrier == NULL)
+            return 0;
         return carrier->getNumPickups();
     }
     
-    void PickupManager::dropPickup(unsigned int index, PickupCarrier* carrier)
+    void PickupManager::dropPickup(int index, PickupCarrier* carrier)
     {
         Pickupable* pickup = carrier->getPickup(index);
@@ -160 +171 @@
     }
     
-    void PickupManager::usePickup(unsigned int index, PickupCarrier* carrier, bool use)
+    void PickupManager::usePickup(int index, PickupCarrier* carrier, bool use)
     {
         Pickupable* pickup = carrier->getPickup(index);

code/branches/pickup4/src/modules/pickup/PickupManager.h

@@ -72 +72 @@
             orxonox::PickupCarrier* getPawn(void);
             
-            unsigned int getNumCarrierChildren(orxonox::PickupCarrier* carrier);
-            orxonox::PickupCarrier* getCarrierChild(unsigned int index, orxonox::PickupCarrier* carrier);
+            int getNumCarrierChildren(orxonox::PickupCarrier* carrier);
+            orxonox::PickupCarrier* getCarrierChild(int index, orxonox::PickupCarrier* carrier);
             
-            unsigned int getNumPickups(orxonox::PickupCarrier* carrier);
-            PickupRepresentation* getPickupRepresentation(unsigned int index, orxonox::PickupCarrier* carrier);
-            void dropPickup(unsigned int index, orxonox::PickupCarrier* carrier);
-            void usePickup(unsigned int index, orxonox::PickupCarrier* carrier, bool use);
+            const std::string& getCarrierName(orxonox::PickupCarrier* carrier);
+            
+            int getNumPickups(orxonox::PickupCarrier* carrier);
+            PickupRepresentation* getPickupRepresentation(int index, orxonox::PickupCarrier* carrier);
+            void dropPickup(int index, orxonox::PickupCarrier* carrier);
+            void usePickup(int index, orxonox::PickupCarrier* carrier, bool use);
             // tolua_end
             

code/branches/pickup4/src/modules/pickup/PickupRepresentation.cc

@@ -86 +86 @@
         this->name_ = "Pickup";
         this->spawnerTemplate_ = "";
-        this->inventoryRepresentation_ = "";
+        this->inventoryRepresentation_ = "Default";
         this->pickup_ = NULL;
     }
@@ -98 +98 @@
         SUPER(PickupRepresentation, XMLPort, xmlelement, mode);
         
-        XMLPortParam(PickupRepresentation, "name", setName, getName, xmlelement, mode);
-        XMLPortParam(PickupRepresentation, "description", setDescription, getDescription, xmlelement, mode);
+        XMLPortParam(PickupRepresentation, "pickupName", setPickupName, getPickupName, xmlelement, mode);
+        XMLPortParam(PickupRepresentation, "pickupDescription", setPickupDescription, getPickupDescription, xmlelement, mode);
         XMLPortParam(PickupRepresentation, "spawnerTemplate", setSpawnerTemplate, getSpawnerTemplate, xmlelement, mode);
         XMLPortParam(PickupRepresentation, "inventoryRepresentation", setInventoryRepresentation, getInventoryRepresentation, xmlelement, mode);

code/branches/pickup4/src/modules/pickup/PickupRepresentation.h

@@ -68 +68 @@
             @param name The name.
             */
-            inline void setName(const std::string& name)
+            inline void setPickupName(const std::string& name)
                 { this->name_ = name; }
             /**
@@ -74 +74 @@
             @param description The Description.
             */
-            inline void setDescription(const std::string& description)
+            inline void setPickupDescription(const std::string& description)
                 { this->description_ = description; }
             /**
@@ -107 +107 @@
             @return Returns the name.
             */
-            inline const std::string& getName(void) { return this->name_; } // tolua_export
+            inline const std::string& getPickupName(void) { return this->name_; } // tolua_export
             /**
             @brief Get the description of the Pickupable represented by this PickupRepresentation.
             @return Returns the description.
             */
-            inline const std::string& getDescription(void) { return this->description_; } // tolua_export
+            inline const std::string& getPickupDescription(void) { return this->description_; } // tolua_export
             /**
             @brief Get the name of spawnerTemplate the Pickupable represented by this PickupRepresentation.