这是一个创建于 2730 天前的主题，其中的信息可能已经有所发展或是发生改变。
大家觉得有必要吗
 | 1 lisonfan 2018-04-19 22:28:47 +08:00 1、没有 2、有必要 |
 | 2 clino 2018-04-19 22:29:54 +08:00  2 有必要 原则上每笔提交都要经过至少另外一名同事的 review |
 | 3 kikyous 2018-04-19 22:35:15 +08:00 via Android 我们公司用 arc,还有其他工具可以用吗？ |
| 4 clino 2018-04-19 22:39:34 +08:00 我们用 google 的那个 gerrit |
 | 5 smilingsun 2018-04-19 22:42:33 +08:00 via Android @clino 我们也用 gerrit，你们 review implementation 还是 unit test |
 | 6 seaswalker 2018-04-19 22:55:25 +08:00 有 |
 | 7 lgh 2018-04-19 23:16:05 +08:00 via iPhone |
 | 8 est 2018-04-19 23:16:12 +08:00 有必要。gitlab 使用中。 |
 | 9 Perry 2018-04-19 23:19:26 +08:00 用的 Bitbucket code review，需要 2 个以上同事批准，有些项目还要确保 Test Coverage 在一个阈值之上 觉得很有必要 |
 | 10 north521 2018-04-19 23:25:23 +08:00 没有 有必要，对提升自己的代码水平和减少公司项目的坑很有帮助 |
 | 11 zpf124 2018-04-19 23:33:34 +08:00 没有， 有必要 另外我们连单元测试也不写几个... |
 | 12 tinycold 2018-04-19 23:40:19 +08:00 via Android 1 我们结对编程。 没开玩笑 |
 | 13 jadec0der 2018-04-19 23:42:00 +08:00 有，有必要但是付出的代价太大了 |
 | 14 huluhulu 2018-04-19 23:42:30 +08:00 via iPhone 非常有必要，有，使用 gerrit，keyuser 有权限+2，且触发自动编译和自动化 case，都过了才会真正合到 trunk |
 | 15 kingcos 2018-04-19 23:46:15 +08:00 via iPhone 有，2 个看过 Review 过才可以，而且最近正在配置 CI … |
 | 16 xiaojie668329 2018-04-20 00:13:15 +08:00 via iPhone 有必要。准备在例会时提出并推动。 |
 | 17 huntzhan 2018-04-20 00:19:13 +08:00 Required, 必须有人 review 才能 check in |
 | 18 Pastsong 2018-04-20 00:20:59 +08:00 有 不强制 是 Proof of Work 的一种方法 |
 | 19 PHPer233 2018-04-20 00:21:10 +08:00 via iPhone 管你怎么写，写出来能跑就行。完成任务，交差回家~ |
 | 20 Guaidaodl 2018-04-20 00:24:10 +08:00 有啊. 绝对有必要. 而且对新人来说有人 Review 你成长才快. |
 | 21 msg7086 2018-04-20 01:10:11 +08:00 老代码没有，屎一样的代码满天飞。 新代码所有的 commit 都要 peer review。 |
 | 22 20015jjw 2018-04-20 03:40:09 +08:00 via Android 必须得有啊 |
 | 23 jerryshao 2018-04-20 03:46:39 +08:00 我就来看看有没有用 Reviewboard 的 |
 | 25 randyzhao 2018-04-20 04:05:52 +08:00 纯人工的 Review 优点很明显，同时也有些瑕疵。 比如：你平时玩的好的同事，找你帮他 “ Review ” 一下。 如果是要避免这样的漏洞，那么可以设置特定的人 (如: Team Leader) 一定要 Review 过才算行。 不过这样的话，Team Leader 其实也很累。 所以团队里一定得都是志同道合的人，才最好。 |
 | 26 fanqianger 2018-04-20 04:24:39 +08:00 有 review 感觉没什么用 |
 | 27 nicevar 2018-04-20 07:28:04 +08:00 via Android 有，相当有必要，有时候人会犯一些很蠢的错误，扫几眼就能发现问题，如果不 review 等部署了才发现浪费大家时间 |
 | 28 eccstartup 2018-04-20 07:35:03 +08:00 via iPhone 不 review 别人怎么用你家产品啊 |
 | 29 chiu 2018-04-20 07:47:04 +08:00 via Android 有专门一个部门在监督 review 的……每次 review 还要记录好多东西 |
 | 30 zhq527725 2018-04-20 08:16:12 +08:00 有。 肥肠有必要，俺们公司没有一个测试人员，代码质量全靠 code review 和自动化测试。 |
 | 31 patx 2018-04-20 08:33:03 +08:00 有 有 |
 | 32 l00t 2018-04-20 08:37:35 +08:00 1.没有 2.有必要 其实我喜欢结对编程。每次结对编程的时候感觉效率可高了。 |
 | 33 EricFuture 2018-04-20 08:45:07 +08:00 没有，非常有必要 |
 | 34 xiangyuecn 2018-04-20 08:48:06 +08:00 好奇你们会不会在自己写的代码里面埋后门，怎样逃过别人 review |
 | 35 crayygy 2018-04-20 08:58:19 +08:00 via Android 有，而且必须两个人同意以后才能提交，而且不存在什么关系好你帮我点一下这种，出了问题大家脸上都不好看，关系好就更帮你认真看了。 |
 | 36 hasbug 2018-04-20 09:00:11 +08:00 不存在的 |
| 37 clino 2018-04-20 09:27:32 +08:00 |
 | 38 Acheron 2018-04-20 09:28:20 +08:00 1.没有 2.没必要 所有偏离代码本质的东西都是无必要的，比如 review，多余的注释，站立会，代码的本质就是自由，这些边边角角限制了自由，但没必要不一定不做，自己 review 自己的代码就可以了。 |
 | 39 ifsoar0712 2018-04-20 09:33:09 +08:00 @jerryshao +1 |
 | 40 minbaby 2018-04-20 09:51:33 +08:00 知道是一回事，做到是另一回事。 想要在项目中，加入 review 这个选项，第一是不能影响产品正常进度（不然老板啊，产品啊，拎着刀就过来了），第二是部门负责人能意识到 Review 的重要性和必要性（发生两次重大事故就“好了”），第三团队成员有这个意思（不然就沦为了“形式“） |
 | 41 EanCuznaivy 2018-04-20 09:54:37 +08:00 直接 review PR，然后老板亲自 merge …… 很有必要。 |
 | 42 vakara 2018-04-20 10:10:51 +08:00 有，我们组内强制，Review 很仔细。 公司使用 GitHub。 |
 | 43 longxboy 2018-04-20 10:16:35 +08:00 有的，强制 review 加各种静态检查编译检查 |
 | 44 tonghuashuai 2018-04-20 10:21:12 +08:00 有 有必要 强制至少一位同事 review 后才可以 merge |
 | 45 lsmgeb89 2018-04-20 10:35:01 +08:00 有必要啊，没有 review 的公司或组就没必要做了。 |
 | 46 ioioioioioioi 2018-04-20 10:43:12 +08:00 有必要的，不然容易有 bug。 |
 | 47 yuxiaofei93 2018-04-20 10:53:59 +08:00 via iPhone 公司里有代码审查，并且出事故代码审查人受同样处罚。 代码审查有必要，至少能避免一眼就看出来的低级失误 |
 | 48 unidentifiedme 2018-04-20 10:58:30 +08:00 有，基本上所有项目强制 code review 很有必要。 所有项目都接 Coverity 有的项目还有 unit test coverage 的要求 |
 | 49 cccRaim 2018-04-20 11:03:00 +08:00 有， 三个人 review 才行 |
 | 50 shaobin0604 2018-04-20 11:15:27 +08:00 经历过几家公司，都有 code review，工具 - gerrit - reviewboard - gitlab merge request |
 | 51 coderluan 2018-04-20 11:35:33 +08:00 公司没有，各个项目自己要求，肯定是有必要的，但是如果一个项目是时间紧张收益低的一锤子买卖，你懂的。 |
 | 52 bluefalconjun 2018-04-20 11:37:38 +08:00 有必要. gerrit 提交: 脚本会检查 1.代码格式和 commit 关键字; 2.编译; 然后+1; 另外需要其他队员 review+1; 最后 merge 权限的人才能 merge 进 master. |
 | 53 jasonchen168 2018-04-20 11:37:39 +08:00 小公司业务代码都写不完。。。 |
 | 54 ihaveadrame 2018-04-20 11:46:30 +08:00 不要问题了 选 C。 我们公司对这块要求比较严格，基本是 pylint flake8 eslint 代码规范检测 tox 单元测试 还有 代码覆盖率测试。 每个 PR 都至少要组长 review 一下。 |
 | 56 kaedea 2018-04-20 12:18:30 +08:00 1. 本来十几人的团队是互相 rv 的。 2. 后来同一个项目多了几个这种规模的团队。 3. 感到他们的代码风格与自己的格格不入后，慢慢放弃了 rv，因为就算提修改建议人家也懒得改。 4. 现在已经没有定期看别人提交了什么代码的习惯了。 |
 | 57 Deeer 2018-04-20 12:18:53 +08:00 via iPhone 我遇到的 review 分为两种、一种是成员代码合并 dev 上进行 review、第二种是从 dev 合并到 test 上的 review、一种是确认代码合并时是否存问题、另一种是合并之后进行检查、不知道你说的是哪种 review ？ |
 | 59 xrlin 2018-04-20 12:21:59 +08:00 via iPhone 看了大家的回答，我要争取去一个注重代码质量的团队，我很想执行 review ci，但是项目就只有我一个人。 |
 | 60 Ranler 2018-04-20 12:29:01 +08:00 有必要 同 Gerrit，强制 2 个人 Review，强制单元测试 PASS 才能合入 |
 | 61 projectzoo 2018-04-20 12:35:03 +08:00 有 有必要 |
 | 62 holy_sin OP 代码风格不统一 貌似是推行 review 机制的一个阻碍 |
 | 63 codeyung 2018-04-20 14:59:26 +08:00 当然有了 // |
 | 64 sgissb1 2018-04-20 15:15:09 +08:00 没有 review，也没有必要。但是作为学习有必要看别人的代码。 review 很费时，搞不好就成茶话会或者秀技大会了，同时可能代码写好写坏有关系吗？会因为写的好点老板多发点钱？人家现在只看结果，只关心快速抢占市场等等。 |
 | 65 wweir 2018-04-20 15:41:02 +08:00 via Android 有必要，主要审两方面东西： 局部的架构设计 代码审查，包括命名风格之类的检查 |
 | 66 vela 2018-04-20 15:46:26 +08:00 有。 有必要。 |
 | 67 jedihy 2018-04-20 15:52:54 +08:00 via iPhone 不 review 就能 checkin 这不是做宝搞吗 |
 | 68 pwf 2018-04-20 15:56:50 +08:00 via Android 用 crucible 的 cr，还有组内投屏... |
 | 69 lizz666 2018-04-20 16:03:49 +08:00 老板的意思是最短的时间用最快的方法写出来上线给客户拿钱，等模板开发好，谁还管你代码质量好不好，全是复制粘贴。 我要学技术。。。 |
 | 70 yuriko 2018-04-20 16:28:32 +08:00 前东家有 用 gerrit 虽然有必要，但是也不能过分 比如合入到 dev 的 master 分支需要三个人加分，感受下…… |
Select Language