Function to create object

[closed account 3R5fjE8b]

I have declaring instances of my equipment class everytime I need to use it. Is there a way I can use a function to do it so I don't have to remember everytime what the specifics of that object were and use the function to create the object then use it in a different function? something like this?

void player::createSword()
{
sword("Sword", 2, 0);
}


my current code... line 63 and 71, 89, 97

#include <iostream>
#include <vector>
#include "equipment.h"
#include "player.h" 




player::player()
{
                setArmor(0);
                setAttackPoints(1);
                
}

void player::setArmor(int newArmor)
{
     armorPoints = newArmor;
}

int player::getArmor()
{
    return armorPoints;
}

void player::setAttackPoints(int newAttack)
{
     attackPoints = newAttack;
}

int player::getAttack()
{
    return attackPoints;
}
     
void player::displayInventory()
{
     if ( inventory.empty() )
     {
          std::cout << "You have nothing in your inventory." << std::endl;
     }
     else
     {
         std::cout << "You have " << inventory.size() << " items in your inventory." << std::endl;
         
         for ( int x = 0; x < inventory.size(); x++ )
         {
             std::cout << inventory[x].itemName;
             if ( inventory[x].itemName == weaponEquipped )
             {
                  std::cout << "   [Equipped]" << std::endl;
             }
             else
             {
                 std::cout << std::endl;
             }
         }     
     }
}

void player::equipSword()
{
     equipment sword("Sword", 2, 0);
     setAttackPoints(sword.getDamage());
     weaponSlot = true;
     weaponEquipped = "Sword";
}
   
void player::addSword()
{    
     equipment sword("Sword", 2, 0);
     inventory.push_back(sword);
     
}

void player::removeSword()
{
     for ( int x = 0; x < inventory.size(); x++ )
     {
         if ( inventory[x].itemName == "Sword" )
         {
              inventory.erase(inventory.begin() + x);
         }
     }
}

void player::equipAxe()
{
     equipment axe("Axe", 4, 0);
     setAttackPoints(axe.getDamage());
     weaponSlot = true;
     weaponEquipped = "Axe";
}

void player::addAxe()
{
     equipment axe("Axe", 4, 0);
     inventory.push_back(axe);
}

void player::removeAxe()
{
     for ( int x = 0; x < inventory.size(); x++ )
     {
         if ( inventory[x].itemName == "Axe" )
         {
              inventory.erase(inventory.begin() + x);
         }
     }
}
     

[lolnyancats]

No you cannot for normal commands such as cin>> or cout<<, so i am pretty sure that in this case it will not work either

[closed account 3R5fjE8b]

I know I can't use it for cin and cout, I just want to create one function for creating the object, so I don't have to retype it everytime. I'm not sure if that makes sense or is possible.

Besides that, does my code look okay?

[nvrmnd]

Do you mean something like this :

// ...

sword* player::createSword()
{
    return new sword("Sword", 2, 0);
}

// ...

int main()
{
    player player1( ... );
    sword* a_sword = player1.createSword();

    some_func( a_sword );

    delete a_sword; // <-- do not forget this
}

Or a more safer way is :

#include <memory>
// ...

std::unique_ptr<sword> player::createSword()
{
    return std::unique_ptr<sword>( new sword("Sword", 2, 0) );
}

// ...

int main()
{
    player player1( ... );
    auto a_sword = player1.createSword(); // a_sword's type is unique_ptr

    some_func( a_sword );

   //delete a_sword; // <-- not needed, the destructor of unique_ptr<> automatically frees memory allocated by new
}