是的，你的代码在遍历 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助手提供，问题来源于学员提问
![int encryption::Base64_Encode(char *pDest, const char *pSrc, int srclen)
{
BYTE input[3], output[4];
size_t i, index_src = 0, index_dest = 0;
for(i = 0; i < srclen; i += 3)
{
//char [0]
input[0] = pSrc[index_src++];
...](https://linuxcpp.0voice.com/zb_users/upload/2023/05/202305162240413467127.png)
![nodeMap: make(map[string]*Node)和nodeMap: map[string]*Node{}这两种写法在Golang中有什么区别](https://linuxcpp.0voice.com/zb_users/upload/2023/05/202305162237121215388.png)
![package main
import (
"fmt"
"sync"
"time"
)
func main() {
chaS := make([]chan struct{}, 10)
var global = 1
wg := sync.WaitGroup{}
for i := 0; i < 10; i++ {](https://linuxcpp.0voice.com/zb_users/upload/2023/05/202305162233565655357.png)
![root@ght:/etc/systemd/system# cat lsyncd.service
[Unit]
Description=Lsync Service
After=network.target
[Service]
Type=simple
ExecStart=/usr/local/sbin/lsyncd /etc/lsyncd.conf.lua
Restart=always
RestartSec=15s
[Install]
WantedBy=multi-user.target](https://linuxcpp.0voice.com/zb_users/upload/2023/05/202305162214438435275.png)
