为什么指针不会存储和打印与它应该匹配的对象相同的数据？(C++)

Question

我正在做一个游戏，在这个游戏中，用户可以选择一个新的游戏。Player 的向量中提取出一个对象 Player 从文本文件中读取的对象。

我要求用户输入一个名字，以便从向量中选择一个球员，然后我在向量中迭代寻找一个匹配的名字（假设每个球员都有不同的名字）。

一旦它们匹配，我想把我的 Player player1 指针指向播放器向量中的那个对象。

然而， player1 指针在找到匹配项后，没有在for循环外打印正确的名称。指针默认打印的是 每次最后一个播放器在文本文件中 而不是符合用户输入的播放器名称。

我该怎么做才能解决这个问题？

vector<Player> playerVectorReadIn;
Player *player1; string name="";

//reading in the data

ifstream inFile;

inFile.open("playerData.txt");
if(!inFile){
    cout<<"Error! Unable to open file";
    exit(1);
}
while(inFile>>name){
    Player playerObject(name);
    playerVectorReadIn.push_back(playerObject);
}
inFile.close();

//finding matching data in vector 

cout<<"\n\tEnter the name of the player from the list you choose: ";
getline(cin,name);

    for(Player p:playerVectorReadIn){
        if(p.getName()==name){
            flag=true;
            player1=&p; //setting pointer to player - https://stackoverflow.com/questions/2988273/c-pointer-to-objects
            cout<<name; //the user entered name (example Bob)
            cout<<p.getName(); //the matching name in the vector (Bob)
            //both print the same name here so it works
        }
    }

if(flag==true){
    cout<<"\nYou will be playing as: "<<player1->getName(); 
    //prints as the name of the last object in the text file (example Ryan)
    //not the matching name as above - why?
}else{
    cout<<"\nPlayer not found.";
}
flag=false; name="";

文本文件内容的例子。

Dave 
Jill 
Bob 
Mary 
Donna
Ryan

Answer 1

for(Player p:playerVectorReadIn){

这是按值迭代。p 是一个对象，它的范围是这个对象的局部的 for 循环，并且是向量中对象的副本。每次这个循环迭代时，这个对象都会被销毁。如果循环再次迭代，一个新的 p 对象被创建。

     player1=&p;

这将保存一个指向这个本地对象的指针。然而，正如我们刚刚发现的那样。p 在循环结束时被销毁。player1 变成了一个指向被销毁对象的悬空指针，随后使用它就变成了未定义的行为。

这就解释了你所观察到的垃圾结果。幸运的是，解决方案非常简单，通过引用进行迭代。

for(Player &p:playerVectorReadIn){

p 现在是对向量中实际对象的引用 只要向量本身没有被随后重新分配，向量中对象的指针就会保持有效。

Answer 2

在这个循环中。

for(Player p : playerVectorReadIn){
        if(p.getName()==name){
            player1=&p; //setting pointer to player 

你正在做一个 拷贝 的 Player 然后获取它的地址。循环结束时，拷贝会死掉，你会留下一个悬空的指针。

你需要通过引用来迭代向量。

for(Player & p:playerVectorReadIn){
        if(p.getName()==name){
            player1=&p; //setting pointer to player (correctly)