golangのdeferでのエラー処理方法

はじめに

golangci-lintで、こんなエラーでた。deferでエラーを返す関数をよんで、エラーを無視してたからみたい。

Error return value of `t.Disconnect` is not checked (errcheck)

修正前

こんな感じでやってた。 ローカルではtest通ってたけど、githubのCIでこけた。
もう、エラー無視してくれたらいいからって気分ではある。

func (t *hoge) hogeFunc(arg []byte) (hogeError error) {
    ごにょごにょ
    if err := t.Connect(); err != nil {
        return err
    }
    defer t.Disconnect()
    ごにょごにょ
}

func (t *hoge) Disconnect() error{
    ごにょごにょ

}

修正後

調べると、無名関数にして、名前付き戻り値にしたエラーに代入したらよいみたい。main()内なら、log.Fatal()とかよんだらいいのかも。

deferのあとで、return errしてるところもあったので、そのまま代入しちゃうとdeferのあとでエラーがでても握りつぶしちゃうから、t.Disconnect()がエラーのときだけ、代入した。deferのあとでエラーがあると、上書きしちゃうから、ほんとはappend的なことをしたほうがいいんやろうけど、サボった。

func (t *hoge) hogeFunc(arg []byte) (hogeError error) {
    ごにょごにょ
    if err := t.Connect(); err != nil {
        return err
    }
    defer func() {
        err := t.Disconnect()
        if err != nil {
            hogeError = err
        }
    }()
    ごにょごにょ
}

おわりに

deferで、エラーあったときどうなるんやろうと思ってたけど、CI通らなかったので真面目に調べて、多少すっきりした。
ただ、自信はないので、変なところあったら、指摘お願いしたいです。

参考

【Go】 deferに渡した関数内のエラーを呼び出し元に返す - Qiita
Deferred Cleanup, Checking Errors, and Potential Problems -- Bleve