JSDN

這篇blog分享了Code Review的套路 1. 代碼格式方面 2. 代碼可讀性方面 3. 幫代碼作者想想他/她有沒有漏掉任何 corner case 4. Error handling 5. 測試例和防坑 6. 小架構 7. 大架構 作者提到如果按照這個 checklist 走過一遍，一些明顯的問題就可以避免個八九不離十了。 覺得裡面講了一個觀念很好，每位工程師應該將Code Review的技能跟寫code這兩個技能同為致力的目標，這兩個技能缺一不可。這讓我想到如果是身在成長很快的團隊，工程師都會很快有機會擔任mentor的角色。 Code Review這個實踐應該也將它設計為新進工程師到職流程的一部份，擔任導師(mentor)的角色，更要將這個實踐當作是指點新進工程師最好的機會，除了提升新手的coding技巧，也大幅提升了團隊的生產力。 文中提的code review應該是用Github的pull request配合作者所說的stamp，有經過review後加上stamp才能merge到master branch。如果對於除了GitHub之外的code review工具有興趣，也可以參考JetBrains的UpSource工具，10人版是免費的，還可以直接跟JetBrains IDE做整合。有興趣可以參考這篇Blog - 於IDE做同儕代碼審查，這篇Blog最後面有附上一個跟Code Review相關的免費電子書-What to Look for in a Code Review 的下載連結。 免費電子書傳送門 : 於IDE做同儕代碼審查 https://goo.gl/H3e2ZP #codereview http://ift.tt/2bt0jSW
via Facebook http://ift.tt/2bt0jSW