您现在的位置是:网站首页> 编程资料编程资料

Go 代码规范错误处理示例经验总结_Golang_

2023-05-26 438人已围观

简介 Go 代码规范错误处理示例经验总结_Golang_

引言

编写代码应该要有极客追求,不要一味的只为了完成功能不加思索而噼里啪啦一顿操作,我认为应该要像一位设计者一样去设计代码完成功能,因为好的代码设计清晰可读易扩展、好修复、更少的学习成本。

因此我们应该学习并制定一些代码规范,如下是我学习Go语言中实战总结的一些经验,仅代表个人观点,大家可以互相讨论一下

一、相关联的声明放到一起

1、导包规范

// Bad import "logics/user_logic" import "logics/admin_logic" import "logics/goods_logic" // good import ( "logics/user_logic" "logics/admin_logic" "logics/goods_logic" ) 

分组导包

内置库

其他库

相关联库放在一起

// Bad import ( "fmt" "logics/user_logic" "logics/admin_logic" "strings" ) // Good import ( "fmt" "strings" // 逻辑处理 "logics/user_logic" "logics/admin_logic" // 数据库相关操作 "db/managers" "db/models" ) 

2、常量、变量、类型声明

在定义一些常量、变量与类型声明的时候,也是一样可以把相关联放到一起

常量

// Bad const YearMonthDay = "2006-01-02" const YearMonthDayHourMinSec = "2006-01-02 15:04:05" const DefaultTimeFmt = YearMonthDayHourMinSec // Good // TimeFormat 时间格式化 type TimeFormat string const ( YearMonthDay TimeFormat = "2006-01-02" // 年月日 yyyy-mm-dd YearMonthDayHourMinSec TimeFormat = "2006-01-02 15:04:05" // 年月年时分秒 yyyy-mm-dd HH:MM:SS DefaultTimeFmt TimeFormat = YearMonthDayHourMinSec // 默认时间格式化 ) 

变量

// Bad var querySQL string var queryParams []interface{} // Good var ( querySQL string queryParams []interface{} ) 

类型声明

// Bad type Area float64 type Volume float64 type Perimeter float64 // Good type ( Area float64 // 面积 Volume float64 // 体积 Perimeter float64 // 周长 ) 

枚举常量

// TaskAuditState 任务审核状态 type TaskAuditState int8 // Bad const TaskWaitHandle TaskAuditState = 0 // 待审核 const TaskSecondReview TaskAuditState = 1 // 复审 const TaskPass TaskAuditState = 2 // 通过 const TaskRefuse TaskAuditState = 3 // 拒绝 // Good const ( TaskWaitHandle TaskAuditState = iota // 待审核 TaskSecondReview // 复审 TaskPass // 通过 TaskRefuse // 拒绝 ) 

在进行Go开发时,指定一些非必选的入参时,不好区别空值是否有意义

如下 AuditState 是非必选参数,而 AuditState 在后端定义是 0 审核中、1复审、2通过、3拒绝,这都没什么问题,但框架解析参数时会把入参模型结构没有传值的参数设置成默认值,字符串类型是空串、数字类型是0等, 这样就会有问题如果前端传递参数的值是0、空值或者没有传递时,则无法判断是前端传递过来的还是框架默认设置的,导致后续逻辑不好写。

// QueryAuditTaskIn 查询任务入参 type QueryAuditTaskIn struct { TeamCode string `query:"team_code" validate:"required"` // 团队编码 TaskType enums.RiskType `query:"task_type" validate:"required"` // 任务类型 TagId int `query:"tag_id" validate:"required"` // 标签id AuditState constants.TaskAuditState `query:"audit_state"` // 审核状态 } 

解决办法就是设计时让前端不要传递一些空值,整型枚举常量设置成 从1开始,这样更好的处理后续逻辑。

// TaskAuditState 定义审核状态类型 type TaskAuditState int8 const ( TaskWaitHandle TaskAuditState = iota + 1 // 待审核 TaskSecondReview // 复审 TaskPass // 通过 TaskRefuse // 拒绝 ) 

二、Go错误处理

在Go开发中会出现好多if err != nil 的判断

尤其我在使用 manager 操作数据库时一调用方法就要处理错误,还要向上层依次传递

manager(数据库操作层) -> logic(逻辑层) -> api(接口层),每一层都要处理错误从而导致

一大堆的 if err != nil

// DelSensitive 删除内部敏感词 func (sl SensitiveLogic) DelSensitive(banWordId uint32) error { banWordManager, err := managers.NewBanWordsManager() if err != nil { return err } banWords, err := banWordManager.GetById(banWordId) if err != nil{ return err } if banWords == nil { return exceptions.NewBizError("屏蔽词不存在") } _, err = banWordManager.DeleteById(banWordId) if err != nil { return err } // 删除对应的敏感词前缀树,下一次文本审核任务进来的时候会重新构造敏感词前缀树 banWordsModel := banWords.(*models.BanWordsModel) sensitive.DelTrie(banWordsModel.Scene, banWordsModel.TeamCode) return nil } 

这样代码太不美观了如果改成如下看看

// DelSensitive 删除内部敏感词 func (sl SensitiveLogic) DelSensitive(banWordId uint32) { banWordManager := managers.NewBanWordsManager() banWords := banWordManager.GetById(banWordId) if banWords == nil { return } banWordManager.DeleteById(banWordId) // 删除对应的敏感词前缀树,下一次文本审核任务进来的时候会重新构造敏感词前缀树 banWordsModel := banWords.(*models.BanWordsModel) sensitive.DelTrie(banWordsModel.Scene, banWordsModel.TeamCode) } 

是不是美观多了,但这样出现出错误不能很好的定位到错误的位置以及日志记录,还会 panic 抛错误出来,导致协程终止执行,要等到 recover 恢复协程来中止 panic 造成的程序崩溃,从而影响性能。处理与不处理各有好处,我个人认为错误应该要处理但不要无脑的 if err != nil , 从而

可以在设计与规范上面来解决,对于一些严重的一定会导致程序奔溃的错误,可以自己统一设计错误类型,例如 数据库error 和 网络error 等,这种是很难避免的,即是避免了,系统也不能正常处理逻辑,因此对于这些 严重的错误可以手动 panic 然后在全局错误处理中记录日志信息,从而减少代码中的 if err != nil 的次数。如下

这样就不用一层一层传递 error ,但缺乏日志信息,虽然可以在上面的代码中打印日志信息,这样不太好,因此可以到全局错误那统一处理

这里是之前的想法可以考虑下,但像一些业务异常太多了就会频繁 panic,导致性能不佳以及后续的一些协程问题,所以我上文提到自己设计错误以及规范,什么错误、异常可以 panic 什么不可以,从而来减少 if err != nil。

其次就是在设计函数的来避免错误的出现

1、失败的原因只有一个时,不使用 error

我们看一个案例:

func (self *AgentContext) CheckHostType(host_type string) error { switch host_type { case "virtual_machine": return nil case "bare_metal": return nil } return errors.New("CheckHostType ERROR:" + host_type) } 

我们可以看出,该函数失败的原因只有一个,所以返回值的类型应该为 bool,而不是 error,重构一下代码:

func (self *AgentContext) IsValidHostType(hostType string) bool { return hostType == "virtual_machine" || hostType == "bare_metal" } 

说明:大多数情况,导致失败的原因不止一种,尤其是对 I/O 操作而言,用户需要了解更多的错误信息,这时的返回值类型不再是简单的 bool,而是 error。

2、没有失败时,不使用 error

error 在 Golang 中是如此的流行,以至于很多人设计函数时不管三七二十一都使用 error,即使没有一个失败原因。我们看一下示例代码:

func (self *CniParam) setTenantId() error { self.TenantId = self.PodNs return nil } 

对于上面的函数设计,就会有下面的调用代码:

err := self.setTenantId() if err != nil { // log // free resource return errors.New(...) } 

根据我们的正确姿势,重构一下代码:

func (self *CniParam) setTenantId() { self.TenantId = self.PodNs } 

于是调用代码变为:

self.setTenantId() 

3、错误值统一定义

很多人写代码时,到处 return errors.New(value),而错误 value 在表达同一个含义时也可能形式不同,比如“记录不存在”的错误 value 可能为:

errors.New("record is not existed.") errors.New("record is not exist!") errors.New("订单不存在") 

这使得相同的错误 value 撒在一大片代码里,当上层函数要对特定错误 value 进行统一处理时,需要漫游所有下层代码,以保证错误 value 统一,不幸的是有时会有漏网之鱼,而且这种方式严重阻碍了错误 value 的重构。

在每个业务系统中维护一个错误对象定义文件,一些公用的错误则封装到Go的公用库中

业务系统错误封装:

package exceptions // err_struct.go // OrderBizError 订单系统业务错误结构体 type OrderBizError struct { message string // 错误信息 code ErrorCode // 响应码 sysName string // 系统名称 } func NewOrderBizError(message string, errorCode ...ErrorCode) *OrderBizError { code := FailCode if len(errorCode) > 0 { code = errorCode[0] } return &OrderBizError{ code: code, message: message, sysName: "HuiYiMall—OrderSystem", // 可抽到微服务公用库中 } } // Code 状态码 func (b OrderBizError) Code() ErrorCode { return b.code } // Message 错误信息 func (b OrderBizError) Message() string { return b.message } // err_const.go // ErrorCode 定义错误code类型 type ErrorCode string const ( OrderTimeoutErrCode ErrorCode = "4000" // 订单超时 OrderPayFailErrCode ErrorCode = "4001" // 订单支付失败 ) var ( OrderTimeoutErr = NewOrderBizError("order timeout", OrderTimeoutErrCode) OrderPayFailErr = NewOrderBizError("order pay fail", OrderPayFailErrCode) ) 

返回错误信息给前端则返回状态码和信息,日志则记录全部的错误信息

Go公用库错误封装:

// err_struct.go // BizError 业务错误结构体 type BizError struct { message string // 错误信息 code ErrorCode // 响应码 } // Code 状态码 func (b BizError) Code() ErrorCode { return b.code } // Message 错误信息 func (b BizError) Message() string { return b.message } func NewBizError(message string, errorCode ...ErrorCode) *BizError { code := FailCode if len(errorCode) > 0 { code = errorCode[0] } return &BizError{ code: code, message: message, } } // err_const.go const ( SuccessCode ErrorCode = "0000" // 成功 FailCode ErrorCode = "0403" // 失败 AuthorizationCode ErrorCode = "0403" // 认证错误 // ... ) var ( Success = NewOrderBizError("Success", SuccessCode) FailErr = NewOrderBizError("Fail", FailCode) AuthorizationErr = NewOrderBizError("Authorization Error", AuthorizationCode) // ... ) 

其实每个业务系统的结构体可以继承公用的

// BizError 业务错误结构体 type BizError struct { message string // 错误信息 code
                
                

-六神源码网