评估 vector::empty() 正在使程序崩溃

Question

我目前正在编写另一个程序（特别是游戏）的扩展，并且我一直坚持解决以下问题已经好几个小时了。

任何时候玩家（客户端）连接到游戏服务器，都会调用OnClientConnected(int client_number); 函数。在这个函数中，我创建了一个 CPlayer 类的实例，该类存储玩家信息，例如健康、速度和我可能需要的任何其他数据。该实例然后由以下类存储：

class CPlayerManager
{
    std::vector<Player*> *player_list;

    public:
    CPlayerManager();
    ~CPlayerManager();
    void AddPlayer(client_id)
    *CPlayer GetPlayerInstance(client_id);
}

CPlayerManager()
{
     player_list = new std::vector<Player*>;
}

~CPlayerManager()
{
     delete player_list;
}

void CPlayerManager::AddPlayer(client_id)
{
     CPlayer *player = new CPlayer(client_id);
     player_list->push_back(player);
}

void *CPlayerManager::GetPlayerInstance(client_id)
{
     if(player_list->empty())
     {
         return NULL;
     }

     for(std::vector<CPlayer*>::size_type i = 0; i != player_list->size(); i++)
 {
    int client = player_list->at(i)->GetClientId();
    if(client_id == client)
    {
        return player_list->at(i);
    }
 }
 return NULL;
}

原始游戏函数使用客户端索引作为参数和返回值，所以我需要一直从客户端索引获取玩家实例。问题出在GetPlayerInstance 函数中。当CPlayerManager 实例尚未初始化和存储任何客户端时，可能会调用此函数，因此我们可能有一个空向量。 每当向量为空时调用GetPlayerInstance 函数，整个游戏服务器都会崩溃。我已经调试了一下，我注意到程序崩溃了：

//Code reaches here
if(player_list->empty())
{
    //code does not reach here
    return NULL;
}

在我对if 语句评估empty 之前，执行bool empty = player_list->empty()) 有效。

什么可能导致这个奇怪的崩溃？

Answer 1

您的CPlayerManager 类没有复制赋值运算符或复制构造函数。因此，如果您复制它，删除一个实例将删除另一个实例的向量，从而导致灾难。 （关注rule of three。）

真的，这段代码应该从轨道上被核爆。您不需要指向向量的指针，也不需要指针向量。这两件事都是自找麻烦。