如何修复 linter 警告“未检查错误返回值”？

Posted 2023-03-22

【中文标题】如何修复 linter 警告“未检查错误返回值”？

【英文标题】：How to fix linter warning `Error return value is not checked`?

【发布时间】：2019-07-08 13:40:56

【问题描述】：

我正在调用带有错误类型值的方法（代码示例中的 foo()）。我不在乎这个结果。什么是正确的代码风格编写方式？ Errcheck linter 让我检查这个错误。

//for example, same method may be called from imported entity
func foo() error 
   if err := someFunction(); err != nil 
       return err
   
   return nil


func process() 
   //linter doesn't like this
   foo()

   //this way leads to unused variable error
   err := foo()

   //is this clean way?
   _ = foo()

  return 

【参考方案1】：

这是惯用的方式：

err := foo()
if err != nil 
  // handle your error here

您不应忽略可能出现的错误。记录或打印到标准输出，但不要忽略它。

【参考方案2】：

是的，将其分配给通配符变量将是忽略错误的好方法。但强烈反对整个实践（忽略错误）。以下是“Effective Go”对此的评价：

有时您会看到为了忽略错误而丢弃错误值的代码；这是可怕的做法。始终检查错误返回；提供它们是有原因的。

   // Bad! This code will crash if path does not exist.
    fi, _ := os.Stat(path)
    if fi.IsDir() 
        fmt.Printf("%s is a directory\n", path)
    

【参考方案3】：

理想情况下，您应该处理错误，但如果它让您烦恼，您可以关闭 goland 中的 linter（无论如何您都应该使用它）：