Go中如何将QueryRow扫描至已有map[string]interface{}？解决未定义错误

错误根源 #

你遇到的undefined:new_data错误是变量作用域问题：new_data是在if method == "POST"的代码块内部声明的，它的作用域仅限于这个代码块，当代码走到最后的return语句时，外部无法访问这个变量。

修复步骤 #

1. 提前在外部声明变量：把new_data的声明移到if代码块外面，确保整个函数都能访问到它。
2. 初始化默认值：如果请求不是POST方法，new_data也需要有合理的默认值（比如空map），避免返回nil或者未初始化的变量。

修复后的完整代码 #

func init() {
    lmhttp.ResponseHook["/login"] = Login()
}

func Login() func(c *lmhttp.Context, code int, data interface{}) (int, interface{}) {
    return func(c *lmhttp.Context, code int, data interface{}) (int, interface{}) {
        method := c.Request.Method
        var req loginReq
        // 提前声明并初始化new_data，确保作用域覆盖到return语句
        new_data := make(map[string]interface{})
        
        if method == "POST" {
            var des string
            if err := db.QueryRow("SELECT des FROM user WHERE email=?", req.Email).Scan(&des); err != nil {
                if err == sql.ErrNoRows {
                    fmt.Printf("查询失败：邮箱%s不存在\n", req.Email)
                } else {
                    fmt.Printf("查询出错：%v\n", err)
                }
                // 出错时直接返回，避免后续无效代码执行
                return code, map[string]interface{}{"data": data, "new_data": new_data}
            }
            
            // 无需序列化再反序列化，直接赋值到new_data即可
            new_data["des"] = des
            fmt.Printf("this is des: %s\n", des)
            fmt.Printf("this is new_data: %v\n", new_data)
        }
        
        return code, map[string]interface{}{"data": data, "new_data": new_data}
    }
}

额外优化建议 #

• 原代码中json.Marshal再json.Unmarshal的操作完全多余，直接把查询到的des赋值到new_data里就行，既节省性能又简化代码。
• 错误处理部分：原代码用fmt.Errorf但未做有效处理，建议用fmt.Printf输出明确日志，并且在出错时直接返回，避免后续不必要的代码执行。
• 变量命名：Go语言推荐小驼峰命名（比如newData），如果项目有特殊约定可以保留下划线风格。

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