孤尽班第十三次课-系统健壮性设计

孤尽班第十三次课-系统健壮性设计

代码评审

什么样的是烂代码（人的视角）

1. 维护者脏话的频率高
2. 维护者脏话的类型丰富
3. 存在打架斗殴的可能性
4. 面向离职编程

什么是烂代码(代码视角)

1. 不遵守代码规约
2. 代码像迷宫
3. 代码流程脚踩西瓜皮
4. 代码的执行效率低
5. 10行代码15 个bug

代码的恶性循环

1. 业务催的紧直接写代码
2. 到处灭火 更是没有时间CodeRevew
3. 没时间填坑，却不断挖坑

星级程序员

1. 写出计算机可以理解的代码 
2. 写出来自己未来可以理解得代码
3. 写出来别人可以自我理解得代码

程序员的自我修养

1. 写出好代码
2. 技术水平不断提高
3. 帮助别人成长

代码评审(简称CR)

1. 团队成长
养成团队成员间的交流文化，有利于团队的知识共享
2. 提升代码规范程度
 通过代码审查，发现纠正不规范情况，慢慢形成良好开发规范
3. 熵减的过程
减少系统混乱
4. 提升代码质量
工程师互相review,  扫除知识盲区，提升代码质量

CR 的副作用

1. 提升自己的抗打击能力
2. 对于自己错误的深刻理解
3. 交流中碰撞出激情的火花
4. 有可能锻炼肺活量

如何做CR

如何做Review  
统一的编码设计规范
完整的技术架构说明与事例
不定期的Review 会议
小项目(3个月内) 可以10 天/ 次。 大项目(6个月以上) 15 天/ 次，前期可以安排密集一些，后期考虑一个月/次
### 推荐的工具
  Phabricator：Facebook 开源的代码审查工具
  Gerrit：非常强的CodeReview + 代码托管工具
  CheckStyle: 代码规范检查工具

CR 建议

	1.  对事不对人
	2.  不要在review  中讨论需求
	3. 明确各模块的负责人
	4. 至少一条正确评价
	5. PR 内容一定要少

健壮性与鲁棒性

如何构建健壮性的系统

面向失败的架构思维

混沌工程