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 * 导致分库分表