
在 go 中使用反射动态创建结构体实例时,若在循环中复用同一指针变量并追加到切片,会导致切片中所有元素指向同一内存地址——最终所有项都显示为最后一次扫描的结果。
在 go 中使用反射动态创建结构体实例时,若在循环中复用同一指针变量并追加到切片,会导致切片中所有元素指向同一内存地址——最终所有项都显示为最后一次扫描的结果。
这个问题本质是 Go 指针与内存模型的经典陷阱:reflect.New(query.structType).Interface() 每次返回的是同一个底层对象的指针封装(interface{}),而你在循环外只调用了一次,之后反复用 bindQuery.Scan(value) 修改该指针所指向的内存内容,并将同一个 interface{} 值(内含相同指针)追加进切片。由于 interface{} 在此处承载的是 *T 类型,其底层数据被反复覆盖,最终切片中所有元素解引用后都得到最后一次扫描的值。
✅ 正确解法是在每次循环迭代中独立创建新实例,确保每个 value 指向不同的内存地址:
func (query Query) RetryingQuery() (results []interface{}) {
var q *gocql.Query
if query.values != nil {
q = c.Session.Query(query.query, query.values)
} else {
q = c.Session.Query(query.query)
}
bindQuery := cqlr.BindQuery(q)
// ✅ 关键修正:将 reflect.New 移入循环内部
for {
value := reflect.New(query.structType).Interface() // 每次迭代新建独立实例
if !bindQuery.Scan(value) {
break
}
// 此时 value 是 *T,但我们需要存储解引用后的值(即副本),避免指针共享
// 方案一:直接取 dereferenced 值(推荐,安全且语义清晰)
results = append(results, reflect.ValueOf(value).Elem().Interface())
}
return
}
? 为什么这样有效?
- reflect.New(query.structType) 每次调用都分配一块全新的堆内存,返回指向该内存的指针;
- reflect.ValueOf(value).Elem().Interface() 将 *T 转为 T(即值拷贝),确保追加的是结构体副本而非指针;
- 即使后续 results 中元素类型为 interface{},它们各自封装的是独立的结构体值,互不影响。
⚠️ 注意事项:
- 若业务逻辑必须保留指针语义(如需后续修改原对象),则应追加 *T 类型指针,但此时必须保证每个指针指向独立内存 —— 仍需在循环内调用 reflect.New,且不可对 results 中的指针做跨迭代复用;
- 避免在循环外声明 value 并在循环内仅调用 Scan():这等同于“一个指针反复写入”,是本问题根源;
- cqlr.BindQuery.Scan() 的参数必须是指针(*T),因此 reflect.New(...).Interface() 是必需的,但追加到切片的内容可以且应该选择值拷贝。
? 进阶建议:若性能敏感且结构体较大,可考虑预分配切片容量(results = make([]interface{}, 0, estimatedCount)),或定义具体返回类型(如 []*MyStruct)替代 []interface{},提升类型安全性与可读性。











