【读书笔记】代码大全21章：团队编程

代码审查

能减少设计、开发、维护等成本
能减少开发时间
能减少缺陷

结对编程

注意事项

协商编码风格
一个人编码，另一个人不要光看
要灵活，简单的功能就不要结对编程了
经常换队友（推荐1天1换）
确保两个人的思路能同步
确保两个人都能看得清代码
不要强制让两个关系不好的人结对编程
两个人之中最好有一个人有过结对编程的经验

好处

减少压力，防止烂代码的产生
提高代码质量（可读性、可理解性等方面）
减少开发时间
提高团队意识、辅导新手、倡导观念“代码属于集体”

正式审查

注意事项

Checklist可以让审查人员的注意力集中在曾经犯过的错误上
审查主要是找出错误，二不是纠正错误
审查大会开始前，审查人员要做好准备，并把他们发现的错误带到大会上
每个参与的人都有不同的角色
审查大会的主持人不能是代码的作者
调解人有过审查方面的专业训练
只有每个参与人都做好了充分的准备才能举办审查大会
审查大会上产生的数据（比如大会时间、看代码的速度等）都要记下来，方面以后改善效率
管理层的人最好不要参加审查大会，除非审查的目的是项目计划或其他管理方面的东西。技术领导可以参加。

角色

主持人
代码的作者
审查人
文案
管理员最好不要参加
审查大会不能少于3个角色

过程

计划
概要
准备
审查大会
审查报告
返工
追踪
第三小时会议，讨论如何解决问题

改善审查。分析会议中产生的数据来优化审查大会
审查大会上避免讨论解决方法

代码走读

注意事项

通常由代码的作者来主持
注意力集中在技术问题上
所有的参与者要做好准备
走读可以让新手更加熟悉整个团队
走读通常持续半小时到一小时
强调找出错误，而不是纠正错误
代码走读是比较灵活的，可以适应各种需求

代码阅读

更加注重审查，每个人的注意力主要集中在审查上，而不是会议上

审查的时候不要让客户参与，不然就不是技术审查了