err 错误使用场景

2023-03-01 15:57:58 浏览数 (1)

err 错误使用场景

在同一个函数中会出现不止一个err乃至很多的err类型,需要注意使用最近的err类型,这个最近体现在两个方面:

  1. 最好err不重新定义,这样每次使用err,之前的err会被重新覆盖,也会强制每个err立刻使用。
  2. 在err出现在不同的作用域的时候,需要使用最近的作用域中的err。

错误的返回 Err

case 1

代码中的不应该使用整个func作用域内的err当做返回值,而应该使用最近作用域内的db.Error当做返回值.

代码语言:javascript复制
// 代码中的不应该使用整个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

0 人点赞