在 Go 中，直接在 range 循环中调用 delete() 删除 map 元素是不安全的，必须通过读写锁协调；推荐先收集待删键、再统一加写锁删除，兼顾性能与线程安全。
在 go 中，直接在 `range` 循环中调用 `delete()` 删除 map 元素是不安全的，必须通过读写锁协调；推荐先收集待删键、再统一加写锁删除，兼顾性能与线程安全。
Go 的原生 map 类型不是并发安全的：任何写操作（包括 delete、赋值、make 后首次写入）都必须与所有其他读/写操作互斥。即使你使用 sync.RWMutex 保护了读操作，在 RLock() 持有期间执行 delete(values, key) 仍会触发运行时 panic——典型错误如：
fatal error: concurrent map read and map write
这是因为 delete 是写操作，而 RLock() 仅允许并发读，绝不允许写。因此，你提供的四种变体中：
✅ Variant 2（收集键后批量删除）是最佳实践：
在读锁保护下遍历并收集需删除的 key（如 []string），释放读锁后获取写锁，再批量 delete。既避免了锁竞争，又最小化写锁持有时间。❌ Variant 1（range 中直接 delete）：绝对禁止，必 panic。
❌ Variant 3（RLock → RUnlock → Lock → delete → Unlock → RLock）：逻辑错误且低效——循环中反复切换锁，易导致竞态（如 key 在两次锁切换间被重复处理或遗漏），且严重降低吞吐。
❌ Variant 4（go l.delete(key)）：语法错误（sync.RWMutex 无 delete 方法），且 goroutine 无法解决锁粒度问题。
以下是修正后的安全实现示例：
var (
values map[string]A
mu sync.RWMutex // 推荐使用更语义清晰的变量名，如 mu 而非 l
)
func main() {
values = make(map[string]A)
go generate()
for {
// Step 1: 安全读取 —— 收集待删键
var keysToDelete []string
mu.RLock()
for key, value := range values {
if value.Error != nil {
keysToDelete = append(keysToDelete, key)
} else {
value.Job() // 注意：value 是副本，修改不影响 map 中原始值
}
}
mu.RUnlock()
// Step 2: 安全删除 —— 批量写操作
if len(keysToDelete) > 0 {
mu.Lock()
for _, key := range keysToDelete {
delete(values, key)
}
mu.Unlock()
}
time.Sleep(10 * time.Second)
}
}
⚠️ 关键注意事项：
- range 遍历时 value 是 map 中元素的拷贝，对其字段修改（如 value.Error = nil）不会影响原 map，需用 values[key].Error = nil（此时需写锁）；
- 若删除逻辑复杂（如依赖删除前状态、需原子性更新+删除），应统一在 Lock() 内完成全部操作；
- 对于高频读写场景，可考虑 sync.Map（适用于读多写少、key 类型固定），但其不支持 range 迭代，且删除仍需注意 LoadAndDelete 的语义。
总结：没有“边读边删”的捷径；安全 = 分离读/写阶段 + 显式锁控制。坚持“读锁收键、写锁删键”模式，即可在并发 map 操作中兼顾正确性与可维护性。
