Changeset 2327

Timestamp:

Dec 3, 2008, 4:30:13 PM (16 years ago)

Author:

polakma

Message:

fixed parentWeaponSystem-Pointer bugs

Location:

code/branches/weapon2/src/orxonox/objects

Files:

• weaponSystem/Weapon.cc (modified) (3 diffs)
• weaponSystem/Weapon.h (modified) (2 diffs)
• weaponSystem/WeaponPack.cc (modified) (3 diffs)
• weaponSystem/WeaponSet.cc (modified) (3 diffs)
• weaponSystem/WeaponSlot.cc (modified) (1 diff)
• weaponSystem/WeaponSystem.cc (modified) (2 diffs)
• weaponSystem/weapons/LaserGun.cc (modified) (1 diff)
• worldentities/pawns/Pawn.cc (modified) (2 diffs)

code/branches/weapon2/src/orxonox/objects/weaponSystem/Weapon.cc

@@ -37 +37 @@
 namespace orxonox
 {
+    CreateFactory(Weapon);
+
     Weapon::Weapon(BaseObject* creator) : BaseObject(creator)
     {
@@ -42 +44 @@
         this->bulletReadyToShoot_ = true;
         this->magazineReadyToShoot_ = true;
-        this->setParentWeaponSystem();
     }
 
@@ -95 +96 @@
      *
      */
-    void Weapon::setParentWeaponSystem()
-    {   this->parentWeaponSystem_ = this->parentWeaponSlot_->getParentWeaponSystem(); }
-
     Munition * Weapon::getAttachedMunition()
     {   return this->munition_; }

code/branches/weapon2/src/orxonox/objects/weaponSystem/Weapon.h

@@ -52 +52 @@
             virtual void fire();
             void attachNeededMunition(std::string munitionType);
-            virtual void setParentWeaponSystem();
             Munition * getAttachedMunition();
 
@@ -72 +71 @@
 
             inline void setParentWeaponSlot(WeaponSlot *parentWeaponSlot)
-                { parentWeaponSlot_=parentWeaponSlot; };
+                { this->parentWeaponSlot_=parentWeaponSlot; };
             inline WeaponSlot * getParentWeaponSlot()
-                { return parentWeaponSlot_; };
+                { return this->parentWeaponSlot_; };
+            inline void setParentWeaponSystem(WeaponSystem *parentWeaponSystem)
+                { this->parentWeaponSystem_=parentWeaponSystem; };
+            inline WeaponSystem * getParentWeaponSystem()
+                { return this->parentWeaponSystem_; };
 
         private:

code/branches/weapon2/src/orxonox/objects/weaponSystem/WeaponPack.cc

@@ -39 +39 @@
 namespace orxonox
 {
+    CreateFactory(WeaponPack);
+
     WeaponPack::WeaponPack(BaseObject* creator) : BaseObject(creator)
     {
@@ -66 +68 @@
         for (int i=0; i < (int) this->weapons_.size(); i++)
         {
-COUT(0) << "WeaponPack::fire"<< i << std::endl;
+COUT(0) << "WeaponPack::fire (attached from WeaponSet)"<< i << std::endl;
             this->weapons_[i]->fire();
         }
@@ -94 +96 @@
     void WeaponPack::addWeapon(Weapon * weapon)
     {
+        weapon->setParentWeaponSystem(this->parentWeaponSystem_);
         this->weapons_.push_back(weapon);
     }

code/branches/weapon2/src/orxonox/objects/weaponSystem/WeaponSet.cc

@@ -39 +39 @@
 namespace orxonox
 {
+    CreateFactory(WeaponSet);
+
     WeaponSet::WeaponSet(BaseObject* creator, int k) : BaseObject(creator)
     {
@@ -53 +55 @@
     void WeaponSet::attachWeaponPack(WeaponPack *wPack)
     {
+COUT(0) << "WeaponSet::attachWeaponPack" << std::endl;
         if ( this->parentWeaponSystem_->getWeaponSlotSize()>0 && wPack->getSize()>0 && ( wPack->getSize() <= this->parentWeaponSystem_->getWeaponSlotSize() ) )
         {
@@ -75 +78 @@
     {
         //fires all WeaponSlots available for this weaponSet attached from the WeaponPack
+COUT(0) << "WeaponSet::fire" << std::endl;
         this->attachedWeaponPack_->fire();
     }

code/branches/weapon2/src/orxonox/objects/weaponSystem/WeaponSlot.cc

@@ -38 +38 @@
 namespace orxonox
 {
+    CreateFactory(WeaponSlot);
+
     WeaponSlot::WeaponSlot(BaseObject* creator) : PositionableEntity(creator)
     {

code/branches/weapon2/src/orxonox/objects/weaponSystem/WeaponSystem.cc

@@ -46 +46 @@
 namespace orxonox
 {
+    CreateFactory(WeaponSystem);
 
     WeaponSystem::WeaponSystem(BaseObject* creator) : BaseObject(creator)
@@ -61 +62 @@
     void WeaponSystem::attachWeaponPack(WeaponPack *wPack, unsigned int firemode)
     {
+        wPack->setParentWeaponSystem(this);
         this->weaponSets_[firemode]->attachWeaponPack(wPack);
         this->weaponPacks_[firemode] = wPack;
-        wPack->setParentWeaponSystem(this);
     }
 
     void WeaponSystem::attachWeaponSlot(WeaponSlot *wSlot)
     {
+        wSlot->setParentWeaponSystem(this);
         this->weaponSlots_.push_back(wSlot);
-        wSlot->setParentWeaponSystem(this);
     }
 
     void WeaponSystem::attachWeaponSet(WeaponSet *wSet)
     {
+        wSet->setParentWeaponSystem(this);
         this->weaponSets_.push_back(wSet);
-        wSet->setParentWeaponSystem(this);
     }
 

code/branches/weapon2/src/orxonox/objects/weaponSystem/weapons/LaserGun.cc

@@ -39 +39 @@
 namespace orxonox
 {
+    CreateFactory(LaserGun);
+
     LaserGun::LaserGun(BaseObject* creator) : Weapon(creator)
     {

code/branches/weapon2/src/orxonox/objects/worldentities/pawns/Pawn.cc

@@ -55 +55 @@
         //WeaponSystem
         weaponSystem_ = new WeaponSystem(this);
+        /*
         WeaponSet * weaponSet1 = new WeaponSet(this,1);
         this->weaponSystem_->attachWeaponSet(weaponSet1);
         //totally bad solution...
         weaponSet1->setParentWeaponSystem(weaponSystem_);
-
+        */
 
         this->registerVariables();
@@ -157 +158 @@
 
     void Pawn::setWeaponSlot(WeaponSlot * wSlot)
-    {   this->weaponSystem_->attachWeaponSlot(wSlot);   }
+    {   
+COUT(0) << "Pawn::setWeaponSlot" << std::endl;
+        this->weaponSystem_->attachWeaponSlot(wSlot);   }
     WeaponSlot * Pawn::getWeaponSlot(unsigned int index) const
     {   return this->weaponSystem_->getWeaponSlotPointer(index);    }
     
     void Pawn::setWeaponPack(WeaponPack * wPack)
-    {   this->weaponSystem_->attachWeaponPack( wPack,wPack->getFireMode() );   }
+    {   
+COUT(0) << "Pawn::setWeaponPack" << std::endl;
+        this->weaponSystem_->attachWeaponPack( wPack,wPack->getFireMode() );   }
     WeaponPack * Pawn::getWeaponPack(unsigned int firemode) const
     {   return this->weaponSystem_->getWeaponPackPointer(firemode);    }
     
     void Pawn::setWeaponSet(WeaponSet * wSet)
-    {   this->weaponSystem_->attachWeaponSet(wSet);   }
+    {   
+COUT(0) << "Pawn::setWeaponSet" << std::endl;
+        this->weaponSystem_->attachWeaponSet(wSet);   }
     WeaponSet * Pawn::getWeaponSet(unsigned int index) const
     {   return this->weaponSystem_->getWeaponSetPointer(index);    }