请问大家的公司有代码 review 吗

[holy_sin]

大家觉得有必要吗

[msg7086]

老代码没有，屎一样的代码满天飞。
新代码所有的 commit 都要 peer review。

[20015jjw]

必须得有啊

[jerryshao]

我就来看看有没有用 Reviewboard 的

[randyzhao]

@jerryshao 握手 哈哈

[randyzhao]

纯人工的 Review 优点很明显，同时也有些瑕疵。
比如：你平时玩的好的同事，找你帮他 “ Review ” 一下。

如果是要避免这样的漏洞，那么可以设置特定的人 (如: Team Leader) 一定要 Review 过才算行。
不过这样的话，Team Leader 其实也很累。
所以团队里一定得都是志同道合的人，才最好。

[fanqianger]

有 review
感觉没什么用

[nicevar]

有，相当有必要，有时候人会犯一些很蠢的错误，扫几眼就能发现问题，如果不 review 等部署了才发现浪费大家时间

[eccstartup]

不 review 别人怎么用你家产品啊

[chiu]

有专门一个部门在监督 review 的……每次 review 还要记录好多东西

[zhq527725]

有。
肥肠有必要，俺们公司没有一个测试人员，代码质量全靠 code review 和自动化测试。

[patx]

有
有

[l00t]

1.没有
2.有必要

其实我喜欢结对编程。每次结对编程的时候感觉效率可高了。

[EricFuture]

没有，非常有必要

[xiangyuecn]

好奇你们会不会在自己写的代码里面埋后门，怎样逃过别人 review

[crayygy]

有，而且必须两个人同意以后才能提交，而且不存在什么关系好你帮我点一下这种，出了问题大家脸上都不好看，关系好就更帮你认真看了。

[hasbug]

不存在的

[clino]

@lgh #7 有很多要导入吗?我们一开始一直用这个,所以没什么导入的问题.

@smilingsun #5 review 实现吧,unittest 非常少

[Acheron]

1.没有
2.没必要

所有偏离代码本质的东西都是无必要的，比如 review，多余的注释，站立会，代码的本质就是自由，这些边边角角限制了自由，但没必要不一定不做，自己 review 自己的代码就可以了。

[ifsoar0712]

@jerryshao +1

[minbaby]

知道是一回事，做到是另一回事。

想要在项目中，加入 review 这个选项，第一是不能影响产品正常进度（不然老板啊，产品啊，拎着刀就过来了），第二是部门负责人能意识到 Review 的重要性和必要性（发生两次重大事故就“好了”），第三团队成员有这个意思（不然就沦为了“形式“）