Question

I’m getting some errors when using strcpy_s and can’t figure out what I’m doing wrong.

The code:

Player.hpp:

string name;
Player(string);

Player.cpp:

Player::Player(string newName)
{
    strcpy_s(name, name.size(), newName);//error is here
    health = 20;
}

The errors:

• Too many arguments in function call
• No instance of overloaded function ‘strcpy_s’ matches the argument list

Answer 1

You cannot use strcpy_s to copy std::string. Actually, you just need to do:

Player::Player(string newName) {
    name = newName;
    health = 20;
}

Even better, you could use a constructor initialization list:

Player::Player(string newName) : name(newName), health(20) {}

As a reference, here you have a detailed description of std::string class.