【问题标题】:How to fix linter warning `Error return value is not checked`?如何修复 linter 警告“未检查错误返回值”?
【发布时间】: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 
}

【问题讨论】:

    标签: go linter


    【解决方案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(无论如何您都应该使用它):

        【讨论】:

          猜你喜欢
          • 1970-01-01
          • 1970-01-01
          • 2014-12-05
          • 1970-01-01
          • 2016-09-28
          • 1970-01-01
          • 1970-01-01
          • 1970-01-01
          • 2022-10-31
          相关资源
          最近更新 更多