团队如何进行CodeReview
六个不要
不要刻意的去寻找代码bug
不要按照自己的编程风格去评论别人的代码
不要带着抨击和质疑别人能力的心态去进行代码评审
不要在不确定的问题上争来争去
不要听不进别人的意见
参与者最好不要自己都没想明白就提意见
三个原则
发现代码的正确性
分享和学习业务逻辑和设计思路
高效迅速的完成CodeReview
Review会议流程
1. 主持者分享业务逻辑和设计思路,技术经验。并且对代码功能进行介绍
2. 检查设计的合理性和业务逻辑的正确性
业务流程是否能够走通
单一职责,入参是否合理
数据库字段设计是否满足需求,是否满足三范式,实体类设计是否合理
是否有异常处理机制
是否影响后续业务的扩展
关注数据结构:Map, HashMap, List, LinkedList
是否有影响性能的代码:数据库批量操作,资源是否关闭
3. 检查代码的可读性和可维护性
必要的注释: 类,方法, 复杂代码段
命名规范
重复代码抽取成方法
繁琐代码,能简单实现的地方是否需要优化
4. 总结优缺点
设计思想、技术方法、业务知识
编程规范、代码风格
如何避坑