作者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/cn.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