关于errMap

Question

func VideoClearExecutor(dc dataChan) error {
	errMap := &sync.Map{}
	
	// ......
	
	errMap.Range(func(k, v interface{}) bool {
		err = v.(error)
		if err != nil {
			return false
		}
		return true
	})
	
	// ......
}

艾文西老师，我感觉把 errMap 挪到函数体外作为包级变量使用好一些，

否则 errMap.Range 每次调用的都是局部创建的新的 errMap ，

那么之前的 goroutine 就算有 error 也不是往当前这个 map 里面写，

那么 errMap.Range 这段逻辑也就没什么用了。

Answer 1

艾文西 2020-03-28 13:43:12

没看懂，之前的goroutine肯定不会往这个map里写啊。

这个map就是捕捉当前的err而已

• 提问者 remembers #1

  那之前的goroutine出的错就记录不了了，也没法终止runner.go中的select

• 艾文西 回复 提问者 remembers #2

  之前的goroutine自然会有之前自己的err记录啊。

• 提问者 remembers #3

  goroutine执行删除操作，局部的errMap.Range中很可能等不到err(因为goroutine很可能没来得及往errMap里写)，然后就执行下一轮的runner.go中的select了。那么由于errMap是局部变量，所以新一轮的errMap接受不到之前一轮goroutine中的err，所以最终停下来的情况是VideoClearDispatcher中的if len(res) == 0，但是errMap.Range 这段逻辑就感觉没什么用了

点开查看后面2条评论