第四次軟件工程作業--結對編程
阿新 • • 發佈:2018-04-07
動態 編程 ase 分享圖片 delete 版式 優先級 名稱 可能
代碼審查
代碼倉庫:https://coding.net/u/qq1537992316/p/pair-work-1/git/blob/729a0ce595aebd1e9422f285cc876e113fbb92f2/WangYHpair1.cpp?public=true
結對夥伴:王嶼涵
代碼審查表
功能模塊名稱 | 詞法分析器 | ||
---|---|---|---|
審查人 | 柴立軒 | 審查日期 | 2018.4.6 |
代碼名稱 | 詞法分析 | 代碼作者 | 王嶼涵 |
文件結構 | |||
重要性 | 審查項 | 結論 | |
頭文件和定義文件的名稱是否合理? | 是 | ||
頭文件和定義文件的目錄結構是否合理? | 是 | ||
版權和版本聲明是否完整? | 否 | ||
重要 | 頭文件是否使用了 ifndef/define/endif 預處理塊? | 否 | |
頭文件中是否只存放“聲明”而不存放“定義” | 是 | ||
程序的版式 | |||
重要性 | 審查項 | 結論 | |
空行是否得體? | 是 | ||
代碼行內的空格是否得體? | 是 | ||
長行拆分是否得體? | 是 | ||
“{” 和 “}” 是否各占一行並且對齊於同一列? | 否 | ||
重要 | 一行代碼是否只做一件事?如只定義一個變量,只寫一條語句。 | 否 | |
重要 | If、for、while、do等語句自占一行,不論執行語句多少都要加 “{}”。 | 否 | |
重要 | 在定義變量(或參數)時,是否將修飾符 * 和 & 緊靠變量名?註釋是否清晰並且必要? | 是 | |
重要 | 註釋是否有錯誤或者可能導致誤解? | 否 | |
重要 | 類結構的public, protected, private順序是否在所有的程序中保持一致? | ||
命名規則 | |||
重要性 | 審查項 | 結論 | |
重要 | 命名規則是否與所采用的操作系統或開發工具的風格保持一致? | 是 | |
標識符是否直觀且可以拼讀? | 是 | ||
標識符的長度應當符合“min-length && max-information”原則? | 是 | ||
重要 | 程序中是否出現相同的局部變量和全部變量? | 否 | |
類名、函數名、變量和參數、常量的書寫格式是否遵循一定的規則? | 是 | ||
靜態變量、全局變量、類的成員變量是否加前綴? | 否 | ||
表達式與基本語句 | |||
重要性 | 審查項 | 結論 | |
重要 | 如果代碼行中的運算符比較多,是否已經用括號清楚地確定表達式的操作順序? | 是 | |
是否編寫太復雜或者多用途的復合表達式? | 否 | ||
重要 | 是否將復合表達式與“真正的數學表達式”混淆 | 否 | |
重要 | 是否用隱含錯誤的方式寫if語句? 例如 | 否 | |
(1)將布爾變量直接與TRUE、FALSE或者1、0進行比較。 | 否 | ||
(2)將浮點變量用“==”或“!=”與任何數字比較。 | 否 | ||
(3)將指針變量用“==”或“!=”與NULL比較。 | 否 | ||
如果循環體內存在邏輯判斷,並且循環次數很大,是否已經將邏輯判斷移到循環體的外面? | |||
重要 | Case語句的結尾是否忘了加break? | ||
重要 | 是否忘記寫switch的default分支? | ||
重要 | 使用goto 語句時是否留下隱患? 例如跳過了某些對象的構造、變量的初始化、重要的計算等。 | ||
常量 | |||
重要性 | 審查項 | 結論 | |
是否使用含義直觀的常量來表示那些將在程序中多次出現的數字或字符串? | 是 | ||
在C++ 程序中,是否用const常量取代宏常量? | |||
重要 | 如果某一常量與其它常量密切相關,是否在定義中包含了這種關系? | ||
是否誤解了類中的const數據成員?因為const數據成員只在某個對象 | |||
生存期內是常量,而對於整個類而言卻是可變的。 | |||
函數設計 | |||
重要性 | 審查項 | 結論 | |
參數的書寫是否完整?不要貪圖省事只寫參數的類型而省略參數名字。 | 是 | ||
參數命名、順序是否合理? | 是 | ||
參數的個數是否太多? | 否 | ||
是否使用類型和數目不確定的參數? | 否 | ||
是否省略了函數返回值的類型? | 否 | ||
函數名字與返回值類型在語義上是否沖突? | 否 | ||
重要 | 是否將正常值和錯誤標誌混在一起返回?正常值應當用輸出參數獲得,而錯誤標誌用return語句返回。 | 否 | |
重要 | 在函數體的“入口處”,是否用assert對參數的有效性進行檢查? | ||
重要 | 使用濫用了assert? 例如混淆非法情況與錯誤情況,後者是必然存在的並且是一定要作出處理的。 | ||
重要 | return語句是否返回指向“棧內存”的“指針”或者“引用”? | 否 | |
是否使用const提高函數的健壯性?const可以強制保護函數的參數、返回值,甚至函數的定義體。“Use const whenever you need” | |||
內存管理 | |||
重要性 | 審查項 | 結論 | |
重要 | 用malloc或new申請內存之後,是否立即檢查指針值是否為NULL?(防止使用指針值為NULL的內存) | ||
重要 | 是否忘記為數組和動態內存賦初值?(防止將未被初始化的內存作為右值使用) | 否 | |
重要 | 數組或指針的下標是否越界? | 否 | |
重要 | 動態內存的申請與釋放是否配對?(防止內存泄漏) | ||
重要 | 是否有效地處理了“內存耗盡”問題? | ||
重要 | 是否修改“指向常量的指針”的內容? | ||
重要 | 是否出現野指針?例如(1)指針變量沒有被初始化;(2)用free或delete釋放了內存之後,忘記將指針設置為NULL。 | 否 | |
重要 | 是否將malloc/free 和 new/delete 混淆使用? | 否 | |
重要 | malloc語句是否正確無誤?例如字節數是否正確?類型轉換是否正 確? | 是 | |
重要 | 在創建與釋放動態對象數組時,new/delete的語句是否正確無誤? | ||
C++ 函數的高級特性 | |||
重要性 | 審查項 | 結論 | |
重要 | 是否違背了繼承和組合的規則? | ||
(1)若在邏輯上B是A的“一種”,並且A的所有功能和屬性對B而言都有意義,則允許B繼承A的功能和屬性。 | |||
(2)若在邏輯上A是B的“一部分”(a part of),則不允許B從A派生,而是要用A和其它東西組合出B。 | |||
其它常見問題 | |||
重要性 | 審查項 | 結論 | |
重要 | 數據類型問題: | ||
(1)變量的數據類型有錯誤嗎? | 否 | ||
(2)存在不同數據類型的賦值嗎? | 否 | ||
(3)存在不同數據類型的比較嗎? | 否 | ||
重要 | 變量值問題: | ||
(1)變量的初始化或缺省值有錯誤嗎? | 否 | ||
(2)變量發生上溢或下溢嗎? | 否 | ||
(3)變量的精度夠嗎? | 是 | ||
重要 | 邏輯判斷問題: | ||
(1)由於精度原因導致比較無效嗎? | 否 | ||
(2)表達式中的優先級有誤嗎? | 否 | ||
(3)邏輯判斷結果顛倒嗎? | 否 | ||
重要 | 循環問題: | ||
(1)循環終止條件不正確嗎? | 正確 | ||
(2)無法正常終止(死循環)嗎? | 否 | ||
(3)錯誤地修改循環變量嗎? | 否 | ||
(4)存在誤差累積嗎? | 否 | ||
重要 | 錯誤處理問題: | ||
(1)忘記進行錯誤處理嗎? | 否 | ||
(2)錯誤處理程序塊一直沒有機會被運行? | 否 | ||
(3)錯誤處理程序塊本身就有毛病嗎?如報告的錯誤與實際錯誤不一致,處理方式不正確等等。 | 否 | ||
(4)錯誤處理程序塊是“馬後炮”嗎?如在被它被調用之前軟件已經出錯。 | 否 | ||
重要 | 文件I/O問題: | ||
(1)對不存在的或者錯誤的文件進行操作嗎? | 否 | ||
(2)文件以不正確的方式打開嗎? | 否 | ||
(3)文件結束判斷不正確嗎? | 正確 | ||
(4)沒有正確地關閉文件嗎? | 否 |
評論
- 總體評價
詞法分析器是上學期編譯原理課程的課設題目。距離上次接觸這段代碼已經過去小半年了,但是當王嶼涵同學通過代碼倉庫發個我這段代碼時,上學期做課設的場景慢慢浮現在眼前,當時發生的故事歷歷在目。這段代碼,可以說是我們班大部分人的啟蒙代碼,無論是其完美的縮進,還是詳細的註釋,一字一符都體現著其作者鮮明的代碼風格,片刻瀏覽,整體的代碼思想很快就把握到了,這得益於其準確的代碼註釋。秉著服務同學的想法,在最後還有一段被註釋的代碼,這是老師當時的思考題,從中可以窺見王嶼涵同學的學習態度以及熱心幫助同學的美德。但這次的重點是代碼復查,對我來說就像是雞蛋裏挑骨頭,沒辦法,對不住了! - 代碼質量
其實總體我覺得還是ok的,但是嚴格按照代碼審查表來做,下面這幾行代碼就屬於“{” 和 “}” 是否各占一行並且對齊於同一列?
If、for、while、do等語句自占一行,不論執行語句多少都要加 “{}”
未達到要求。
如程序108-111行:
if (keyWord.count(temp))
cout << "<" << keyWord[temp] << ",->,";
else
cout << "<" << "1" << "," << temp << ">,";
靜態變量、全局變量、類的成員變量是否加前綴?
因為這個程序的代碼量不是很大,所以在變量命名時並未采用前綴反而使代碼更加容易閱讀,但隨著代碼量的增加,對變量的命名要更加規範,加前綴才能有效地提高代碼的可讀性。
- FINAl
這次作業,嗯,怎麽說,如果完成的不好的話,還是怪自己吧。畢竟,連一個像樣的代碼拿給同學都有點困難;畢竟,有好多同學都可以做的很好;畢竟,老師都已經很照顧學生的學習狀況了......兩天前,老師在講到結對編程時,幻燈片上貼出了下面這張照片
雖然想表達的意思是團隊合作之類的,不過我卻從中想到了整個計算機專業。層出不窮的技術就像各種各樣的樂器,而掌握一種技術亦如掌握一種樂器。想想如何學會吉他和認知音樂以及整個世界,那麽無論是腳下的困難還是未來的險阻都會迎刃而解吧。
第四次軟件工程作業--結對編程