C++|高質量代碼的規範要求與自我審查

1 文件結構

1.1 頭文件和定義文件的名稱是否合理?

1.2 頭文件和定義文件的目錄結構是否合理?

1.3 版權和版本聲明是否完整?

1.4 頭文件是否使用了 ifndef/define/endif 預處理塊?

1.5 頭文件中是否只存放“聲明”而不存放“定義”?
最後,如果大家如果在自學遇到困難,想找一個C++的學習環境,可以加入我們的C++學習圈,點擊我加入吧,會節約很多時間,減少很多在學習中遇到的難題。
C++|高質量代碼的規範要求與自我審查

2 程序的版式

2.1 空行是否得體?

2.2 代碼行內的空格是否得體?

2.3 長行拆分是否得體?

2.4 “{” 和 “}” 是否各佔一行並且對齊於同一列?

2.5 一行代碼是否只做一件事?如只定義一個變量,只寫一條語句。

2.6 If、for、while、do 等語句自佔一行,不論執行語句多少都要加“{}”。

2.7 在定義變量(或參數)時,是否將修飾符 * 和 & 緊靠變量名?

2.8 註釋是否清晰並且必要?

2.9 註釋是否有錯誤或者可能導致誤解?

2.10 類結構的 public, protected, private 順序是否在所有的程序中保持一致?

3 命名規則

3.1 命名規則是否與所採用的操作系統或開發工具的風格保持一致?

3.2 標識符是否直觀且可以拼讀?

3.3 標識符的長度應當符合“min-length && max-information”原則?

3.4 程序中是否出現相同的局部變量和全部變量?

3.5 類名、函數名、變量和參數、常量的書寫格式是否遵循一定的規則?

3.6 靜態變量、全局變量、類的成員變量是否加前綴?

4 表達式與基本語句

4.1 如果代碼行中的運算符比較多,是否已經用括號清楚地確定表達式的操作順序?

4.2 是否編寫太複雜或者多用途的複合表達式?

4.3 是否將複合表達式與“真正的數學表達式”混淆?

4.4 是否用隱含錯誤的方式寫 if 語句? 例如

(1) 將布爾變量直接與 TRUE、FALSE 或者 1、0 進行比較。
(2) 將浮點變量用“==”或“!=”與任何數字比較。
(3) 將指針變量用“==”或“!=”與 NULL 比較。
4.5 如果循環體內存在邏輯判斷,並且循環次數很大,是否已經將邏輯判斷移到循環體的外面?

4.6 Case 語句的結尾是否忘了加break?

4.7 是否忘記寫 switch 的 default 分支?

4.8 使用 goto 語句時是否留下隱患? 例如跳過了某些對象的構造、變量的初始化、重要的計算等。

5 常量

5.1 是否使用含義直觀的常量來表示那些將在程序中多次出現的數字或字符串?

5.2 在 C++ 程序中,是否用const 常量取代宏常量?

5.3 如果某一常量與其它常量密切相關,是否在定義中包含了這種關係?

5.4 是否誤解了類中的const 數據成員?因爲const 數據成員只在某個對象生存期內是常量,而對於整個類而言卻是可變的。

C++|高質量代碼的規範要求與自我審查

6 函數設計

6.1 參數的書寫是否完整?不要貪圖省事只寫參數的類型而省略參數名字。

6.2 參數命名、順序是否合理?

6.3 參數的個數是否太多?

6.4 是否使用類型和數目不確定的參數?

6.5 是否省略了函數返回值的類型?

6.6 函數名字與返回值類型在語義上是否衝突?

6.7 是否將正常值和錯誤標誌混在一起返回?正常值應當用輸出參數獲得,而錯誤標誌用 return 語句返回。

6.8 在函數體的“入口處”,是否用 assert 對參數的有效性進行檢查?

6.9 使用濫用了 assert? 例如混淆非法情況與錯誤情況,後者是必然存在的並且是一定要作出處理的。

6.10 return 語句是否返回指向“棧內存”的“指針”或者“引 用”?

6.11 是否使用const 提高函數的健壯性?const 可以強制保護函數的參數、返回值,甚至函數的定義體。“Use const whenever you need”

7 內存管理

7.1 用 malloc 或 new 申請內存之後,是否立即檢查指針值是否爲 NULL?(防止使用指針值爲 NULL 的內存)

7.2 是否忘記爲數組和動態內存賦初值?(防止將未被初始化的內存作爲右值使用)

7.3 數組或指針的下標是否越界?

7.4 動態內存的申請與釋放是否配對?(防止內存泄漏)

7.5 是否有效地處理了“內存耗盡”問題?

7.6 是否修改“指向常量的指針”的內容?

7.7 是否出現野指針?例如

I 指針變量沒有被初始化。
II 用 free 或delete 釋放了內存之後,忘記將指針設置爲 NULL。
7.8 是否將 malloc/free 和 new/delete 混淆使用?

7.9 malloc 語句是否正確無誤?例如字節數是否正確?類型轉換是否正確?

7.10 在創建與釋放動態對象數組時,new/delete 的語句是否正確無誤?

8 C++ 函數的高級特性

8.1 重載函數是否有二義性?

8.2 是否混淆了成員函數的重載、覆蓋與隱藏?

8.3 運算符的重載是否符合制定的編程規範?

8.4 是否濫用內聯函數?例如函數體內的代碼比較長,函數體內出現循環。

8.5 是否用內聯函數取代了宏代碼?

9 類的構造函數、析構函數和賦值函數

9.1 是否違背編程規範而讓 C++ 編譯器自動爲類產生四個缺省的函數:

(1)缺省的無參數構造函數;
(2)缺省的拷貝構造函數;
(3)缺省的析構函數;
(4)缺省的賦值函數。
9.2 構造函數中是否遺漏了某些初始化工作?

9.3 是否正確地使用構造函數的初始化表?

9.4 析構函數中是否遺漏了某些清除工作?

9.5 是否錯寫、錯用了拷貝構造函數和賦值函數?

9.6 賦值函數一般分四個步驟:

(1)檢查自賦值;
(2)釋放原有內存資源;
(3)分配新的內存資源,並複製內容;
(4)返回 *this。是否遺漏了重要步驟?
9.7 是否正確地編寫了派生類的構造函數、析構函數、賦值函數?注意事項:

I 派生類不可能繼承基類的構造函數、析構函數、賦值函數。
II 派生類的構造函數應在其初始化表裏調用基類的構造函數。
III 基類與派生類的析構函數應該爲虛(即加 virtual 關鍵字)。
IV 在編寫派生類的賦值函數時,注意不要忘記對基類的數據成員重新賦值。
10 類的高級特性

是否違背了繼承和組合的規則?

I 若在邏輯上 B 是 A 的“一種”,並且 A 的所有功能和屬性對B 而言都有意義,則允許B 繼承A 的功能和屬性。
II 若在邏輯上 A 是 B 的“一部分”(a part of),則不允許 B 從A 派生,而是要用A 和其它東西組合出 B。
11 其它常見問題

11.1 數據類型問題:

(1)變量的數據類型有錯誤嗎?
(2)存在不同數據類型的賦值嗎?
(3)存在不同數據類型的比較嗎?
11.2 變量值問題:

(1)變量的初始化或缺省值有錯誤嗎?
(2)變量發生上溢或下溢嗎?
(3)變量的精度夠嗎?
11.3 邏輯判斷問題:

(1)由於精度原因導致比較無效嗎?
(2)表達式中的優先級有誤嗎?
(3)邏輯判斷結果顛倒嗎?
11.4 循環問題:

(1)循環終止條件不正確嗎?
(2)無法正常終止(死循環)嗎?
(3)錯誤地修改循環變量嗎?
(4)存在誤差累積嗎?
11.5 錯誤處理問題:

(1)忘記進行錯誤處理嗎?
(2)錯誤處理程序塊一直沒有機會被運行?
(3)錯誤處理程序塊本身就有毛病嗎?如報告的錯誤與實際錯誤不一致,處理方式不正確等等。
(4)錯誤處理程序塊是“馬後炮”嗎?如在被它被調用之前軟件已經出錯。
11.6 文件 I/O 問題:

(1)對不存在的或者錯誤的文件進行操作嗎?
(2)文件以不正確的方式打開嗎?
(3)文件結束判斷不正確嗎?
(4)沒有正確地關閉文件嗎?

發表評論
所有評論
還沒有人評論,想成為第一個評論的人麼? 請在上方評論欄輸入並且點擊發布.
相關文章