我目前正在学习 go,我的一些代码如下所示:
a, err := doA() if err != nil { return nil, err } b, err := doB(a) if err != nil { return nil, err } c, err := doC(b) if err != nil { return nil, err } ... and so on ...
这对我来说看起来有点错误,因为错误检查需要大部分行。有没有更好的方法来处理错误?我可以通过一些重构来避免这种情况吗?
更新:感谢您的所有回答。请注意,在我的示例中,doB 取决于 a,doC 取决于 b,依此类推。因此,大多数建议的重构在这种情况下不起作用。还有什么建议吗?
这是一个常见的抱怨,并且有多种答案。
以下是一些常见的:
1 - 还不错
这是对这些投诉的一种非常普遍的反应。事实上,你的代码中有几行额外的代码实际上并没有那么糟糕。这只是一点点廉价的打字,而且在阅读方面很容易处理。
2 - 这实际上是一件好事
这是基于这样一个事实,即输入和阅读这些额外的行是一个很好的提醒,实际上您的逻辑可能会在那时逃逸,并且您必须撤消您在它之前的行中放置的任何资源管理。这通常是与异常相比提出的,异常可以隐式地破坏逻辑流程,迫使开发人员始终牢记隐藏的错误路径。
3 - 使用恐慌/恢复
在某些特定情况下,您可以通过使用panic已知类型来避免某些工作,然后recover在您的包代码发布之前使用,将其转换为正确的错误并返回。这种技术最常用于展开递归逻辑,例如 (un) marshalers。
panic
recover
我个人尽量不要过度滥用这一点,因为我与第 1 点和第 2 点的关联更紧密。
4 - 稍微重新组织代码
在某些情况下,您可以稍微重新组织一下逻辑以避免重复。
作为一个简单的例子,这个:
err := doA() if err != nil { return err } err := doB() if err != nil { return err } return nil
也可以组织为:
err := doA() if err != nil { return err } return doB()
5 - 使用命名结果
有些人使用命名结果从 return 语句中去除 err 变量。不过,我建议不要这样做,因为它节省的代码很少,降低了代码的清晰度,并且当在救助返回语句之前定义一个或多个结果时,会使逻辑容易出现微妙的问题。
6 - 在 if 条件之前使用语句
正如 Tom Wilde 在下面的评论中很好地提醒的那样,ifGo 中的语句在条件之前接受一个简单的语句。所以你可以这样做:
if
if err := doA(); err != nil { return err }
这是一个很好的 Go 习语,并且经常使用。
在某些特定情况下,我宁愿避免以这种方式嵌入语句,只是为了清楚起见使其独立,但这是一件微妙且个人的事情。