小编典典

避免检查错误是否为零重复?

go

我目前正在学习 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,依此类推。因此,大多数建议的重构在这种情况下不起作用。还有什么建议吗?


阅读 198

收藏
2021-12-26

共1个答案

小编典典

这是一个常见的抱怨,并且有多种答案。

以下是一些常见的:

1 - 还不错

这是对这些投诉的一种非常普遍的反应。事实上,你的代码中有几行额外的代码实际上并没有那么糟糕。这只是一点点廉价的打字,而且在阅读方面很容易处理。

2 - 这实际上是一件好事

这是基于这样一个事实,即输入和阅读这些额外的行是一个很好的提醒,实际上您的逻辑可能会在那时逃逸,并且您必须撤消您在它之前的行中放置的任何资源管理。这通常是与异常相比提出的,异常可以隐式地破坏逻辑流程,迫使开发人员始终牢记隐藏的错误路径。

3 - 使用恐慌/恢复

在某些特定情况下,您可以通过使用panic已知类型来避免某些工作,然后recover在您的包代码发布之前使用,将其转换为正确的错误并返回。这种技术最常用于展开递归逻辑,例如 (un) marshalers。

我个人尽量不要过度滥用这一点,因为我与第 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 err := doA(); err != nil {
    return err
}

这是一个很好的 Go 习语,并且经常使用。

在某些特定情况下,我宁愿避免以这种方式嵌入语句,只是为了清楚起见使其独立,但这是一件微妙且个人的事情。

2021-12-26