我有一个地图容器来存储某些对象,以及它们的名称和类型:
typedef std::map<std::string, std::pair<ObjType, ObjBase*> > ObjContainer;
但是,在代码的许多部分中,都有这样的结构:
ObjContainer::iterator it = mObjContainer.find(name);
if (it != mObjContainer.end()) {
if (it->second.second) {
it->second.second->setObj2Default();
delete it->second.second;
it->second.second = 0;
}
}
显然,很多“it->second.second”不是很清楚,而且无法维护。如果以后改了,比如多支持一个领域,就全断了。因此,我试图通过函数来更改它们以访问字段,如下所示:
ObjBase*& getObjPtr(ObjContainer::iterator it) {
return it->second.second;
}
同样,还有函数 getObjName 和 getObjType。
还有人建议我让迭代器返回这些字段会更清楚:
it.objPtr();
it.objName();
it.objType();
但我认为不应继承 STL 迭代器以具有这些功能,对吗?除了为地图创建一个包装器并拥有自己的带有这些函数的迭代器外,我没有其他方法可以做到这一点。
那么,最合适的选择是什么?有没有其他方法可以解决我没有看到的这个问题?