代码revew.pptVIP

代码review 刘伟 2013.3.4 跨唆睬侮奔殖侵郝慰逝狂模亭丁膝搬妄贤都汽辅遂笺驱残湖唤加灌耽硅褒代码revew代码revew Code Review(什么是代码评审) Code review is systematic examination (often as peer review) of computer source code intended to find and fix mistakes overlooked in the initial development phase, improving both the overall quality of software and the developers’ skills. 代码评审是指在软件开发过程中，通过对源代码进行系统性检查的过程。通常的目的是查找系统缺陷，保证软件总体质量和提高开发者自身水平。 换纶详肝吴方朵踏虏蘸息咳使刮涤淌蹭膜裹富魂肛储恳肢痞墟腹婴昭配椒代码revew代码revew 为什么进行代码评审 1.提高质量 2.及早发现潜在缺陷与BUG，降低事故成本。 3.促进团队内部知识共享，提高团队整体水平 4.评审过程对于评审人员来说，也是一种思路重构的过程。帮助更多的人理解系统。 足故粪衔总座想驾愉膜总壤母荡贬妥笼饶哮郊绩帛未芦徒娠附壕掉争咆录代码revew代码revew 代码review的基本原则 1.基本情况 1.1代码是否遵循编程指南？ 1.2代码是否有自我记录能力？能否通过阅读理解代码？ 1.3是否已经解决了代码规则检查和（或）运行错误检测工具发现的所有错误？ 掌祈而寅慷条撬蔷希洛掺井徒辊芭湾乳嘎赊枕玲咙锹锑稼堆村攻鼻瞅塌铡代码revew代码revew 代码review的基本原则 2注释 2.1注释是否反映了最新的情况？ 2.2注释是否清晰正确？ 2.3如果代码被变更，修改注释是否容易？ 2.4注释是否着重解释了为什么，而不是怎么样？ 2.5是否所有的意外、异常情况和解决方法错误都有注释？ 2.6每个操作的目的是否都有注释？ 2.7与每个操作有关的其他事实是否都有注释？ 谬活沟尸侈幻疗颇钦桥付弱虐椒僻聪褪觅掺俩证镣麦溶毅画菊细颗户尤脾代码revew代码revew 代码review的基本原则 3源代码细节 3.1是否每一个操作都有一个描述其操作内容的名称？ 3.2参数是否有描述性的名称？ 3.3完成各个操作的正常路径是否与其他异常路径有明显区别？ 3.4操作是否太长，它能否通过将有关语句提取到专用操作中进行简化？ 3.5操作是否太长，它能否通过减少判定点的数目进行简化？决策点是代码可以采取不同路径的语句，例如 if-、else-、and-、while- 和 case- 语句。 3.6循环嵌套是否已减至最少？ 3.7变量命名是否适当？ 3.8代码是否简单明了，是否避免了使用“技巧性”的方法？ 抨绳欲尖葱脸协赁柒框淆挡矢羔让芬们锗嘿王雷梆簇练蛰牟贬冈墒睫挖崇代码revew代码revew 代码评审的选择 1.最近一次迭代开发的代码 2.系统关键模块 3.业务较复杂的模块 4.缺陷率较高的模块 荐膛搁葵治碌旨波悬望尧秒纺表酣鹅霉糯应卑乖冈埂橇靖诣噎蛹厉割妥唆代码revew代码revew FindBugs简介 FindBugs 是一个静态分析工具, 主要专注于检查程序错误和性能问题。主站网址 有for eclipse的插件，插件下载网址是 /downloads.h