c++ - 在 vector 中正确存储对象

Question

我是 C++ 的新手，这是我第一次发帖(灾难的秘诀)。我花了大约一天的时间来解决我的问题/问题，同时在论坛上找不到容易识别的解决方案。有可能我的问题已经被问到，或者解决方案已经发布，但我忽略了它或误解了它。存在类似的帖子 here ，但我不清楚如何处理这些信息。

我将提供这段代码要点的简明、高级摘要。然后，我将提出我无法找到答案的具体问题，然后我将遵循我编写的代码。

总结:我正在创建一个程序来帮助为游戏做簿记。游戏可能有任意数量的玩家，每个玩家都有一小部分属性/成员(例如 playerName、playerAllegiance 等)，这些是 Player 类/对象的元素。首先要求用户输入每个玩家的名字 (enteredName)，程序必须为每个输入的名字创建一个新的 Player 对象。这似乎可以由动态数组适本地处理，所以我选择使用一个 vector (称为 playerIndex)来存储每个 Player 对象。 for 循环允许用户输入名称，每个名称实例化一个新的 Player 对象，该对象将使用 vector::push_back 存储(复制？)到 playerIndex 中。在 for 循环结束时，应该为用户留下一个 Player 对象 vector ，每个 Player 对象在其 playerName 成员中存储一个名称。

问题/问题:在上述 for 循环中监视 vector 时，代码似乎可以正常工作。在用户输入第 N 个玩家的名字后，程序立即使用 Player 类函数 getPlayerName() [实际代码:playerIndex[playerCounter].getPlayerName()] 吐出存储在 playerIndex 的第 N 个元素中的 playerName 字符串。一旦用户输入了一个空白的 playerName(即在没有输入名字的情况下按回车)，这表明用户已经输入了所有玩家的名字，因此 for 循环终止。在此循环之后，旨在输出存储在 playerIndex 中的每个 Player 对象的 playerName 的循环不会输出预期的名称。我不知道为什么会这样，但根据我对构造函数的极少了解，我猜它与 Player 类的复制或移动构造函数有关。谁能清楚地解释如何处理这个问题？我担心我可能犯了一个愚蠢得可怜的新手错误和/或误解了 C++ 的一个关键概念。

代码:此代码被裁剪/简化为尽可能清晰。例如，Player 类显示只有一个成员 (playerName)，而在原始代码中，它有四五个其他成员。

//HeaderPlayerClass.hpp
#include <iostream>
#include <string>

#ifndef PLAYERCLASS_HPP
#define PLAYERCLASS_HPP

using std::string;

class Player {
    private:
        string *playerName;

    public:
        Player();
        Player(string);
        ~Player();
        string getPlayerName();
};

#endif




//PlayerClass.cpp
#include "HeaderPlayerClass.hpp"
#include <iostream>
#include <string>

using std::string;

Player::Player() {
    playerName = new string;
}

Player::Player(string enteredName) {
    playerName = new string;    
    *playerName = enteredName;
}

Player::~Player() {
    delete playerName;
}

string Player::getPlayerName() {
    return *playerName;
}





//main.cpp
#include <cstdio>
#include <iostream>
#include <string>
#include <vector>
#include "HeaderPlayerClass.hpp"

using std::cin;
using std::cout;
using std::string;
using std::vector;

int main(int argc, char** argv) {

    string buffer;
    vector<Player> playerIndex;

    int playerCounter = 0;

    for(;;) {
        if(playerCounter==0) {
            cout << "\nEnter player name and press enter; leave blank and press enter to continue.\n";
        }

        cout << "\nPlayer " << playerCounter+1 << ":";
        getline(cin, buffer);

        if(buffer == "esc") {
            cout << "PROGRAM EXITED BY USER\n";
            return 0;
        }

        if(buffer.empty()) {
            break;
        }

        playerIndex.push_back(Player(buffer));
        cout << "Player " << playerCounter+1 << "'s name:" << playerIndex[playerCounter].getPlayerName() << "\n";

        ++playerCounter;
    }

    for(int ii = 0 ; ii < playerIndex.size() ; ii++) {
        cout << "\nThis should display player " << ii+1 << "'s name:" << playerIndex[ii].getPlayerName();
    }

    return 0;
}

Answer 1

class Player {
    private:
        string *playerName;

不存储指向字符串的指针，存储字符串本身

class Player {
    private:
        string playerName;

构造函数

Player::Player() {
    playerName = new string;
}

如果您存储字符串本身，这是不必要的 - 默认构造函数将为您初始化它。

这就是问题开始的地方 - 这将使您容易受到内存泄漏的影响，除非您仔细编写析构函数

Player::Player(string enteredName) {
    playerName = new string;    
    *playerName = enteredName;
}

所有你需要的，如果你存储字符串本身:

Player::Player( const string& enteredName)
: playerName ( enteredName )
{}

你的构造函数真的很可怕。当您存储字符串本身时，只需将事情留给默认析构函数即可

Player::~Player() {
    delete playerName;
}

接下来，您太努力工作了，无法保留自己的柜台。 std::vector 维护自己的计数并使用

playerIndex.size()

会省去麻烦和bug