作者life1347 (黑人)
看板Soft_Job
標題[心得] Google 如何進行 Code Review
時間Sat Sep 7 19:48:55 2019
Blog 好讀版:
http://bit.ly/how-to-do-a-code-review
最近 Google 釋出內部如何進行 code review 的相關文件!
這兩天花了點時間作翻譯
文章共分為六篇:
* 程式審核準則
* 程式審核過程中要看些什麼?
* 審核中該如何在 CL 巡航
* 程式審核的速度
* 如何寫審核評論
* 如何面對被推遲處理的評論
文章內容對於原文的翻譯外，亦有個人對於內容的理解
另外，加入自己在貢獻開源軟體中瞭解到的事情
自己本身並非專業譯者，文句可能不通暢或出現錯誤
分享給大家，也請各位多多指教與指正
--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 223.136.251.13 (臺灣)
※ 文章網址: https://webptt.com/m.aspx?n=bbs/Soft_Job/M.1567856937.A.C4F.html
1F：推 kiddqoo: 推用心！ 09/07 19:58
2F：推 kentity: 感謝分享 09/07 20:03
3F：推 ianlee1228: 推用心！！ 09/07 20:25
4F：推 brianwu1201: 推！ 09/07 20:33
5F：推 Serge45: 推! 09/07 20:33
6F：推 yupog2003: 感謝分享 09/07 21:07
7F：推 alog: 感謝分享 += 1 09/07 21:28
8F：推 v420746k: ++感謝分享 09/07 21:47
9F：推 seal0112: 感謝分享 09/07 21:50
10F：推 goldie: 謝謝分享 09/07 22:00
11F：推 wvwvwvwvwv: 推 09/07 22:07
12F：推 sailoruranu: 推 09/07 23:01
13F：推 jason710068: 讚 09/07 23:07
14F：推 somefatguy: 推 09/07 23:18
15F：推 BlazarArc: 推 09/07 23:44
16F：推 johnny12728: 推 09/08 00:09
17F：噓 musie: 這套系統不是大家都適用 你的公司太小這樣玩會死 09/08 02:34
個人不完全認同因為「小」而否定內容所提及的幾個原則
規則是死的，它可以根據不同狀況來改變
我相信服務出包這件事，不管大小公司都是很嚴重的問題
因此人數少的話，能做的改變可能是較為寬鬆的審查流程
比方說只檢查會造成嚴重問題的設計，確保服務不會出現狀況
另外，當工作年資增加時越會需要習得相關「軟」技能
畢竟職位越往上越強調相關的技術，而非開發速度多快來決定
分享自己的想法，給各位參考看看 :)
18F：推 supernow: 感謝翻譯分享 09/08 02:48
19F：推 NDark: Google如何QA: 就是多請一個SEQ. 09/08 03:04
20F：→ NDark: 不過全球級的產品預算投多點合理。不然出包影響範圍很大。 09/08 03:05
21F：推 Rocker5566: 推 09/08 07:17
22F：推 flightheory: 推 09/08 09:47
23F：推 gohere1213: 推推 09/08 10:50
24F：推 abccbaandy: 這種東西都比不上老闆一句：我明天就要 09/08 11:08
25F：推 liuderchi: 推 謝謝分享 09/08 11:31
26F：推 waynegon: 推 09/08 11:54
27F：推 s890510: 推，感謝分享 09/08 12:38
28F：推 sharek: 推，無關小不小，這是開發者的自我成長 09/08 13:59
29F：推 fanatics5566: 推 09/08 14:00
30F：推 jhengsiaomin: 感謝分享 09/08 14:26
31F：推 joejoe321321: 推, 好文章 09/08 17:19
32F：推 joy0520: push 09/08 19:30
※ 編輯: life1347 (223.136.188.228 臺灣), 09/08/2019 19:39:08
33F：推 sunrise1748: 感謝分享 09/08 21:10
34F：推 Neistpoint: 值得參考, 不知道實際上在內部執行狀況如何? 09/08 22:34
根據個人經驗來說蠻有效的
很多文章內的技巧
我是透過翻閱 github 上大型專案的 review comments
加上跟其他人共事所獲得的經驗
但實際執行效果，每家公司各有不同 XD
能肯定的是有持續保持下去，而非三天打魚兩天曬網
過陣子會像文章內 speed of code review 所說的
審核速度加快、程式品質、同事素質會提高
當然一開始要導入非常嚴格的審核一定會造成反彈
建議標準從鬆然後逐日變緊，這樣大家程度也能逐漸跟上步伐
※ 編輯: life1347 (223.136.188.228 臺灣), 09/08/2019 23:44:03
35F：推 toothlesses: 推 09/09 21:32
36F：→ JasperChang: 然後Android SDK證實這個方式也不會讓程式品質更好 09/09 23:40
37F：推 jammy50605: 好麻煩的感覺 如果更動到style跟function 還要分兩次 09/10 10:46
38F：→ jammy50605: 進... 不過嚴謹也是對啦 09/10 10:46
39F：推 Neistpoint: 可以給幾個不錯的 review comments 的連結嗎? 09/10 18:24
建議直接找大型 open source 專案看
比方說出自 google 的 kubernetes
https://github.com/kubernetes/kubernetes/pull/82289
類似這種多人參與的 review 就會不錯
※ 編輯: life1347 (223.140.105.204 臺灣), 09/10/2019 22:52:14
40F：推 abc53: 推 09/11 02:48
41F：推 qaz0101: 推 09/11 11:56
42F：推 crazyjamie: 推 09/11 21:50
43F：推 kotorichan: 推 謝謝分享 09/12 22:12
44F：推 boy955403: 推 感謝分享 09/17 12:20
45F：推 yenru: 推 09/18 23:42
46F：→ cha122977: 內部執行很單純:客觀且對事不對人 09/20 01:52
47F：→ cha122977: 能做到大概就會遵守準則了 09/20 01:52