Почему мой конструктор копий не работает?

Question

Это, наверное, нелегко, но я немного в тупике. У меня были проблемы с моим vectors not behaving nicely, и теперь похоже, что я нашел преступника. Ниже приведена моя версия Player.

class Player { 
private: 
    std::string _firstName; 
    std::string _lastName; 
public: 
    Player(std::string firstName, std::string lastName) { 
     _firstName = firstName; 
     _lastName = lastName; 
    }; 
    Player(const Player& otherPlayer) { 
     _firstName = otherPlayer._firstName.c_str(); 
     _lastName = otherPlayer._lastName.c_str(); 
     std::cout << "Created " << _firstName << " " << _lastName << std::endl; // Why doesn't _firstName and _lastName contain anything? 
    }; 
    std::string GetName() { return _firstName + " " + _lastName; }; 
}; 

int main(int argc, const char * argv[]) 
{ 

    Player player1 = Player("Bill", "Clinton"); 
    Player player2 = Player(player1); 

    std::cout << "Player: " << player2.GetName() << std::endl; 

    return 0; 
} 

Выходной сигнал является скудным Player:. Я не уверен, почему мой конструктор копий не делает то, что я хочу, чтобы это сделать, в частности, в свете рекомендаций, таких как this (комментарий Zac Howland учитывает c_str(); -part). Я нарушаю the rule of three (что, кстати, я до сих пор не полностью опустил голову)? Я был бы очень благодарен, если бы кто-нибудь мог указать мне в правильном направлении!

Answer 1

Это работает для меня: http://ideone.com/aenViu

Я просто addded:

#include <iostream> 
#include <string> 

Но есть что-то я не понимаю:

_firstName = otherPlayer._firstName.c_str(); 
_lastName = otherPlayer._lastName.c_str(); 

Почему .c_str()? Вы конвертируете string в char*, чтобы присвоить его новому string?

EDIT: Из комментария, Zac Хоулэнд указал: «До C++ 11, если вы хотите, чтобы убедиться, ваша строка была скопирована (вместо подсчета ссылок), вы должны были использовать метод c_str(), чтобы заставить он копирует строку. Новый стандарт устраняет это, но если он использует старый компилятор или тот, который еще не полностью реализовал C++ 11, он обеспечит глубокую копию ».

Вобще:

_firstName = otherPlayer._firstName; 
_lastName = otherPlayer._lastName; 

И, вы действительно нужен этот конструктор копирования? По умолчанию будет делать то, что вы хотите, я думаю ...

---

Кроме того, вместо назначения членов:

Player(std::string firstName, std::string lastName) { 
    _firstName = firstName; 
    _lastName = lastName; 
} 

использовать член-инициализации-лист вместо:

Player(std::string firstName, std::string lastName) : 
    _firstName(std::move(firstName)), 
    _lastName(std::move(lastName)) 
{} 

В первом случае, когда вызывается конструктор по умолчанию для строки, а затем оператор присваивания строки string, определенно могут быть (небольшие) потери эффективности compa красный во второй случай, который непосредственно вызывает конструктор-копию.

Последняя вещь, когда это возможно, не передавать значения в качестве аргументов метода, передавать ссылки и даже константные ссылки, когда они не должны быть изменены:

Player(const std::string& firstName, const std::string& lastName) 
//  ^^^^^   ^   ^^^^^   ^
    : _firstName(firstName) // no move here, since args are constant references 
    , _lastName(lastName) 
{} 

---

Рабочая live example всего изменения.