softnshare / candcpp

C & C++ 技術討論/知識管理共筆
https://softnshare.wordpress.com/slack/cc/
32 stars 5 forks source link

免費電子書《What to Look for in a Code Review》 #18

Open mht opened 8 years ago

mht commented 8 years ago

本書集結多篇 JetBrains 家 Upsource 部落格文章,探討 Code Review 時應該看哪裡,該怎麼看。

理論上 Code Review 有助提昇程式碼品質,實務上,效果因人而異。有 Code Review 經驗的人 Review 的成效會比沒有經驗的人好。讀這本小書,吸取別人的經驗,也是好事。

書裡也簡短說明 SOLID Principles。「Code Reviewer 懂 SOLID,讓程式碼更 SOLID」,是這樣的概念。

註:書中的範例碼使用 Java,但不影響閱讀。

http://leanpub.com/whattolookforinacodereview

xchehub commented 8 years ago

的確 內行的眉眉角角都會點出來 沒法度的 會變成 流程上的 一顆橡皮圖章 還曾在某家公司看過 code review 是寫在 word 上的

mht commented 8 years ago

「寫在 Word 上的 Code Review」...我的大腦 Buffer Overflow 了 XD

橡皮圖章式的 Code Review 對專案的危害遠大於好處,不如不做。這些年工具的發展突飛猛進,讓 Code Review 的門檻低了許多。不過,成事還是在人,然後就複雜了。

maoyang commented 8 years ago

大家目前用哪一種工具做code review輔助?

mht commented 8 years ago

敝公司目前使用 Fog Creek 家的 Kiln 版本控制服務,Code Review 為加值功能需額外付費。挑 Kiln 的主要原因是用他們家的 FogBugz 做 Issue Tracking 與專案管理有些年,很滿意。加上 Kiln 跟 FogBugz 有整合,就買單了。

不過,我們目前的 Code Review 不夠徹底,算是半吊子,還有很多改進空間。

https://www.fogcreek.com/kiln/