一、代碼審查表

參考博客：c/c++代碼審查
結(jié)對伙伴：何濤
代碼地址：Coding.net

功能模塊名稱	將賦值語句轉(zhuǎn)化為四元式
審查人	劉子君	審查日期	2018.4.5
代碼名稱	四元式實驗	代碼作者	何濤
文件結(jié)構(gòu)
重要性? ? ??	審查項	結(jié)論
? ?? ?? ?? ?? ??	頭文件和定義文件的名稱是否合理？	是
	頭文件和定義文件的目錄結(jié)構(gòu)是否合理？	是
	版權(quán)和版本聲明是否完整？	否
重要	頭文件是否使用了 ifndef/define/endif 預(yù)處理塊?	否
	頭文件中是否只存放“聲明”而不存放“定義”	是
程序的版式
重要性? ? ??	審查項	結(jié)論
	空行是否得體？	是
	代碼行內(nèi)的空格是否得體？	否
	長行拆分是否得體？	是
	“{” 和 “}” 是否各占一行并且對齊于同一列？	是
重要	一行代碼是否只做一件事？如只定義一個變量，只寫一條語句。	否
重要	If、for、while、do等語句自占一行，不論執(zhí)行語句多少都要加 “{}”。	是
重要	在定義變量（或參數(shù)）時，是否將修飾符 * 和 ＆ 緊靠變量名？注釋是否清晰并且必要？	否
重要	注釋是否有錯誤或者可能導(dǎo)致誤解？	無
重要	類結(jié)構(gòu)的public, protected, private順序是否在所有的程序中保持一致？	無
命名規(guī)則?
重要性? ? ??	審查項	結(jié)論
重要	命名規(guī)則是否與所采用的操作系統(tǒng)或開發(fā)工具的風(fēng)格保持一致？	是
	標(biāo)識符是否直觀且可以拼讀？	否
	標(biāo)識符的長度應(yīng)當(dāng)符合“min-length && max-information”原則？	否
重要	程序中是否出現(xiàn)相同的局部變量和全部變量？	是
	類名、函數(shù)名、變量和參數(shù)、常量的書寫格式是否遵循一定的規(guī)則？	是
	靜態(tài)變量、全局變量、類的成員變量是否加前綴？	否
表達(dá)式與基本語句?
重要性? ? ??	審查項	結(jié)論
重要	如果代碼行中的運算符比較多，是否已經(jīng)用括號清楚地確定表達(dá)式的操作順序？	是
	是否編寫太復(fù)雜或者多用途的復(fù)合表達(dá)式？	否
重要	是否將復(fù)合表達(dá)式與“真正的數(shù)學(xué)表達(dá)式”混淆？	否
重要	是否用隱含錯誤的方式寫if語句? 例如
	（1）將布爾變量直接與TRUE、FALSE或者1、0進行比較。	無
	（2）將浮點變量用“==”或“！=”與任何數(shù)字比較。	否
	（3）將指針變量用“==”或“！=”與NULL比較。	否
	如果循環(huán)體內(nèi)存在邏輯判斷，并且循環(huán)次數(shù)很大，是否已經(jīng)將邏輯判
	斷移到循環(huán)體的外面？	否
重要	Case語句的結(jié)尾是否忘了加break？	無
重要	是否忘記寫switch的default分支？	無
重要	使用goto 語句時是否留下隱患? 例如跳過了某些對象的構(gòu)造、變量的初始化、重要的計算等。	無
常量?
重要性? ? ??	審查項	結(jié)論
	是否使用含義直觀的常量來表示那些將在程序中多次出現(xiàn)的數(shù)字或字符串？	是
	在C++ 程序中，是否用const常量取代宏常量？	無
重要	如果某一常量與其它常量密切相關(guān)，是否在定義中包含了這種關(guān)系？	否
	是否誤解了類中的const數(shù)據(jù)成員？因為const數(shù)據(jù)成員只在某個對象	無
	生存期內(nèi)是常量，而對于整個類而言卻是可變的。	否
函數(shù)設(shè)計?
重要性? ? ??	審查項	結(jié)論
	參數(shù)的書寫是否完整？不要貪圖省事只寫參數(shù)的類型而省略參數(shù)名字。	否
	參數(shù)命名、順序是否合理？	否
	參數(shù)的個數(shù)是否太多？	否
	是否使用類型和數(shù)目不確定的參數(shù)？	否
	是否省略了函數(shù)返回值的類型？	否
	函數(shù)名字與返回值類型在語義上是否沖突？	否
重要	是否將正常值和錯誤標(biāo)志混在一起返回？正常值應(yīng)當(dāng)用輸出參數(shù)獲得，而錯誤標(biāo)志用return語句返回。	否
重要	在函數(shù)體的“入口處”，是否用assert對參數(shù)的有效性進行檢查？	否
重要	使用濫用了assert？ 例如混淆非法情況與錯誤情況，后者是必然存在的并且是一定要作出處理的。	無
重要	return語句是否返回指向“棧內(nèi)存”的“指針”或者“引用”？	否
	是否使用const提高函數(shù)的健壯性？const可以強制保護函數(shù)的參數(shù)、返回值，甚至函數(shù)的定義體。“Use const whenever you need”	否
內(nèi)存管理?
重要性? ? ??	審查項	結(jié)論
重要	用malloc或new申請內(nèi)存之后，是否立即檢查指針值是否為NULL？（防止使用指針值為NULL的內(nèi)存）	無
重要	是否忘記為數(shù)組和動態(tài)內(nèi)存賦初值？（防止將未被初始化的內(nèi)存作為右值使用）	是
重要	數(shù)組或指針的下標(biāo)是否越界？	否
重要	動態(tài)內(nèi)存的申請與釋放是否配對？（防止內(nèi)存泄漏）	無
重要	是否有效地處理了“內(nèi)存耗盡”問題？	否
重要	是否修改“指向常量的指針”的內(nèi)容？	否
重要	是否出現(xiàn)野指針？例如（1）指針變量沒有被初始化;（2）用free或delete釋放了內(nèi)存之后，忘記將指針設(shè)置為NULL。	否
重要	是否將malloc/free 和 new/delete 混淆使用？	無
重要	malloc語句是否正確無誤？例如字節(jié)數(shù)是否正確？類型轉(zhuǎn)換是否正 確？	無
重要	在創(chuàng)建與釋放動態(tài)對象數(shù)組時，new/delete的語句是否正確無誤？	無
C++ 函數(shù)的高級特性?
重要性? ? ??	審查項	結(jié)論
	重載函數(shù)是否有二義性？	否
重要	是否混淆了成員函數(shù)的重載、覆蓋與隱藏？	否
	運算符的重載是否符合制定的編程規(guī)范？	否
	是否濫用內(nèi)聯(lián)函數(shù)？例如函數(shù)體內(nèi)的代碼比較長，函數(shù)體內(nèi)出現(xiàn)循環(huán)。	是
重要	是否用內(nèi)聯(lián)函數(shù)取代了宏代碼？	否
類的構(gòu)造函數(shù)、析構(gòu)函數(shù)和賦值函數(shù)
重要性? ? ??	審查項	結(jié)論
重要	是否違背編程規(guī)范而讓C++ 編譯器自動為類產(chǎn)生四個缺省的函數(shù)：
	（1）缺省的無參數(shù)構(gòu)造函數(shù)；
	（2）缺省的拷貝構(gòu)造函數(shù)；
	（3）缺省的析構(gòu)函數(shù)；
	（4）缺省的賦值函數(shù)。
重要	構(gòu)造函數(shù)中是否遺漏了某些初始化工作？	無
重要	是否正確地使用構(gòu)造函數(shù)的初始化表？	無
重要	析構(gòu)函數(shù)中是否遺漏了某些清除工作？	無
	是否錯寫、錯用了拷貝構(gòu)造函數(shù)和賦值函數(shù)？	無
重要	賦值函數(shù)一般分四個步驟：	無
	（1）檢查自賦值；
	（2）釋放原有內(nèi)存資源；
	（3）分配新的內(nèi)存資源，并復(fù)制內(nèi)容；
	（4）返回 *this。是否遺漏了重要步驟？????????	無
重要	是否正確地編寫了派生類的構(gòu)造函數(shù)、析構(gòu)函數(shù)、賦值函數(shù)？	無
	注意事項：
	（1）派生類不可能繼承基類的構(gòu)造函數(shù)、析構(gòu)函數(shù)、賦值函數(shù)。
	（2）派生類的構(gòu)造函數(shù)應(yīng)在其初始化表里調(diào)用基類的構(gòu)造函數(shù)。
	（3）基類與派生類的析構(gòu)函數(shù)應(yīng)該為虛（即加virtual關(guān)鍵字）。
	（4）在編寫派生類的賦值函數(shù)時，注意不要忘記對基類的數(shù)據(jù)成員重新賦值
類的高級特性
重要性? ? ??	審查項	結(jié)論
重要	是否違背了繼承和組合的規(guī)則？	無
	（1）若在邏輯上B是A的“一種”，并且A的所有功能和屬性對B而言都有意義，則允許B繼承A的功能和屬性。
	（2）若在邏輯上A是B的“一部分”（a part of），則不允許B從A派生，而是要用A和其它東西組合出B。
其它常見問題?
重要性? ? ??	審查項	結(jié)論
重要	數(shù)據(jù)類型問題：
	（１）變量的數(shù)據(jù)類型有錯誤嗎？	否
	（２）存在不同數(shù)據(jù)類型的賦值嗎？	是
	（３）存在不同數(shù)據(jù)類型的比較嗎？	否
重要	變量值問題：
	（１）變量的初始化或缺省值有錯誤嗎？	否
	（２）變量發(fā)生上溢或下溢嗎？	否
	（３）變量的精度夠嗎？????????	是
重要	邏輯判斷問題：
	（１）由于精度原因?qū)е卤容^無效嗎？	否
	（２）表達(dá)式中的優(yōu)先級有誤嗎？	否
	（３）邏輯判斷結(jié)果顛倒嗎？????????	否
重要	循環(huán)問題：
	（１）循環(huán)終止條件不正確嗎？	否
	（２）無法正常終止（死循環(huán)）嗎？	否
	（３）錯誤地修改循環(huán)變量嗎？	否
	（４）存在誤差累積嗎？????????	否
重要	錯誤處理問題：
	（１）忘記進行錯誤處理嗎？	否
	（２）錯誤處理程序塊一直沒有機會被運行？	否
	（３）錯誤處理程序塊本身就有毛病嗎？如報告的錯誤與實際錯誤不一致，處理方式不正確等等。	否
	（４）錯誤處理程序塊是“馬后炮”嗎？如在被它被調(diào)用之前軟件已經(jīng)出錯。	否
重要	文件I/O問題：
	（１）對不存在的或者錯誤的文件進行操作嗎？	無
	（２）文件以不正確的方式打開嗎？	無
	（３）文件結(jié)束判斷不正確嗎？	無
	（４）沒有正確地關(guān)閉文件嗎?	無

二、代碼審查

在寫這個作業(yè)的時候，通過填寫代碼審查表，了解到編程時有那么多需要注意的地方，讓我對它有了新的認(rèn)識，也才了解到自己編寫的程序還要注意版權(quán)信息的填寫。因為平時的作業(yè)也都比較簡單，而且有的是在網(wǎng)上找的，所有根本沒有想過這個問題。填完表后，下面是我對何濤同學(xué)程序的一些看法。

首先，何濤同學(xué)的代碼是將賦值語句轉(zhuǎn)化成四元式，他運用了結(jié)構(gòu)體和幾個函數(shù)來解決這個問題，但是函數(shù)名過于簡單都是用一個字母表示，并不能大致分析出函數(shù)的功能，而且沒有注釋，復(fù)審的時候很麻煩得一點點分析，在編寫函數(shù)的時候沒有參數(shù)，所以不能直接看出這個函數(shù)是對什么變量進行計算。在他的整個程序中用的都是全局變量，沒有在函數(shù)中單獨定義，可能會容易出現(xiàn)錯誤。

其次，看了一下格式，if后面并不是都有‘{’和‘}’，操作符兩邊沒有空格，這樣會更容易閱讀，其它的都較為標(biāo)準(zhǔn)。總體來說，格式還是不錯的，該對齊的都對齊了，每個功能塊還是很直接的。

這個程序最大的問題就是沒有注釋了，這要不是知道這個程序是編譯原理的實驗三，恐怕連程序是干什么的都不知道。現(xiàn)在想想自己以前編的程序也沒有注釋，還好只有自己看，這要是別人看的話估計也很費勁。通過這次的審查別人的代碼，讓我也意識到了自己的很多問題，缺少注釋就是個很大的問題了，以后要注意。還有就是格式，要讓別人看的很舒服，畢竟你的程序別人也會看的。

轉(zhuǎn)載于:https://www.cnblogs.com/lzj1997/p/8718969.html

總結(jié)

以上是生活随笔為你收集整理的代码审查的全部內(nèi)容，希望文章能夠幫你解決所遇到的問題。

如果覺得生活随笔網(wǎng)站內(nèi)容還不錯，歡迎將生活随笔推薦給好友。