map 的一个神奇的问题

[zkdfbb]

使用下面的一段代码来统计链接的访问情况，使用方法就是用 Incr 每次访问加 1 单独测试的时候，比如用 wrk 来压测都挺正常的 神奇的是，一换成 nginx，c.data 里面的 key 就变得不正常，把他打印出来，发现有很多相同的 key 比如本来就只有一个 key 的，他会出现 key: 1, key: 2, key: 1, key: 1 这种，搞不懂。。。

type Counter struct {
	sync.RWMutex
	data map[string]int
}

func (c *Counter) Get(key string) int {
	c.RLock()
	count := c.data[key]
	c.RUnlock()
	return count
}

func (c *Counter) Incr(key string) int {
	c.Lock()
	c.data[key]++
	count := c.data[key]
	c.Unlock()
	return count
}

func (c *Counter) Delete(key string) {
	c.Lock()
	delete(c.data, key)
	c.Unlock()
}

[skiy]

```defer c.RUnlock()```

[yuguorui96]

你的 nginx 不会是 fork 吧…

[zkdfbb]

@yuguorui96 nginx 是多进程，但是这个 go 的后端只有一个，应该没影响啊

[zkdfbb]

@skiy 这个代码简洁了一点~性能下降了一点~

[no1xsyzy]

至少这里看不出什么问题
能提供最小可复现样本吗？

[sujin190]

map 不可能多个 key 的，反常规的话也许是不可见字符？要么就是你输出代码有问题

[GTim]

show 你的完整测试代码

[AngryPanda]

盲猜这些 key 在不同协程中

[Lpl]

你这段代码槽点比较多，性能会很差：
1. 可以考虑用 atomic 而不是用锁
2. 可以考虑用管道来做 Producer-Consumer，然后多个协程消费，而不是考虑用锁
3. 实在想加锁，就给某个 key 加锁，而不是给 Counter 对象加锁。锁的粒度太粗了

给个比较完整的代码看看

[zkdfbb]

@Lpl 批评的对🤣
atomic 的话，由于 id 不是唯一的，好像不太方便
没试过用管道，我直观以为管道性能会更差一点，看来我理解错了，可以试一下
有看到过用分段锁的例子，concurrent-map 这个，不过我测了一下就用一把锁好像也足够用了就先这么用了，结果就出现了这个神奇的问题，完整的代码补上了，你看看能不能看出来哪里的问题🤣

[zkdfbb]

@AngryPanda 按理加了锁应该不同的协程也是 OK 的

[zkdfbb]

@no1xsyzy
@GTim

补在后面了，看看能不能发现什么问题

[Lpl]

脑袋疼..你这个问题出在 init 里边的那个协程了：
accessLog = Counter{data: make(map[string]int)}

里边多了这句话，我理解你这个协程就是想打印下当前已存在的数据？

[makdon]

@Lpl 我理解他这里的意思是，先保存下来当前的状态，然后重置 counter
这里的问题我理解只是重置全局 counter 的时候没有加锁，所以会丢失部分统计数据，但是不应该会如楼主所说“这里 abcde 就是相同的”

我 review 楼主的代码没有 review 出啥问题, 复现的成本对我来说有点高。
如果按照“这里 abcde 就是相同的” 这里来推断，因为 map 里面的 key 值都是唯一的，所以猜想可能楼主实际上使用的并不是 map[string]int, 而是自定义的结构体 /指针 /浮点数
然后多个不同的 id，虽然内容是一致的，但是 hashkey 不一致

[zkdfbb]

@makdon 不是的，我用的就是 map[string]int，统计的就是一个字符串 id 的访问次数，hashkey 是一样的
要是正常我就不问了，问就是因为不正常😂
我也感觉光看这段代码应该看不出什么问题，但是事实就是这么猝不及防的发生了，脑阔疼

[zkdfbb]

@Lpl 这个只是个简化的例子，实际上还需要处理一下，不过意思是一样的

[joesonw]

accessLog = Counter{data: make(map[string]int)}

没加锁

[zkdfbb]

@joesonw 是要加个锁，不过不是因为这个，我试过把这句注释掉，也是一样的

[joesonw]

是打印那的问题吗? 没换行? 现在的打印没有打印 map, 只打了 map 的长度.

[zkdfbb]

@joesonw 嗯，就是本来只要几百个 key 的，结果 map 长度可以上万，也打印过 map 本身，看到的就是有重复的 key