代码review.pptVIP

代码review 刘伟 2013.3.4 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. 代码评审是指在软件开发过程中，通过对源代码进行系统性检查的过程。通常的目的是查找系统缺陷，保证软件总体质量和提高开发者自身水平。 为什么进行代码评审 1.提高质量 2.及早发现潜在缺陷与BUG，降低事故成本。 3.促进团队内部知识共享，提高团队整体水平 4.评审过程对于评审人员来说，也是一种思路重构的过程。帮助更多的人理解系统。 代码review的基本原则 1.基本情况 1.1代码是否遵循编程指南？ 1.2代码是否有自我记录能力？能否通过阅读理解代码？ 1.3是否已经解决了代码规则检查和（或）运行错误检测工具发现的所有错误？ 代码review的基本原则 2注释 2.1注释是否反映了必威体育精装版的情况？ 2.2注释是否清晰正确？ 2.3如果代码被变更，修改注释是否容易？ 2.4注释是否着重解释了为什么，而不是怎么样？ 2.5是否所有的意外、异常情况和解决方法错误都有注释？ 2.6每个操作的目的是否都有注释？ 2.7与每个操作有关的其他事实是否都有注释？ 代码review的基本原则 3源代码细节 3.1是否每一个操作都有一个描述其操作内容的名称？ 3.2参数是否有描述性的名称？ 3.3完成各个操作的正常路径是否与其他异常路径有明显区别？ 3.4操作是否太长，它能否通过将有关语句提取到专用操作中进行简化？ 3.5操作是否太长，它能否通过减少判定点的数目进行简化？决策点是代码可以采取不同路径的语句，例如 if-、else-、and-、while- 和 case- 语句。 3.6循环嵌套是否已减至最少？ 3.7变量命名是否适当？ 3.8代码是否简单明了，是否避免了使用“技巧性”的方法？ 代码评审的选择 1.最近一次迭代开发的代码 2.系统关键模块 3.业务较复杂的模块 4.缺陷率较高的模块 FindBugs简介 FindBugs 是一个静态分析工具, 主要专注于检查程序错误和性能问题。主站网址 有for eclipse的插件，插件下载网址是 /downloads.html 工作原理: 它检查类或者 JAR 文件，将字节码与一组缺陷模式进行对比以发现可能的问题。 规则定义:默认规则列表，可以自选需检查的项目. FindBugs检查问题分类 正确性（Correctness）：这种归类下的问题在某种情况下会导致bug，比如错误的强制类型转换等。 不良实践（Bad practice）：这种类别下的代码违反了公认的最佳实践标准，比如某个类实现了equals方法但未实现hashCode方法等。 多线程正确性（Multithreaded correctness）：关注于同步和多线程问题。 性能（Performance）：潜在的性能问题。 安全（Security）：安全相关。 高危（Dodgy）：Find