進行代碼復審訓練

20180320作業2:進行代碼復審訓練

任務說明:

一、結對,找到一個伙伴進行結對;(在作業中標注自己的伙伴博客鏈接)

王小梅:?http://www.cnblogs.com/CURRY-30/p/8570290.html

二、各自對自己的伙伴上周進行的“單元測試”練習所完成的代碼進行復審,形成“代碼復審檢查表”。

代碼復審檢查表

概要部分

代碼符合需求和規格說明么?

基本符合。

代碼設計是否考慮周全?

不是很周全。

代碼可讀性如何?

較簡單。

有冗余的或重復的代碼嗎?

沒有。

代碼的每一行都執行并檢查過了嗎?

是的。

設計規范部分

設計是否遵從已知的設計模式或項目中常用的模式?

是的,

有沒有硬編碼或字符串存在?

沒有。

代碼有沒有依賴于某一平臺?

沒有。

有沒有無用的代碼可以清除?

沒有。

代碼規范部分

修改的部分符合代碼標準和風格么?

代碼基本符合規范標準和風格。

具體代碼部分

數據結構中有沒有用不到的元素?

沒有。

對于調用的外部函數,是否檢查了返回值?

是的。

效能

代碼的效能如何?

基本達到了任務要求。

循環中是否有明顯可優化的部分?

沒有明顯可優化部分。

可讀性

代碼可讀性如何?有沒有足夠的注釋?

可讀性中等。

可測試性

是否需要更新或創建新的單元測試?

需要,代碼存在部分問題,測試時會報錯但仍能顯示結果。


8、代碼復審感想
?

個人認為,代碼復審這一步在這個流程中是非常重要的,通過復審可以找出很多的錯誤,也可以完善之前的漏洞。而且復審自己也可以學到很多知識,通過對別人代碼的檢查,從總學到了別人對代碼的理解,取其精華去其糟粕。而且檢查出錯誤的同時,也可以給自己一個教訓,以免下次不會犯同樣的錯誤。

轉載于:https://www.cnblogs.com/ftfive/p/8638707.html

本文來自互聯網用戶投稿,該文觀點僅代表作者本人,不代表本站立場。本站僅提供信息存儲空間服務,不擁有所有權,不承擔相關法律責任。
如若轉載,請注明出處:http://www.pswp.cn/news/453185.shtml
繁體地址,請注明出處:http://hk.pswp.cn/news/453185.shtml
英文地址,請注明出處:http://en.pswp.cn/news/453185.shtml

如若內容造成侵權/違法違規/事實不符,請聯系多彩編程網進行投訴反饋email:809451989@qq.com,一經查實,立即刪除!

相關文章

freeradius 3.0 時間限制_創意營銷3.0新模式下,易企秀要成為中國的Adobe

近幾年,隨著大數據和人工智能技術的發展,智能化、程序化營銷在國內獲得高速發展。從以創意內容、提升效率的工具到現在驅動企業數字化轉型的智能營銷,營銷云在國內的熱度與成熟度不斷提升。營銷云起源于“Enterprise Marketing Software Suit…

用計算機三級處理文件,【題目】計算機三級題目,獻給為計算機三級掙扎的同學們...

該樓層疑似違規已被系統折疊 隱藏此樓查看此樓(165)Internet的郵件傳輸協議標準為_______。A)SNMP B)SMTP C)FTP D)TELNET答 案:B知識點:瀏覽器、電了郵件及Web服務器的安全特性評 析:SNMP即簡單網絡管理協議;SMTP即簡單郵件傳輸協…

匈牙利命名法的優缺點

匈牙利約定優點匈牙利約定與其它命名約定一樣,擁有由命名約定所帶來的一切共同優點。由于有這樣多的標準名稱,因此在任何一個單個子程序或程序中要特殊記憶的名字是非常少的。匈牙利約定完全可以在不同項目中采用。匈牙利約定可以使得在命名中容易產生定…

搶占式和非搶占式的進程調度

非搶占式(Nonpreemptive) 讓進程運行直到結束或阻塞的調度方式 容易實現 適合專用系統,不適合通用系統 搶占式(Preemptive) 允許將邏輯上可繼續運行的在運行過程暫停的調度方式 可防止單一進程長時間獨占…

圖形學基礎知識

本篇主要給大家介紹圖形學基礎知識,了解Unity圖像渲染機制,以及圖像渲染管線流程。 主要是因為伴隨著VR/AR的飛速發展,為了滿足VR高清高幀率的極限渲染,著色器編程(Shader)也成為了Unity程序開發人員的必備…

12種排序算法

https://blog.csdn.net/love_gaohz/article/details/46673907 轉載于:https://www.cnblogs.com/kongk/p/8641077.html

計算機應用基礎精品課程申報表,《計算機應用基礎》精品課程申報書(修改意見)...

4-3教學條件(含教材使用與建設;促進學生主動學習的擴充性資料使用情況;配套實驗教材的教學效果;實踐性教學環境;網絡教學環境(網絡平臺,網絡課件))突出的地方用紅色表示。1.教材建設本課程目前使用的教材是…

用博客見證自己的成長

感謝CSDN博客,給我提供了一個學習交流的平臺,自從開通博客以來,感覺自己的生活一下充實了很多。通過寫博客來積累知識的同時,也給生活帶來的很多的喜悅。 第一次發表博客;第一次有訪客;第一次得到博客勛章&…

調度隊列模型

調度隊列模型及準則 1 僅有進程調度的調度隊列模型: 每個進程在執行時都可能出現以下三種情況: (1) 任務在給定的時間片內已經完成,該進程便在釋放處理機后進入完成狀態 (2) 任務在本次分得的時間片內尚未完成,OS便將該任務再放入…

linux基礎命令(-)

最近在學linux,老師課堂上講的估計下課就忘了,現整理下來以備不時之需。 PS1x //修改終端提示符su root //切換用戶ssh root192.168.116.129 //加密遠程連接 yum -y install sl //小火車yum -y install oneko //鼠標兔子date //查看日期cal 11 19…

數據庫相關整理

一、MySQL 1、mysql如何做分頁 mysql數據庫做分頁用limit關鍵字,它后面跟兩個參數startIndex和pageSize 2、mysql引擎有哪些,各自的特點是什么? http://www.cnblogs.com/ctztake/p/8453990.html 3、數據庫怎么建立索引 create index account_…

api接口怎么對接_系統對接項目管理方面怎么做?從一次項目接口對接說起

故事:最近業務方有一個新的業務合作模式,需要與第三方公司進行系統的對接,原本預期2周可以完成的項目,最后要用到3周時間才能完成,出現的現象其實還挺典型的,也不是沒遇到過,因為自己這邊的進度…

周轉時間 平均周轉時間 帶權周轉時間 平均帶權周轉時間

1.周轉時間 2.平均周轉時間 平均周轉時間是對n個而言的 3.帶權周轉時間 真正的運行時間指的是進程占有處理機的時間 4.平均帶權周轉時間 即n個平均的帶權周轉時間

程序員第一定律:關于技能與收入

在軟件這個行業里有些規則是很有殺傷力的,比如很有名的摩爾定律。 總結出這些規則的意義在于可以大致的照明方向,免得努力來努力去卻走到了陰溝里。 現實中種種利益紛爭、觀點之爭看似紛繁,但在大時間尺度下來看卻都是規則的實現手段。 這就好…

快速高效 | iOS身份證識別

一:簡介 demo新增身份證國徽面識別,可識別出簽發機關和有效期。 快捷支付涉及到方方面面,同時安全問題既是用戶所關心的,也是制作者不容忽視的重要部分。 比如涉及到支付和金錢的app,商戶端app等等,都需要進…

用計算機怎樣給文章分解,計算機試卷6分解..pdf

計算機試卷 6一、單選題1. 第三代計算機采用 ______ 作為主要的電子器件。A. 電子管B. 晶體管C.大規模集成電路D.小規模集成電路答案: D2. 數據是信息的載體,包括數值、文字、語言、圖形和 ______等不同形式。A. 多媒體B.表達式C. 圖像D. 函數答案&#…

unity應用開發實戰案例_「簡歷」STAR法則的實戰應用,附手把手教學案例

關注應屆生求職網,了解更多求職信息本文共2072字,預計閱讀需3分鐘本期分享導師-Anna_青云導師51Job職場導師知乎職場千贊答主多年獵頭及多行業人力資源管理經歷職業生涯規劃師、職業生涯咨詢師、心理咨詢師終身學習踐行者、斜杠青年、職場教練在簡歷優化…

一起看一下主流應用使用了哪些三方庫

背景 我們在進行Android開發時往往會面臨技術選型的問題, 面對如此多的開源框架如何進行選擇、選擇的標準是什么,這是一個值得思考的問題. 為此我在后臺爬取了6000多個主流應用,逐個反編譯統計它們使用了哪些開源框架,因此做了一個款應用 基本思路 首先我們要有Apk才可以進行分…

黑客——技術的凝聚者???

寫在前面: 不知道在中國,有多少人能真正的理解什么是hack,當hack被加上商業的成分就失去了本來的色彩,如今多如繁星的大小類黑客站點如雨后春筍般出現,不過是否有技術,統統叫hack,拿著別人的工具去砍掉幾個…