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