我写了一个通用的FetchAll函数，想从数据库查询对应表的所有数据并返回结构体切片，但调用后返回的全是nil，代码如下：

func FetchAll(parameter interface{}) []interface{} {
    var model interface{}
    var resultArray []interface{}
    db := common.GetDB()
    rows, err := db.Model(parameter).Where(parameter).Rows()
    if err != nil {
        fmt.Print(err.Error())
        return nil
    }
    for rows.Next() {
        db.ScanRows(rows, &model)
        resultArray = append(resultArray, model)
    }
    fmt.Println(resultArray)
    return resultArray
}

调用方式：

c.JSON(200, FetchAll(&ProductImage{ProductID: productID}))

执行后输出是[<nil>,<nil>,<nil>]，请问这是哪里出问题了？

问题根源分析 #

嘿，这个问题其实很好解决！你写的var model interface{}只是声明了一个空接口变量，它并没有指向任何具体的ProductImage实例。当你调用db.ScanRows(rows, &model)时，GORM完全不知道要把数据库返回的数据映射到什么类型的对象上，自然没办法完成赋值，所以每次循环添加的都是nil。

修复方案：用反射创建目标类型实例 #

要实现通用的查询函数，我们需要借助Go的反射机制，在每次循环时创建一个传入参数类型的新实例，再把数据扫描进去。修改后的代码如下：

import "reflect"

func FetchAll(parameter interface{}) []interface{} {
    // 获取传入结构体的类型（因为parameter是指针，所以要取Elem()拿到结构体本身的类型）
    modelType := reflect.TypeOf(parameter).Elem()
    var resultArray []interface{}
    db := common.GetDB()
    
    rows, err := db.Model(parameter).Where(parameter).Rows()
    if err != nil {
        fmt.Print(err.Error())
        return nil
    }
    defer rows.Close() // 一定要记得关闭rows，避免数据库连接泄漏！
    
    for rows.Next() {
        // 创建一个目标类型的指针实例，这样GORM才能正确扫描数据
        model := reflect.New(modelType).Interface()
        
        // 扫描数据到这个实例里，同时处理扫描错误
        if scanErr := db.ScanRows(rows, model); scanErr != nil {
            fmt.Printf("扫描数据失败: %v\n", scanErr)
            continue // 可以根据业务需求选择是否终止循环
        }
        
        // 如果需要返回结构体值而非指针，就用Elem()取出指针指向的对象
        resultArray = append(resultArray, reflect.ValueOf(model).Elem().Interface())
        // 要是你需要保留指针类型，直接写append(resultArray, model)就行
    }
    
    // 别忘了处理rows的遍历错误
    if err := rows.Err(); err != nil {
        fmt.Printf("遍历行失败: %v\n", err)
    }
    
    fmt.Println(resultArray)
    return resultArray
}

关键修改点说明 #

• 反射创建实例：通过reflect.New(modelType)动态生成目标结构体的指针实例，让GORM明确知道要把数据映射到什么对象上。
• 关闭Rows资源：添加defer rows.Close()确保数据库连接被正确释放，避免内存泄漏。
• 完善错误处理：增加了ScanRows和rows.Err()的错误处理，避免单个错误导致整个函数崩溃。
• 灵活返回类型：可以选择返回结构体值或者指针，根据你的业务需求调整。

更优方案：用Go泛型实现类型安全的查询 #

如果你使用的是Go 1.18及以上版本，更推荐用泛型来实现这个功能——比反射更简洁，还能保证类型安全：

func FetchAll[T any](condition T) ([]T, error) {
    var result []T
    db := common.GetDB()
    
    err := db.Model(&condition).Where(&condition).Find(&result).Error
    return result, err
}

调用的时候也更直观：

images, err := FetchAll(ProductImage{ProductID: productID})
if err != nil {
    // 这里处理错误，比如返回500状态码
    c.JSON(500, gin.H{"error": err.Error()})
    return
}
c.JSON(200, images)

泛型版本不需要手动处理反射和实例创建，GORM会自动帮你完成映射，代码可读性和维护性都更高。

内容的提问来源于stack exchange，提问作者ErikDz1