Code Review要点以及Java系统开发几大经典故障

Code Review要点

1. CR应当3个人。旁观者的作用很重要。“CR是一次三方的研发协同过程”
2. CR应该频繁进行，应当对每个最小提交单元进行CR（不一定要约会，可以异步）。小步快跑。想象一下，重大发布的前夜，你的同事对你说：“来来帮我review一下这5000行代码…”
3. CR的威力来自于社交效应：不想自己的代码被打脸。所以每个成员都有评论的权利并且可以cc其他人。

Java经典故障：

1. float比较
2. jdk7 HashMap扩容时候会死锁。因为移动时的链条倒挂+并发
3. ThreadLocal。弱引用。使用前务必set
4. 日期与字符串的转换YYYYMMdd。 YYYY返回什么？hint：与yyyy可不一样！
5. getter: boolean vs Boolean => 生成的方法一个是is..一个是get..
6. for循环批处理时候，case用了return。
7. select * 导致分库分表