在同一个函数中会出现不止一个err乃至很多的err类型,需要注意使用最近的err类型,这个最近体现在两个方面:
代码中的不应该使用整个func作用域内的err当做返回值,而应该使用最近作用域内的db.Error当做返回值.
// 代码中的不应该使用整个func作用域内的err当做返回值,而应该使用最近作用域内的db.Error当做返回值
func NewTransOrderDaoInstance() error {
var err error
db = db.Create(order)
if db.Error != nil {
logs.CtxError(ctx, "Create transOrder error, %v", err)
return err
}
return nil
}
// 首先代码中不应该在同一个作用域内命令两个err变量err与err2
// 在返回时也应该使用更近的err2,而不是更远的err作为返回
func NewTransOrderDaoInstance() error {
var err error
err2 := db.Create(order).Error
return nil,err
}
var err error
var isDuplicated bool
tx := model.BeginForShardingDBWithDbName(ctx, DBNAMe)
if tx == nil {
logs.CtxError(ctx, "Task BeginTrans failed")
return false, fmt.Errorf("Task BeginTrans failed")
}
defer func() {
if r := recover(); r != nil {
logs.Error("task panic: %+v", r)
err = fmt.Errorf("Task Trans panic")
err = EndTransaction(ctx, tx, err)
panic(r)
} else {
err = EndTransaction(ctx, tx, err)
}
}()
// 对 err 进行了重新定义里, 不再是 var err error 中的err ,如果这里 err 是异常,事务将不会进行回滚
_, isDuplicated, err := model.NewTpAccountOrderDaoInstance().CreateTpAccountOrder(ctx, tx, tpAccountOrder)
if err != nil {
return isDuplicated, err
}