今天說幾個我曾經在管理項目和團隊要求的基本編碼規范。實際執行下來成本比較低,長期堅持下來的確有助于項目的維護。
雖然是幾個非常基本的代碼規范,但我們只在團隊比較穩定的時候堅持下來過,后來隨著人員更迭,懂得都懂。 這里也不是吐槽誰的代碼習慣不好, 我也干過復制舊代碼過來就能用,媽呀真香,趕緊上線吧這種事情。
類型 | 規則 | 正確示例 | 錯誤示例 |
數據表名 | 使用SnakeCase 命名法多個單詞用下劃線 _ 分割使用單詞的復數形式命名 | vip_members | vipMembers vipMember vip_member |
數據表字段名 | 使用SnakeCase 命名法多個單詞用下劃線 _ 分割 | user_name | userName UserName _user_name |
數據表在代碼中的Model 名 | 使用CamelCase命名 單詞使用單數形式 | VipMember vipMember | VipMembers Members vip_member |
關于為啥數據表用復數,Model用單數,我的理解是Model代表的是這類東西,在英語里應該用復數。
下面說幾個Go語言里比較容易堅持執行下去且能有助于我們減少BUG的編碼規范。其他語言像Java的話,看阿里出的《阿里巴巴Java手冊》就可以,里面要求的比較細致。
函數名、參數名、參數類型、返回值類型要表達清楚要做的事情,避免產生歧義。這一條,感覺說簡單非常簡單,但是實際項目開發中,總是有不少人直接copy類似的函數,名字也不按使用場景去調整,讓看代碼的人就很難受。
錯誤案例:
func handleSomething(delay int) { for { // ... time.Sleep(time.Duration(delay) * time.Millisecond) }}poll(10) // delay參數定義成int 每次加的延遲是10毫秒還是10秒,還需要看poll函數的實現才知道
正確案例:
func handleSomething(delay time.Duration) { for { // ... time.Sleep(delay) }}poll(10 * time.Second) //delay參數定義成time.Duration類型, 調用時根據需求傳遞執行任務時要延遲的時間段 // 或者用參數名,明確告訴調用者,傳遞要延遲的秒數func handleSomething(delaySeconds int) { for { // ... time.Sleep(delaySeconds * time.Second) }}
在邏輯判斷里使用類似判斷屬性值是否等于某個硬編碼的值時會使得代碼晦澀難懂,應該使用更能從字面上看明白含義的常量來代替這些邏輯判斷里硬編碼的值。
錯誤案例
if prize.Type != 1 && prize.Type != 2{ ......}
正確案例:
const ( PRIZE_TYPE_COUPON = 1 PRIZE_TYPE_MONEY = 2 PRIZE_TYPE_VIPSCORE = 3)if prize.Type != PRIZE_TYPE_COUPON && prize.Type != PRIZE_TYPE_MONEY { ......}
注意程序的完全確定性,不要依賴init執行的順序實現功能,比如在后執行的init函數中對前面已初始化后的全局變量進行更改。
未初始化的map 默認值是nil , 可以對nil map進行讀取,但是寫入會直接panic:
var aMap map[string]stringaMap["foo"] = "bar" // panic
未初始化的slice,可以進行讀取和append操作,但不做初始化遇到接口中要返回的某個字段查不到數據直接返回,該字段在JSON里會用null表示而不是[], 有一定幾率造成前端錯誤。
type Person struct { Friends []string} func main() { var f1 []string f2 := make([]string, 0) json1, _ := json.Marshal(Person{f1}) json2, _ := json.Marshal(Person{f2}) fmt.Printf("%s/n", json1) fmt.Printf("%s/n", json2)} {"Friends":null} {"Friends":[]}
向未初始化的nil chan 寫入會造成goroutine阻塞,程序最終會死鎖:
func main() { //fmt.Println("Called heapAnalysis", heapAnalysis()) var achan chan struct{} achan <- struct{}{} // fatal error: all goroutines are asleep - deadlock! }
struct指針默認為nil , 未初始化直接使用,假如程序邏輯里是查不到數據就不對指針指向的struct進行復制,后續邏輯代碼再使用指針引用struct里的字段進行判斷時會因為嘗試對nil pointer 解引用直接panic
func QueryData(a int) (data *Data, err error) { // data 返回值直接使用時,默認是nil // 確保安全應該先對data 進行初始化 data = new(Data) data, err := querySomeData() if errors.IsNotFoundErr(err) { return; }} func main() { dataP, err := QueryData() if err != nil { return err } if dataP.State == STATE_ACTIVE { // 此處有可能嘗試對nil pointer進行解引用,會造成空指針問題程序崩潰。 // active logic }}
代碼應通過盡可能先處理錯誤情況/特殊情況并盡早返回或繼續循環來減少嵌套。減少嵌套多個級別的代碼的代碼量。
錯誤案例:
for _, v := range data { if v.F1 == 1 { v = process(v) if err := v.Call(); err == nil { v.Send() } else { return err } } else { log.Printf("Invalid v: %v", v) }}
正確案例:
for _, v := range data { if v.F1 != 1 { log.Printf("Invalid v: %v", v) continue } v = process(v) if err := v.Call(); err != nil { return err } v.Send()}
注意下面兩種寫法的直觀感受:
var a intif b { a = 100} else { a = 10} // 減少了不必要的else塊// 如果在 if 和 else 兩個分支中都設置了變量,則可以將其替換為單個 if。a := 10if b { a = 100}
在函數的參數中盡量不使用map[string]interface{}, map[string][string]這種類型的參數,IDE沒法幫助提示這些參數的內部結構,這讓其他人使用這個代碼時就會很苦惱,還需要先看看函數實現里具體用到了字典的哪些鍵。
針對比較復雜的代表一類事物的參數,應該先定義結構體,然后使用結構體指針或者結構體指針切片作為參數。
錯誤案例:
func AuthenticateUser(input map[string]interface{}) error { name, _ := input[name].(string) password, _ := input[name].(string) findUser(input["name"], input["password"]) ...}
正確案例:
type UserAuth struct{ Name string Age int32 Password string}func AuthenticateUser(input *UserAuth) error { findUser(input.Name, input.Password) ...}
本文鏈接:http://www.www897cc.com/showinfo-26-81250-0.html幾個祖傳代碼不遵守就想罵的代碼規范
聲明:本網頁內容旨在傳播知識,若有侵權等問題請及時與本網聯系,我們將在第一時間刪除處理。郵件:2376512515@qq.com
上一篇: 用代碼實現流水線部署,像詩一般優雅
下一篇: 我們一起聊聊如何在云上搭建無服務架構?