列表中的指针问题

Question

由于某种原因，当我尝试读取 std::list (playerlist) 内的对象 (GamePlayer) 指针的属性时，它起初工作，但是当我稍后尝试在另一个函数中访问它时，我得到一堆随机数，而不是客户端套接字的数字。那是一口，抱歉。我希望有人能够阐明这一情况。我将包含有缺陷代码的简化版本。

class GameRoom  {

    list<GamePlayer*> playerlist;
    locigPort( LogicObj );
}


bool GameRoom::logicPort( LogicObj logit )  {   // This is room[1]

    list<GamePlayer*>::iterator it;

        for (it = playerlist.begin(); it != playerlist.end(); it++){  

                cout << "socket numbers " << (*it)->socketno << endl;  
                /* (*it)->socketno gives me a bunch of random numbers, 
                   not the socket numbers I was looking for! */
        }  

    return true;

}

bool RoomDB::addPlayer( GamePlayer *playerpoint )   {

    roomlist[1].playerlist.push_back( playerpoint );    
    // This adds the player object to the Gameroom object

    cout << "player point " << playerpoint->socketno << " roomno: " << roomno;  
    // This shows everything should be ok so far

    return true;
}

Answer 1

最可能的解释是，您调用 addPlayer 时使用的指针在调用 logicPort 时变得无效。一种可能性是您使用堆栈上对象的地址调用 addPlayer，并且该对象在堆栈展开时消失。

编辑 问题就在这里：

bool PlayerDB::addPlayer( int sid, GamePlayer tempplayer ) {
   ...
   roomman.addPlayer( &tempplayer, tempplayer.roomno );
}

PlayerDB::addPlayer 按值获取第二个参数。这意味着它会获得一个在该方法的生命周期内存在的副本。然后，您将指针指向该副本，并将其添加到列表中。一旦PlayerDB::addPlayer返回，指针就变得无效。

如果没有看到更多代码，很难提出好的修复建议。一种可能性是让 PlayerDB::addPlayer 将指针作为其第二个参数，并确保您不会在调用链的上一级重复相同的错误。

更好的可能性是将 playerlist 转换为 list：从您的代码来看，列表似乎不需要包含指针 /em>.这将大大简化事情。