Question

我有一张声明为

的地图

std::map<std::string, Texture*> textureMap;

我用它来将纹理文件的路径与实际纹理配对，这样我就可以通过路径引用纹理，而不会为个别精灵加载相同的纹理。我不知道该怎么做才能正确销毁ResourceManager类（地图所在的）的析构函数中的纹理。

我想过使用带有这样的迭代器的循环：

ResourceManager::~ResourceManager()
{
    for(std::map<std::string, Texture*>::iterator itr = textureMap.begin(); itr != textureMap.end(); itr++)
    {
        delete (*itr);
    }
}

但这不起作用，它说删除预期指针。现在已经很晚了，所以我可能只是错过了一些明显的东西，但我想让它在睡前工作。所以我是关闭还是我完全朝着错误的方向发展？

Answer 1

就您的示例代码而言，您需要在循环内执行此操作：

delete itr->second;

地图有两个元素，您需要删除第二个元素。在您的情况下，itr->first为std::string，itr->second为Texture*。

如果您需要删除特定条目，可以执行以下操作：

std::map<std::string, Texture*>::iterator itr = textureMap.find("some/path.png");
if (itr != textureMap.end())
{
    // found it - delete it
    delete itr->second;
    textureMap.erase(itr);
}

您必须确保该条目存在于地图中，否则您在尝试删除纹理指针时可能会出现异常。

替代方法可能是使用std::shared_ptr而不是原始指针，然后可以使用更简单的语法从地图中删除项目并让std::shared_ptr处理适当时删除底层对象。这样，您可以将erase()与关键参数一起使用，如下所示：

// map using shared_ptr
std::map<std::string, std::shared_ptr<Texture>> textureMap;

// ... delete an entry ...
textureMap.erase("some/path.png");

这将做两件事：

从地图中删除条目（如果存在）
如果没有对Texture*的其他引用，则该对象将被删除

为了使用std::shared_ptr，您需要最近的C ++ 11编译器或Boost。

Answer 2

你没有使用合适的工具。

指针不应“拥有”数据。

改为使用boost::ptr_map<std::string, Texture>。

Answer 3

答案并没有完全解决循环问题。至少Coverty（TM）不允许在循环内擦除迭代器并仍然使用它来继续循环。无论如何，删除内存后，在地图上调用clear（）应该完成其余的工作：

ResourceManager::~ResourceManager()
{
    for(std::map<std::string, Texture*>::iterator itr = textureMap.begin(); itr != textureMap.end(); itr++)
    {
        delete (itr->second);
    }
    textureMap.clear();
}

正确销毁std :: map中的指针

3 个答案:

指针不应“拥有”数据。