是的,你的代码在遍历 keyFrameMap
的同时删除元素时会有问题。这是因为在 C++ 中,当你从容器中删除元素后,迭代器会失效,接下来的操作可能导致未定义行为。
具体来说,当你调用 keyFrameMap.erase(it);
后,it
变得无效。在下一次循环中,如果继续使用这个无效的迭代器,就会导致错误。因此,需要特别小心处理这种情况。
解决方案
可以使用 erase
函数返回的迭代器来继续遍历。这里有两种常用的方法:
方法 1: 使用 erase()
返回值
for (auto it = keyFrameMap.begin(); it != keyFrameMap.end();) {
int64_t key = it->first;
const std::string& value = it->second;
if (std::abs(key - frame->pts) < TIME_BASE_DIFFEN) {
saveKeyFrame(frame, value.c_str());
it = keyFrameMap.erase(it); // erase returns the next iterator
} else {
++it; // only increment if not erasing
}
}
方法 2: 使用范围-based for 循环并收集要删除的键
如果需要在遍历期间进行多个删除操作,可以考虑先收集要删除的键,然后再进行一次遍历来执行删除操作。
std::vector<int64_t> keysToRemove;
for (const auto& pair : keyFrameMap) {
int64_t key = pair.first;
const std::string& value = pair.second;
if (std::abs(key - frame->pts) < TIME_BASE_DIFFEN) {
saveKeyFrame(frame, value.c_str());
keysToRemove.push_back(key);
}
}
for (const auto& key : keysToRemove) {
keyFrameMap.erase(key);
}
总结
- 避免直接在循环中删除:当你在遍历容器时,不应该直接修改(添加或删除)其内容。
- 安全地使用返回值:当调用
erase()
删除元素时,利用其返回值更新迭代器以继续遍历。 - 额外步骤:对于更复杂的条件,可以通过收集要删除的项然后统一处理来避免错误。
内容由零声教学AI助手提供,问题来源于学员提问