這周,一個產品團隊內進行Go代碼評審時,得到了一個結論:所有的if err != nil的地方都應該輸出錯誤日志。然而,這種做法并不是最佳實踐,它存在一些問題。
首先,打印過多的錯誤日志會導致日志文件變得冗長和難以閱讀。其次,重復的錯誤信息會增加冗余。此外,每一層都打印錯誤日志,一旦錯誤信息設計不當,可能會導致上下文信息的丟失。
讓我們來看一個示例,說明為什么同時輸出錯誤日志和返回錯誤值會導致問題。假設我們有一個五層的Go函數調用棧,其中最底層的函數level4Function出現了一個錯誤:
package mainimport ( "fmt" "log")func main() { if err := topFunction(); err != nil { log.Printf("Error: %v", err) }}func topFunction() error { err := level1Function() if err != nil { log.Printf("topFunction: %v", err) return err } return nil}func level1Function() error { err := level2Function() if err != nil { log.Printf("level1Function: %v", err) return err } return nil}func level2Function() error { err := level3Function() if err != nil { log.Printf("level2Function: %v", err) return err } return nil}func level3Function() error { err := level4Function() if err != nil { log.Printf("level3Function: %v", err) return err } return nil}func level4Function() error { err := fmt.Errorf("something went wrong") log.Printf("level4Function: %v", err) return err}
在這個示例中,我們在每個函數中都輸出錯誤日志并返回錯誤值。我們運行一下這個程序:
$go run main.go 2024/04/14 23:10:05 level4Function: something went wrong2024/04/14 23:10:05 level3Function: something went wrong2024/04/14 23:10:05 level2Function: something went wrong2024/04/14 23:10:05 level1Function: something went wrong2024/04/14 23:10:05 topFunction: something went wrong2024/04/14 23:10:05 Error: something went wrong
當我們運行程序時,日志文件會出現重復的錯誤信息,并且上下文信息不易于進行鏈式追蹤,因為每個函數只打印了特定錯誤的信息,而沒有提供之前錯誤的上下文。
為了解決上述問題,我們需要采用一種更好的實踐方法。面向調用層次較深的函數調用棧,我們應該只在最頂層的函數中輸出錯誤日志,而在下層函數中返回錯誤值。但是,我們需要精心構造錯誤值,以形成基于wrapped error的錯誤鏈。
讓我們修改示例代碼,按照最佳實踐進行錯誤處理:
package mainimport ( "fmt" "log")func main() { if err := topFunction(); err != nil { log.Printf("Error: %v", err) }}func topFunction() error { err := level1Function() if err != nil { return fmt.Errorf("topFunction: %w", err) } return nil}func level1Function() error { err := level2Function() if err != nil { return fmt.Errorf("level1Function: %w", err) } return nil}func level2Function() error { err := level3Function() if err != nil { return fmt.Errorf("level2Function: %w", err) } return nil}func level3Function() error { err := level4Function() if err != nil { return fmt.Errorf("level3Function: %w", err) } return nil}func level4Function() error { err := fmt.Errorf("something went wrong") return fmt.Errorf("level4Function: %w", err)}
在這個修改后的示例中,我們在每個函數中使用fmt.Errorf+%w將錯誤包裝為一個wrapped error,并將前一層的錯誤作為參數傳遞。通過這種方式,我們構建了一個錯誤鏈,其中每個錯誤都包含了之前發生的錯誤上下文。在最頂層的main函數中,我們使用日志庫輸出錯誤日志,下面是示例程序的運行結果:
2024/04/14 23:12:16 Error: topFunction: level1Function: level2Function: level3Function: level4Function: something went wrong
我們看到:通過這種方法,我們避免了重復的錯誤日志,并保留了錯誤的上下文信息,快速定位了根因。當運行修改后的程序時,我們會看到日志文件中只打印了完整的錯誤鏈,而不是重復的錯誤信息。通過調用鏈和精心設計的錯誤上下文,我們還可以看到函數調用鏈,這使得錯誤的調試和處理變得更加方便和可靠。
在前面的示例中,我們展示了同時輸出錯誤日志和返回錯誤值的問題,并介紹了如何使用wrapped error來構建錯誤鏈。通過合理地處理錯誤,我們可以提高代碼的可讀性和可維護性,同時也有助于快速定位和解決問題。
總之,在編寫Go代碼時,請記住要么返回錯誤值,要么輸出日志,不要兩者都做。通過合理地處理錯誤,我們可以編寫出更可靠、更易于調試的代碼。
本文鏈接:http://www.tebozhan.com/showinfo-26-83619-0.html要么返回錯誤值,要么輸出日志,別兩樣都做
聲明:本網頁內容旨在傳播知識,若有侵權等問題請及時與本網聯系,我們將在第一時間刪除處理。郵件:2376512515@qq.com