【转】Golang关于channel传递引用引发的坑

原文： https://studygolang.com/articles/12310/comment/17923

--------------------------------------------------------------------------------------

Time won't go back I won't turn back.

时光不会倒着走，我也不会再回头。

其实这个问题其实是出现在引用类型( 此处是slice )上, 这个是 slice 的数据结构，它很简单，一个指向真实 array 地址的指针 ptr ，slice 的长度 len 和容量 cap 。

结构图解1

每次cap改变的时候指向array内存的指针都在变化， 在实际使用中，我们最好事先预期好一个cap，这样在使用append的时候可以避免反复重新分配内存复制之前的数据，减少不必要的性能消耗。

现在上实例，来看看坑所在：

package main

import "fmt"
import "time"

func main() {
	ch := make(chan []byte, 10)
	go func() {
		for {
			select {
			case data := <-ch:
				fmt.Println(string(data))
			}
		}
	}()
	data := make([]byte, 0, 32)
	data = append(data, []byte("bbbbbbbbbb")...)
	ch <- data

	// fmt.Printf("%p\n", data)
	data = data[:0]
	// fmt.Printf("%p\n", data)

	data = append(data, []byte("aaa")...)
	ch <- data

	time.Sleep(time.Second * 5)
}

预测的运行结果：

bbbbbbbbbb

aaa

但是肯定是有坑的：

前面新起了一个协程来等待通道接受信息， 主进程继续执行， 当data传递给了通道之后， 立刻修改了data指向数组的值（第二次append）, 所以通道第一次接收的值就已经改变了， 因为我们传递的是引用，不是值类型。

解决方案呢就是加锁 或者新变量拷贝。