Code Review要点
- CR应当3个人。旁观者的作用很重要。“CR是一次三方的研发协同过程”
- CR应该频繁进行，应当对每个最小提交单元进行CR（不一定要约会，可以异步）。小步快跑。想象一下，重大发布的前夜，你的同事对你说：“来来帮我review一下这5000行代码…”
- CR的威力来自于社交效应：不想自己的代码被打脸。所以每个成员都有评论的权利并且可以cc其他人。
Java经典故障：
- float比较
- jdk7 HashMap扩容时候会死锁。因为移动时的链条倒挂+并发
- ThreadLocal。弱引用。使用前务必set
- 日期与字符串的转换YYYYMMdd。 YYYY返回什么？hint：与yyyy可不一样！
- getter: boolean vs Boolean => 生成的方法一个是is..一个是get..
- for循环批处理时候，case用了return。
- select * 导致分库分表