关于golang中拷贝次数进而影响性能的问题

Question

先来看这么一个例子：

var aMap =map[int]int
aMap[1]=1

tmp:=aMap[1]
testMap1(tmp)
testMap2(aMap[1])

假设testMap1和testMap2的功能都是拿到参数之后打印一下参数值，那么到在他们打印前，testMap1完成这个动作进行了两次拷贝，第一次拷贝是tmp:=aMap[1]，第二次拷贝是tmp到参数值。testMap2只需要一次拷贝。那么单从代码性能来看，肯定不应该用tmp:=aMap[1]这种方式去调用testMap。

那么我们再看go1.9中的sync.Map，我需要获取map中的value需要调用其Load方法，那么这肯定会产生一次拷贝 即 tmp,_:=syncmap.Load(key)，而如果不用sync.Map，用普通Map，就可以像上面那样直接 map[key]取出来用，这样就少了一次拷贝。

不知道我这样想的对不对，碰到value是那种很大的struct，拷贝的开销应该不小吧，有没有什么好的解决这个拷贝开销的思路，如果用sync.Map的话

Answer 1

后经证明，两种方式没区别，编译器会进行优化

Answer 2

你要取map中的值，无论是原生map还是sync.Map，难免会遇到值拷贝，除非你存进去的是一个指针/地址，大的“对象”可以存指针进去。
此外，sync.Map底层使用的是interface{}作为键值对，因此效率不是很高，可以参考一下gf框架的gmap包，以下是基准测试结果：

john@johnstation:~/Workspace/Go/GOPATH/src/gitee.com/johng/gf/g/container/gmap$ go test *.go -bench=".*"
goos: linux
goarch: amd64
BenchmarkGmapSet-8            10000000           181 ns/op
BenchmarkSyncmapSet-8          5000000           366 ns/op
BenchmarkGmapGet-8            30000000            82.6 ns/op
BenchmarkSyncmapGet-8         20000000            95.7 ns/op
BenchmarkGmapRemove-8         20000000            69.8 ns/op
BenchmarkSyncmapRmove-8       20000000            93.6 ns/op
PASS
ok      command-line-arguments    27.950s

Answer 3

做优化时, 应该使用性能分析工具而不单纯靠主观想法.

比如, 这样做 map 与 sync.Map 的对比测试

// 代码仅做演示用途
// 执行 "go test -bench xx.go"
package main

import (
    "sync"
    "testing"
)

func Benchmark_map(b *testing.B) {
    m := make(map[int]int)
    for i := 0; i < b.N; i++ {
        m[1] = 1
    }
}

func Benchmark_sync_map(b *testing.B) {
    var m sync.Map
    for i := 0; i < b.N; i++ {
        m.Store(1, 1)
    }
}

参考

1. Profiling Go Programs, https://blog.golang.org/profi...
2. A Quick Guide to Go's Assembler, https://golang.org/doc/asm