std::list 迭代器

Question

以下代码不是将指针返回到它返回的 audioResource 其他无效的东西，我已经用调试器完成了，问题出在这一行

return *list_it;

这是我的功能：

AudioResource* AudioManager::getResource(const unsigned int ID)
{
    std::list<AudioResource*>::iterator list_it;
    for(list_it = m_resources.begin(); list_it!= m_resources.end(); list_it++)
    {
        if((*list_it)->getID()==ID)
            {
std::cout<<*(*list_it)->getFileName();
            return *list_it;
        }
    }
    return nullptr;
}

O，我试过把它写成 (*list_it) 但我得到了相同的结果 =s

它是如何填充的...

Resource* AudioManager::addResource(const unsigned int ID, 
      const std::string fileName,  const unsigned int scope,
      const std::string type)
{
     AudioResource* temp;
     if(type == "AUDIO_TYPE_SAMPLE")
     {
          temp = new AudioResource(ID,fileName,scope,
                      RESOURCE_AUDIO,AUDIO_TYPE_SAMPLE);
          m_resources.push_back(temp);
     }
     else if(type == "AUDIO_TYPE_STREAM")
     {
          temp = new AudioResource(ID,fileName,scope,
                    RESOURCE_AUDIO,AUDIO_TYPE_STREAM);
          m_resources.push_back(temp);
     }

     return temp;
}

调用获取资源

cout<<AudioManager::getInstance()->getResource(IDnum)->getFileName();

Answer 1

如果type 不是这两个值中的任何一个，则将未初始化的指针添加到m_resources：

AudioResource* temp;
if(type == "AUDIO_TYPE_SAMPLE")
{
    temp = new AudioResource(ID,fileName,scope,RESOURCE_AUDIO,AUDIO_TYPE_SAMPLE);
}
else if(type == "AUDIO_TYPE_STREAM")
{
    temp = new AudioResource(ID,fileName,scope,RESOURCE_AUDIO,AUDIO_TYPE_STREAM);
}
m_resources.push_back(temp);

将temp初始化为NULL，如果temp != NULL仅添加到m_resources。

此外，该函数返回相同的未初始化指针。

Answer 2

如果ID 不存在，您将返回nullptr，但您永远不会在调用站点检查它，如果ID 不存在，这将为您提供空指针访问，并且可能会创建问题。

AudioManager::getInstance()->getResource(IDnum)->getFileName();

改成

AudioResource* res = AudioManager::getInstance()->getResource(IDnum);
if(res)
  std::cout << res->getFileName();