发起个讨论，你们公司有 code review 吗？

[zzzzzy]

code review 好处很多，可以规范代码、传递知识和保证代码质量等，但是因为项目进度和其他因素，不一定保证 review 被有效执行。请教各路大神都是怎么做的？有没有啥套路、工具、方法之类的。

[aitaii]

每周 4 下午有一个 code review 确实能规范公司的代码风格

[jmc891205]

我们用 code collaborator

不过别人憋了两礼拜的代码一口气传上来让 review 的时候。。真的觉得不想去看

[quericy]

产品比开发都多,还来得及 review?

[sudoz]

极个别的组有 review ，真的很少会审查代码，加班都来不及马丹的

[clino]

我们这里不自觉做 code review,做不好 code review 的工程师设计质量和代码质量都比较差

[rekulas]

500 强外企 用的 gerrit 来 view view 要经手好多人 最后的审核者还在国外流程有点慢

[rekulas]

而且审核的也很严格 比如行尾多了一个空格。。。是过不了的

[repus911]

@jmc891205 没办法 有时候会在例会上说 今天我 review 代码 1 天 嗯 1 天

[odirus]

加班赶进度都来不及，还啥子 view 哦， just make it work

公司人力资源多、钱多、注重技术，一般都会。

[menc]

one on one ，用 gerrit 来做 review 。

说不 review 的，都是没 review 过的， review 一下就知道了，注释， coding style 还有 bug ，问题茫茫多。

代码不仅是写给自己看的，在公司里也是写给别人看的， review 不止有挑 bug ，也有保证自己代码别人能读懂的关系在。 code review 不仅会找出 bug ，也会要求修改格式，添加注释以增加代码可读性。

在 V2 上看到有人推崇不写注释，要求代码自解释，来一个稍微大点的团队就知道，这根本就是狗屎。

[imswing]

技术经理：什么是 code review ？

[DevineRapier]

不知道什么才算 review ，我们这写好了提交个 diff ，过了就可以提交。

review 是，先提交，过后在一起回顾审核？

感觉我们是 preview...

[cnly1987]

人多 就需要 review ，不然你敢 merge 其他人的分支吗

[expkzb]

@DevineRapier 提交 diff 在 phabricator 里就算 review 。感觉这样挺好的

[monkeylyf]

没有 最后代码库简直臭气熏天

[4everLoveU]

RD 提测， QAreview

[nullp]

全靠自觉，，接收人骂爹

[hemingway]

有

[gefranks]

据说是有 review 的,也有 reviewer 的名字,但是仍然不工作或者 bug 百出

[hahastudio]

https://www.reviewboard.org/