err 错误使用场景
在同一个函数中会出现不止一个err乃至很多的err类型,需要注意使用最近的err类型,这个最近体现在两个方面:
- 最好err不重新定义,这样每次使用err,之前的err会被重新覆盖,也会强制每个err立刻使用。
- 在err出现在不同的作用域的时候,需要使用最近的作用域中的err。
错误的返回 Err
case 1
代码语言:javascript复制代码中的不应该使用整个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
}
case 2
代码语言:javascript复制// 首先代码中不应该在同一个作用域内命令两个err变量err与err2
// 在返回时也应该使用更近的err2,而不是更远的err作为返回
func NewTransOrderDaoInstance() error {
var err error
err2 := db.Create(order).Error
return nil,err
}
案例2: 覆盖 err 导致 事务不回滚
代码语言:javascript复制 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
}
如何解决
- 提交代码的时候,可以相互cr,看一下是不是存在
- 1、返回 err是否返回正确,或者 err 赋值错误;
- 2、关注是否重新定义了 err