[原文链接](http://www.bugclosed.com/post/16) : http://www.bugclosed.com/post/16
## 背景
go语言中切片slice是方便且好用的强大数据结构，但是使用的时候需要注意，不然容易出问题，最近因为遇到了一个slice的使用问题，比较典型。
有一个功能需求，用户需要获取1-20的不重复随机序列。
## 逻辑实现
由于是需要固定的1-20共20个不同数字，所以直接定义好了唯一序列如下：
```
var(
originalNumbers = []uint32{1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12,13,14,15,16, 17, 18, 19, 20}
)
```
因为每个用户获取的数据的序列都需要随机打乱,实现的逻辑如下：
```
func shuffle(list []uint32) []uint32 {
n := len(list)
for i := n - 1; i > 0; i-- {
j := dist.Int63n(int64(i + 1))
list[i], list[j] = list[j], list[i]
}
return list
}
func getOriginalNumbers() []uint32{
return originalNumbers
}
func GetRandomNumbers(cardType int) []uint32 {
return shuffle(getOriginalNumbers())
}
```
## 问题暴露
通过仔细分析，从以上的逻辑其实是可以发现问题的，只是写代码的时候疏忽导致没有主要到潜在问题。运行的时候发现逻辑不正确，偶尔有用户得到的序列是有重复的数字。
从原始数据的初始化来看，数字是1-20初始化到slice里面的，绝对不会出现重复。仔细看了GetRandoNumbers和shuffle打乱逻辑是存在并发访问问题的。
首先originalNumbers是一个slice，参数传递slice时仅仅是传递的切片的指针，并非复制一份切片。所以在并发的情况下，每个用户的GetRandomNumbers都会获取到同一个slice地址。而shuffle函数会对得到切片数据进行写操作（数据打乱），当出现并发写问题的时候，数据发生错乱就不足为奇了。
## 问题解决
这个问题本质就是并发写问题，只需要将数据分离即可解决问题。
```
func getOriginalNumbers() []uint32{
tmp := make([]uint32, len(originalNumbers))
copy(tmp, shortDeck)
return tmp
}
```
## 总结
这是一个很典型的slice误用问题，slice是一个数据结构，他会指向底层真正的内存数据块，可以认为slice传递的是内存的指针。
有疑问加站长微信联系（非本文作者））
