代码评审
烂代码
人的视角
维护者脏话的频率高、维护者脏话的类型丰富、存在打架斗殴的可能性、面向离职编程
代码的视角
不遵守代码规约、代码像迷宫、代码流程脚踩西瓜皮、代码执行效率低、10行代码15个bug
星级代码
☆ 计算机可以理解
☆☆ 自己未来可以理解
☆☆☆ 别人可以自我理解
程序员自我修养
代码评审(CR)
• 熵减的过程 :减少系统混乱
• 团队成长 :养成团队成员间的交流文化,有利于团队的知识共享
• 提升代码规范度 :通过代码审查,发现纠正不规范情况,慢慢形成良好开发规范
• 提高代码质量 : 工程师互相review,扫除知识盲区,提升代码的质量
CR的副作用
• 提高自己的抗打击能力
• 对于自己错误的深刻理解
• 交流中碰撞出激情的火花
• 有可能锻炼肺活量
如何做CR
统一的编码与设计规范、完整的技术架构说明与事例
不定期的Review会议
小项目(3个月内)可以10天/次;大项目(6个月以上)15天/次;
前期可以安排密集一些