我们在使用Golang时,不可避免会遇到异常情况的处理,与Java、Python等语言不同的是,Go中并没有try...catch...这样的语句块,我们知道在Java中使用try...catch...这种模式不仅能分离的错误与返回值和参数,也提供了结构化处理异常的可能,通过面向对象的思想,我们可以自定义错误类、子类,它们又可以包装其他错误,确保错误上下文不会丢失。但是在Go中,异常是作为函数返回值,返回给调用方的,这个时候我们如何才能更好的处理异常呢?
对于异常的处理,我们应该把握三个原则:
func read(filePath string) (string, error) {
content ,err :=ioutil.ReadFile(filePath)
if err !=nil {
log.Printf("Read file err: %v", err)
return "", err
}
return string(content), nil
}
func parse(content string) (Employ, error) {
//解析文件得到Employ对象
}
func checkAttr(attr interface{}) error {
//校验对象属性
}
func commitEmployInfoFromFile(filePath string) error {
content, err :=read(filePath)
if err !=nil {
return errors.New("Read object file error")
}
employ, err :=parse(content)
if err !=nil {
return errors.New("Parse object content error")
}
if err = checkAttr(employ.Name); err !=nil {
returnerr
}
if err = checkAttr(employ.Age); err !=nil {
returnerr
}
if err = checkAttr(employ.Salary); err !=nil {
returnerr
}
returnnil
}
我们分析上面的代码,可以很明显看到read函数中违背了【不重复处理异常】的原则,虽然这里仅仅是打印,但是只要你向上抛异常,调用方很有可能再次打印,这就导致日志中存在大量重复信息,不便于分析。因为我们修改read函数:
func read(filePath string) (string, error) {
content ,err :=ioutil.ReadFile(filePath)
if err !=nil {
return "", err
}
return string(content), nil
}
再来看看这一部分代码,日志中仅仅打印了错误信息,但是缺少错误堆栈,这样非常不利于问题代码的定位。
content, err :=read(filePath)
if err !=nil {
return errors.New("Read object file error")
}
employ, err :=parse(content)
if err !=nil {
return errors.New("Parse object content error")
}
上面的代码还有一个问题,那就是错误信息都是简单的字符串信息,缺少上下文信息,比如:
errors.New("Read object file error")
我们只能知道是文件读取出错了,但无法得知是哪个文件有问题,因此我们最好加入文件信息到日志中。改良后的代码如下:
content, err :=read(filePath)
if err !=nil {
return fmt.Errorf("Read object file %v error: %v", filePath, err)
}
employ, err :=parse(content)
if err !=nil {
return fmt.Errorf("Parse object content error: %v", err)
}
最后,我们再看看这一段代码,这种写法非常常见,很多刚使用Golang的朋友都觉得非常头痛,由于Golang中没有throw或raise机制,所以会导致代码中使用大量if对错误进行处理,非常不优雅。
if err = checkAttr(employ.Name); err !=nil {
returnerr
}
if err = checkAttr(employ.Age); err !=nil {
returnerr
}
if err = checkAttr(employ.Salary); err !=nil {
returnerr
}
对于这类代码我们可以使用匿名函数进行简化,我们将checkAttr和err的判断封装在匿名函数check中,一旦某一次check出现error,则都不会在进行后续的属性校验。
check := func(attr interface{}){
if err !=nil{
return}
err =checkAttr(attr)
}
check(employ.Name)
check(employ.Age)
check(employ.Salary)
return err
当然,这种方式是还需要创建一个匿名函数以及一个error变量,这会让我们的commitEmployInfoFromFile函数显得不太干净,我们可以进一步优化:
type EmployChecker struct{
err error
}
func (c *EmployChecker) check(attr interface{}) {
if c.err ==nil {
c.err =checkAttr(attr)
}
}
func commitEmployInfoFromFile(filePath string) error {
content, err :=read(filePath)
if err !=nil {
return fmt.Errorf("Read object file %v error: %v", filePath, err)
}
employ, err :=parse(content)
if err !=nil {
return fmt.Errorf("Parse object content error: %v", err)
}
checker :=EmployChecker{}
checker.check(employ.Name)
checker.check(employ.Age)
checker.check(employ.Salary)
err =checker.err
returnerr
}
当然,这种方式是有一定局限性的,它只能在对于同一个业务对象的不断操作下可以简化错误处理,对于多个业务对象的话,还是得需要各种 if err != nil
的方式。
其实,对于Go的异常处理,我们不能说Golang不支持try catch,那它就不行,君不见try catch嵌套有多可怕,我们没必要一味追求代码的简洁,从而使用各种奇技淫巧去“优化”它,只要代码不冗余,清晰,简单就可以了。
版权说明 : 本文为转载文章, 版权归原作者所有 版权申明
原文链接 : https://www.cnblogs.com/alalazy/p/16849272.html
内容来源于网络,如有侵权,请联系作者删除!