最近在使用xorm,并使用了sql builder来构建sql查询没想到升级后原来可以使用的代码居然报错了。

0x00 代码

sql, args, _ := builder.Select("*").
		From("user").
		Where(builder.Eq{"uid": 1}).
		ToSQL()
res, err := orm.QueryString(sql, args...)

0x01 对比

发现xorm在0.6.3 和 0.6.4间做了改动,如图
0.6.3
xorm的sql builder

0.6.4
xorm的sql builder

原来如此,去掉了第一个参数,改为全部可变参数了,于是机智的把args...,改为了args
关于可变参数的问题,可以参考我的这篇文章

0x02 新错误

没想到编译没错,运行时报错了,提示

sql: converting argument $1 type: unsupported type []interface {}, a slice of interface

即类型错误。继续追踪代码,发现session_query.go里有生成sql的函数,代码如下:

func (session *Session) genQuerySQL(sqlorArgs ...interface{}) (string, []interface{}, error) {
	if len(sqlorArgs) > 0 {
		return sqlorArgs[0].(string), sqlorArgs[1:], nil
	}
    //省略
}

由于sqlorArgs是slice,而且builder.ToSql的args也是slice,那么sqlorArgs[1:]又创建了一个新的slice,就让最后返回的slice变成了二元slice了,所以出现了上面的类型错误。

0x03 解决办法

想了一下,其实我觉得上一个版本的函数签名更好,两个参数,一个负责接受sql语句,一个负责接收sql变量。给作者提了issue,或许作者有更好的解决方案。
下面是我的临时解决方法:

func (session *Session) genQuerySQL(sqlorArgs ...interface{}) (string, []interface{}, error) {
	if len(sqlorArgs) > 0 {
		if len(sqlorArgs) == 2 && reflect.TypeOf(sqlorArgs[1]).Kind() == reflect.Slice {
			return sqlorArgs[0].(string), sqlorArgs[1].([]interface{}), nil
		}
		return sqlorArgs[0].(string), sqlorArgs[1:], nil
	}
    //省略
}

作者已经更新,方法更加巧妙,增加了builder类型
xorm的sql builder

用法,直接传builder即可
xorm的sql builder

分类:

技术点:

相关文章: