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 * 導致分庫分表