代码审查案例.pptx

代码审查技术;C/C++代码审查;不懂开发怎么做代码审查？;循环语句的效率;在多重循环中，如果有可能，应当将最长的循环放在最内层，最短的循环放在最外层，以减少CPU 跨切循环层的次数： for (col=0; col5; col++ ) { for (row=0; row100; row++) { sum = sum + a[row][col]; } } ;指针的使用;switch Statement Should Include a Default Case ;根据编码规范，每个switch流程控制语句都必须带一个default分支，以保证逻辑分支的完整性。 如果没有第15~16行的default代码，代码审查将给出警告。 1. switch (formatType)2. {3. 　case 1:4. 　　formatStr = yyyyMMddHHmmss;5. 　　break;6. 　case 2:7. 　　formatStr = yyyy-MM-dd HH:mm:ss;8. 　　break;9. 　case 3:10.　　formatStr = yyyy.MM.dd HH:mm:ss;11.　　break;12.　case 4:13.　　formatStr = yyyy年MM月dd HH:mm:ss;14.　　break;15.　default:16.　　formatStr = yyyy-MM-dd HH:mm:ss;17. };Complex Assignment ;往往有些程序员热衷于将Java的语法发挥到极致，以资其对Java语法精通的凭据。如果是为了练习语法、理解语法，无可厚非。但如果在需要充分协作沟通的软件项目中，简洁明了，清晰易懂将会受到推崇，晦涩难懂的语句将会受到奚落。 故此，大部分的软件公司的规范都对语句的精简明了提出了要求。;1. int[] arr = new int[size];2. if (arr == null) 3. {4.　 return null;5. };流程控制中存在不可到达的语句：死代码有些流程控制由于测试条件恒为false，则流程中的程序无法到达。1. int[] arr = new int[size];2. if (arr == null) //由于arr不为空，则该测试逻辑不可能通过，程序无法进入该程序块中3. {4.　 return null;5. };添加()清晰化复杂的表达式 写复杂的表达式时不应过度依赖运算操作符的计算优先顺序，而应养成使用“()”的好习惯，当一个逻辑表达式由多个逻辑运算组成时，应该用“()”划分不同的部分。 1. boolean a, b, c;2. ...3. if (a || b c) //应该替换成if ((a || b) c)4. {5. ...6. };Coding Style：- Multiple Statements on One Line - Place Statement in Block - Use L instead of l at the End of Integer Constants ; 不应将多行语句写在同一行代码中。 代码块应以“{}”框起来，虽然增长了代码，但代码结构性更强。 声明长整型使用大写的“L”类型指定符，而非小写的“l”，因为后者和数字1相似。;命名规范;“匈牙利”法;“匈牙利”法的缺点;骆驼式命名法;帕斯卡（Pascal）;代码审查清单;练习：读代码，找错误;PC-Lint;FxCop;FxCop的使用;DevPartner ;DevPartner的Code Review功能 ;在Rule Manager中编辑代码规则 ;JBuilder的Code Audits功能的使用;JavaScript代码检查工具-JSLint