如何修复‘被调用者返回nil错误类型，但调用者判断err!=nil仍为true’问题

问题根源 #

你遇到的是Go语言error接口的经典陷阱：当error接口变量的动态类型非nil，但动态值为nil时，err != nil的判断结果会是true，但调用该error的方法（比如Error()）时，会因底层值为nil触发nil pointer reference panic。

结合你的代码，问题出在这几个环节：

1. engine.Context().InsertOne()可能返回了一个“伪nil error”——也就是实现了error接口的具体类型的nil值（比如*xorm.MySQLError(nil)），此时err变量的动态类型是该具体错误类型，动态值是nil。
2. IsMySQLDuplicatedError函数没有先判断err是否为nil，直接进行类型断言或错误码检查，导致在err是伪nil时返回true，最终DbFunc返回了这个伪nil error。
3. 调用方判断err != nil为true，尝试调用err.Error()时，触发nil指针panic。

修复方案 #

方案1：在错误判断函数中先校验nil

修改IsMySQLDuplicatedError，先拦截nil情况再执行后续逻辑：

func IsMySQLDuplicatedError(err error) bool {
    // 先拦截nil，避免后续处理伪nil error
    if err == nil {
        return false
    }
    // 原有的错误判断逻辑，比如类型断言+错误码检查
    mysqlErr, ok := err.(*xorm.MySQLError) // 假设xorm的错误类型是*MySQLError
    return ok && mysqlErr.Code == 1062 // MySQL唯一键重复的错误码是1062
}

这样就能避免在err为nil时进入分支，DbFunc会直接返回真正的nil error，调用方的err != nil会判断为false，不会执行err.Error()。

方案2：在DbFunc中先判断err是否为nil

如果不想修改IsMySQLDuplicatedError，可以在DbFunc中先校验err非nil，再调用错误判断函数：

func DbFunc(ctx context.Context) (err error) {
   user := UserTab{UserId: 1}
   err = engine.Context(ctx).Table("user_tab").InsertOne(user)
   // 先确保err非nil，再判断错误类型
   if err != nil && IsMySQLDuplicatedError(err) {
     return err
   }
   return nil
}

这个方案同样能过滤掉伪nil error的情况，确保返回的要么是有效错误，要么是真正的nil。

方案3：修正伪nil error的源头

如果是xorm库返回了伪nil error，可以在接收err后手动修正：

import "reflect"

func DbFunc(ctx context.Context) (err error) {
   user := UserTab{UserId: 1}
   err = engine.Context(ctx).Table("user_tab").InsertOne(user)
   // 检查是否是伪nil error：接口非nil但底层值为nil
   if err != nil {
       val := reflect.ValueOf(err)
       if val.Kind() == reflect.Ptr && val.IsNil() {
           err = nil
       }
   }
   if err != nil && IsMySQLDuplicatedError(err) {
     return err
   }
   return nil
}

不过这种方式依赖反射，不够优雅，优先推荐前两种方案。

补充说明 #

Go语言中，接口类型的变量只有当动态类型和动态值都为nil时，err == nil才会返回true。只要其中一个不为nil，判断结果就是true，这就是你遇到的“明明err是nil但判断为非nil”的核心原因。

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