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

•  代码审查
  • 能减少设计、开发、维护等成本
  • 能减少开发时间
  • 能减少缺陷
• 结对编程
  • 注意事项
    
    • 协商编码风格
    • 一个人编码，另一个人不要光看
    • 要灵活，简单的功能就不要结对编程了
    • 经常换队友（推荐1天1换）
    • 确保两个人的思路能同步
    • 确保两个人都能看得清代码
    • 不要强制让两个关系不好的人结对编程
    • 两个人之中最好有一个人有过结对编程的经验
  • 好处
    • 减少压力，防止烂代码的产生
    • 提高代码质量（可读性、可理解性等方面）
    • 减少开发时间
    • 提高团队意识、辅导新手、倡导观念“代码属于集体”
• 正式审查
  • 注意事项
    
    • Checklist可以让审查人员的注意力集中在曾经犯过的错误上
    • 审查主要是找出错误，二不是纠正错误
    • 审查大会开始前，审查人员要做好准备，并把他们发现的错误带到大会上
    • 每个参与的人都有不同的角色
    • 审查大会的主持人不能是代码的作者
    • 调解人有过审查方面的专业训练
    • 只有每个参与人都做好了充分的准备才能举办审查大会
    • 审查大会上产生的数据（比如大会时间、看代码的速度等）都要记下来，方面以后改善效率
    • 管理层的人最好不要参加审查大会，除非审查的目的是项目计划或其他管理方面的东西。技术领导可以参加。
  • 角色
    • 主持人
    • 代码的作者
    • 审查人
    • 文案
    • 管理员最好不要参加
    • 审查大会不能少于3个角色
  • 过程
    • 计划
    • 概要
    • 准备
    • 审查大会
    • 审查报告
    • 返工
    • 追踪
    • 第三小时会议，讨论如何解决问题
  • 改善审查。分析会议中产生的数据来优化审查大会
  • 审查大会上避免讨论解决方法
• 代码走读
  • 注意事项
    • 通常由代码的作者来主持
    • 注意力集中在技术问题上
    • 所有的参与者要做好准备
    • 走读可以让新手更加熟悉整个团队
    • 走读通常持续半小时到一小时
    • 强调找出错误，而不是纠正错误
    • 代码走读是比较灵活的，可以适应各种需求
• 代码阅读
  • 更加注重审查，每个人的注意力主要集中在审查上，而不是会议上
• 审查的时候不要让客户参与，不然就不是技术审查了