高级开发以及 leader 素质必备之代码评审(Code Review)

什么是代码评审?

1.代码评审是指在软件开发过程中,通过对源代码进行系统性检查的过程。通常的目的是查找系统缺陷,保证软件总体质量和提高开发者自身水平。
2.通常在企业开发中,会定期面对面(face to face)对代码进行评审

Code Review的意识
作为一个 Developer,不仅要提交可工作的代码(Deliver working code),更要提交可维护的代码(Deliver maintainable code)
必要时进行重构,随着项目的迭代,在计划新增功能的同时,开发要主动计划重构的工作项
开放的心态,虚心接受大家的评审建议(Review Comments)
代码评审的方式
开 Code Review 会议
团队内部会整理 Check List
团队内部成员交换代码
找出可优化方案
多问问题,例如:“这块儿是怎么工作的?”、“如果有XXX 情况,你这个怎么处理?”
区分重点,优先抓住设计,可读性,健壮性等重点问题
整理好的编码实践,用来作为 Code Review 的参考
评审内容
[1]架构/设计/常规
1.单一职责原则
这是经常被违背的原则。一个类只能干一个事情,一个方法最好也只干一件事情。比较常见的违背是一个类既干UI的事情,又干逻辑的事情,这个在低质量的客户端代码里很常见
2.行为是否统一,例如:
1.缓存是否统一
2.错误处理是否统一
3.错误提示是否统一
4.弹出框是否统一
……
3.代码污染
代码有没有对其他模块强耦合
4.重复代码-->应该抽取
5.开闭原则
6.面向接口编程
7.健壮性
是否考虑线程安全
数据访问是否一致性
边界处理是否完整
逻辑是否健壮
是否有内存泄漏
有没有循环依赖
有没有野指针
是否检查了数组的“越界“错误
……
8.错误处理
9.改动是不是对代码的提升
新的改动是打补丁,让代码质量继续恶化,还是对代码质量做了修复
10.效率/性能
1)关键算法的时间复杂度多少?有没有可能有潜在的性能瓶颈
2)客户端程序对频繁消息和较大数据等耗时操作是否处理得当
[2]代码风格
1.可读性
衡量可读性的可以有很好实践的标准,就是 Reviewer 能否非常容易的理解这个代码。如果不是,那意味着代码的可读性要进行改进
2.命名
命名对可读性非常重要
是否跟系统属性命名造成冲突
英语用词尽量准确一点,必要时可以查字典
3.函数长度/类长度
函数太长的不好阅读
类太长了,检查是否违反的 单一职责 原则
4.注释
恰到好处的注释,不是注释越多越好
5.参数个数
不要太多,一般不要超过 3 个
Review Your Own Code First
每次提交前整体把自己的代码过一遍非常有帮助,尤其是看看有没有犯低级错误

你可能感兴趣的:(高级开发以及 leader 素质必备之代码评审(Code Review))