代码复审训练
任務說明:
一、結對,找到一個伙伴進行結對;(在作業中標注自己的伙伴博客鏈接)。
http://www.cnblogs.com/crowndaisy/p/8594831.html
二、各自對自己的伙伴上周進行的“單元測試”練習所完成的代碼進行復審,形成“代碼復審檢查表”。
?
?
| ? | 代碼復審檢查表 | ? |
| ?概要部分 | 代碼符合需求和規格說明么? | 符合 |
| ?代碼設計是否考慮周全? | 否 | |
| 代碼可讀性如何? | 可讀性好 | |
| 有冗余的或重復的代碼嗎? | 沒有 | |
| 代碼的每一行都執行并檢查過了嗎? | 否 | |
| 設計規范部分 | 設計是否遵從已知的設計模式或項目中常用的模式? | 否 |
| 有沒有硬編碼或字符串存在? | 有 | |
| 代碼有沒有依賴于某一平臺? | 沒有 | |
| 有沒有無用的代碼可以清除? | 沒有 | |
| 代碼規范部分 | 修改的部分符合代碼標準和風格么(詳細條文略)? | 符合代碼標準 |
| 具體代碼部分 | 數據結構中有沒有用不到的元素? | 沒有 |
| 對于調用的外部函數,是否檢查了返回值? | 是 | |
| 效能 | 代碼的效能如何? | 效能好 |
| 循環中是否有明顯可優化的部分? | 沒有 | |
| 可讀性 | 代碼可讀性如何?有沒有足夠的注釋? | 可讀性高,但沒有足夠的注釋 |
| 可測試性 | 是否需要更新或創建新的單元測試? | 需要 |
| ? | 伙伴的代碼看起來很簡便,但是我還沒接觸python之前還是看不懂那段代碼的格式和含義。這就是沒有注釋而導致有些代碼沒看懂,我需要查找信息來了解它。而我寫的代碼中也沒有注釋會給別人帶來麻煩。看別人的代碼能放映出我代碼的缺點,有種當局者迷旁觀者清的感覺。 | |
| 代碼復審感想 | ||
| ? | ||
轉載于:https://www.cnblogs.com/janbin/p/8609934.html
總結
- 上一篇: 学会学习
- 下一篇: maven 工程依赖了某个jar包 但还