简短版本：错误14错误C2660：“ Player :: addSpell”：函数未采用1个参数

Question

I'm a slow learner to C++ here. My game code is a text based RPG. Here the player chooses to by a spell and add it to his spell book called 'Grimoire.' Grimoire is a vector type that contains 'Spell' types as elements. Spell is a struct type that has values such as it's name, damage range, MP, and price. If the code snippet is too vague for the issue concerned see the original question titled:
Error 14 error C2660: 'Player::addSpell' : function does not take 1 arguments

The problematic line is: Player.addSpell(mMagic[0]);

//Store.cpp
      ...       ...       ...

                    //If player already has the spell
                    if (Player.getSpellName() == mMagic[0].mSpell)
                    {
                        cout << "You already have this spell." << endl << endl;
                        break;
                    }
                    else
                    {
                        //Increase size of Grimoire to store the purchased spell
                        Player.increaseGrimoire();

                        //Add spell to Grimoire

                        Player.addSpell(mMagic[0]); //HERE IS THE PROBLEM*******


//Player.H
class Player
{
public:
    //Constructor.
    Player();

    //Methods

    void addSpell(Spell magic);

private:
    //Data members.
vector<Spell>mGrimoire;

};


//Player.cpp
void Player::addSpell(Spell magic) 
{
    int i = 0;
    for (; i < mGrimoire.size(); ++i)
    {
        if (mGrimoire[i].mSpell == magic.mSpell)
            continue;
        else if (!(mGrimoire[i].mSpell == magic.mSpell))
        {
            mGrimoire[i] = magic;
            break;
        }

    }
}

//Store.h
 struct Store
{
public:

private:

    vector<Spell>mMagic;

};

Answer 1

Player is a class and not an object. You need to create an object (an instance) of the Player class, and use that.

For example:

Player p;

...

p.addSpell(mMagic[0]);

Answer 2

您需要实例化一个对象（类Player的对象）以调用成员函数（addSpell（Spell））。只有静态成员函数和staic成员变量可以使用类名来调用，因为它们是类特定的，成员函数是对象特定的，因此您需要有一个对象实例来调用它们。

Answer 3

I think your real problem is that you need access to the same Player instance from many different modules.
Usually for that you should store Player instance in your "game logic" module (that should be easily accessible from anywhere) and implement accessor to that instance.

Try something like this:

//Game.h
class Game
{
    ...
    private:
        static Game *instance;
        Player *player;
    public:
        static Game* getInstance();
        Player *getPlayer();
        Game::init();
}

//Game.cpp
Game* Game::instance = NULL;
Game* Game::getInstance()
{
    return instance;
}

void Game::init()
{
    ...
    instance=this;
    ...
    player = new Player();
    ...
}

Player* Game::getPlayer()
{
    return player;
}

//Store.cpp
...
    Player* player=Game::getInstance()->getPlayer();
    //If player already has the spell
    if (player.getSpellName() == mMagic[0].mSpell)
    {
        cout << "You already have this spell." << endl << endl;
        break;
    }
    else
    {
        //Increase size of Grimoire to store the purchased spell
        player.increaseGrimoire();

        //Add spell to Grimoire

        player.addSpell(mMagic[0]); //HERE IS THE PROBLEM*******
    }

For more elaborate/aesthetic ways to handle such problems you should read about Design Patterns and Game Design/Programming Patterns.