大家觉得有必要吗
This topic created in 3123 days ago, the information mentioned may be changed or developed.
 |
lisonfanApr 19, 2018
1
1、没有
2、有必要 |
 |
2
有必要
原则上每笔提交都要经过至少另外一名同事的 review |
2 |
kikyousApr 19, 2018 via Android
3
我们公司用 arc,还有其他工具可以用吗？
|
|
clinoApr 19, 2018
4
我们用 google 的那个 gerrit
|
 |
smilingsunApr 19, 2018 via Android
5
@clino 我们也用 gerrit，你们 review implementation 还是 unit test
|
 |
seaswalkerApr 19, 2018
6
有
|
 |
lghApr 19, 2018 via iPhone
7
|
 |
estApr 19, 2018
8
有必要。gitlab 使用中。
|
 |
PerryApr 19, 2018
9
用的 Bitbucket code review，需要 2 个以上同事批准，有些项目还要确保 Test Coverage 在一个阈值之上
觉得很有必要 |
 |
north521Apr 19, 2018
10
没有
有必要，对提升自己的代码水平和减少公司项目的坑很有帮助 |
 |
zpf124Apr 19, 2018
11
没有，
有必要 另外我们连单元测试也不写几个... |
 |
12
我们结对编程。 没开玩笑
|
 |
jadec0derApr 19, 2018
13
有，有必要但是付出的代价太大了
|
 |
huluhuluApr 19, 2018 via iPhone
14
非常有必要，有，使用 gerrit，keyuser 有权限+2，且触发自动编译和自动化 case，都过了才会真正合到 trunk
|
 |
kingcosApr 19, 2018 via iPhone
15
有，2 个看过 Review 过才可以，而且最近正在配置 CI …
|
 |
voidcommaApr 20, 2018 via iPhone
16
有必要。准备在例会时提出并推动。
|
 |
huntzhanApr 20, 2018
17
Required, 必须有人 review 才能 check in
|
 |
PastsongApr 20, 2018
18
有
不强制 是 Proof of Work 的一种方法 |
 |
PHPer233Apr 20, 2018 via iPhone
19
管你怎么写，写出来能跑就行。完成任务，交差回家~
|
 |
GuaidaodlApr 20, 2018
20
有啊. 绝对有必要.
而且对新人来说有人 Review 你成长才快. |
 |
msg7086Apr 20, 2018
21
老代码没有，屎一样的代码满天飞。
新代码所有的 commit 都要 peer review。 |
 |
20015jjwApr 20, 2018 via Android
22
必须得有啊
|
 |
jerryshaoApr 20, 2018
23
我就来看看有没有用 Reviewboard 的
|
 |
randyzhaoApr 20, 2018
25
纯人工的 Review 优点很明显，同时也有些瑕疵。
比如：你平时玩的好的同事，找你帮他 “ Review ” 一下。 如果是要避免这样的漏洞，那么可以设置特定的人 (如: Team Leader) 一定要 Review 过才算行。 不过这样的话，Team Leader 其实也很累。 所以团队里一定得都是志同道合的人，才最好。 |
 |
fanqiangerApr 20, 2018
26
有 review
感觉没什么用 |
 |
nicevarApr 20, 2018 via Android
27
有，相当有必要，有时候人会犯一些很蠢的错误，扫几眼就能发现问题，如果不 review 等部署了才发现浪费大家时间
|
 |
eccstartupApr 20, 2018 via iPhone
28
不 review 别人怎么用你家产品啊
|
 |
chiuApr 20, 2018 via Android
29
有专门一个部门在监督 review 的……每次 review 还要记录好多东西
|
 |
zhq527725Apr 20, 2018
30
有。
肥肠有必要，俺们公司没有一个测试人员，代码质量全靠 code review 和自动化测试。 |
 |
patxApr 20, 2018
31
有
有 |
 |
l00tApr 20, 2018
32
1.没有
2.有必要 其实我喜欢结对编程。每次结对编程的时候感觉效率可高了。 |
 |
EricFutureApr 20, 2018
33
没有，非常有必要
|
 |
xiangyuecnApr 20, 2018
34
好奇你们会不会在自己写的代码里面埋后门，怎样逃过别人 review
|
 |
crayygyApr 20, 2018 via Android
35
有，而且必须两个人同意以后才能提交，而且不存在什么关系好你帮我点一下这种，出了问题大家脸上都不好看，关系好就更帮你认真看了。
|
 |
hasbugApr 20, 2018
36
不存在的
|
|
clinoApr 20, 2018
37
|
 |
AcheronApr 20, 2018
38
1.没有
2.没必要 所有偏离代码本质的东西都是无必要的，比如 review，多余的注释，站立会，代码的本质就是自由，这些边边角角限制了自由，但没必要不一定不做，自己 review 自己的代码就可以了。 |
 |
ifsoar0712Apr 20, 2018
39
@jerryshao +1
|
 |
minbabyApr 20, 2018
40
知道是一回事，做到是另一回事。
想要在项目中，加入 review 这个选项，第一是不能影响产品正常进度（不然老板啊，产品啊，拎着刀就过来了），第二是部门负责人能意识到 Review 的重要性和必要性（发生两次重大事故就“好了”），第三团队成员有这个意思（不然就沦为了“形式“） |
 |
EanCuznaivyApr 20, 2018
41
直接 review PR，然后老板亲自 merge ……
很有必要。 |
 |
vakaraApr 20, 2018
42
有，我们组内强制，Review 很仔细。
公司使用 GitHub。 |
 |
longxboyApr 20, 2018
43
有的，强制 review 加各种静态检查编译检查
|
 |
tonghuashuaiApr 20, 2018
44
有
有必要 强制至少一位同事 review 后才可以 merge |
 |
lsmgeb89Apr 20, 2018
45
有必要啊，没有 review 的公司或组就没必要做了。
|
 |
ioioioioioioiApr 20, 2018
46
有必要的，不然容易有 bug。
|
 |
yuxiaofei93Apr 20, 2018 via iPhone
47
公司里有代码审查，并且出事故代码审查人受同样处罚。
代码审查有必要，至少能避免一眼就看出来的低级失误 |
 |
unidentifiedmeApr 20, 2018
48
有，基本上所有项目强制 code review
很有必要。 所有项目都接 Coverity 有的项目还有 unit test coverage 的要求 |
 |
cccRaimApr 20, 2018
49
有，
三个人 review 才行 |
 |
shaobin0604Apr 20, 2018
50
经历过几家公司，都有 code review，工具
- gerrit - reviewboard - gitlab merge request |
 |
coderluanApr 20, 2018
51
公司没有，各个项目自己要求，肯定是有必要的，但是如果一个项目是时间紧张收益低的一锤子买卖，你懂的。
|
 |
bluefalconjunApr 20, 2018
52
有必要.
gerrit 提交: 脚本会检查 1.代码格式和 commit 关键字; 2.编译; 然后+1; 另外需要其他队员 review+1; 最后 merge 权限的人才能 merge 进 master. |
 |
jasonchen168Apr 20, 2018
53
小公司业务代码都写不完。。。
|
 |
ihaveadrameApr 20, 2018
54
不要问题了 选 C。
我们公司对这块要求比较严格，基本是 pylint flake8 eslint 代码规范检测 tox 单元测试 还有 代码覆盖率测试。 每个 PR 都至少要组长 review 一下。 |
 |
kaedeaApr 20, 2018
56
1. 本来十几人的团队是互相 rv 的。
2. 后来同一个项目多了几个这种规模的团队。 3. 感到他们的代码风格与自己的格格不入后，慢慢放弃了 rv，因为就算提修改建议人家也懒得改。 4. 现在已经没有定期看别人提交了什么代码的习惯了。 |
 |
DeeerApr 20, 2018 via iPhone
57
我遇到的 review 分为两种、一种是成员代码合并 dev 上进行 review、第二种是从 dev 合并到 test 上的 review、一种是确认代码合并时是否存问题、另一种是合并之后进行检查、不知道你说的是哪种 review ？
|
 |
xrlinApr 20, 2018 via iPhone
59
看了大家的回答，我要争取去一个注重代码质量的团队，我很想执行 review ci，但是项目就只有我一个人。
|
 |
RanlerApr 20, 2018
60
有必要
同 Gerrit，强制 2 个人 Review，强制单元测试 PASS 才能合入 |
 |
projectzooApr 20, 2018
61
有
有必要 |
 |
62
代码风格不统一 貌似是推行 review 机制的一个阻碍
|
 |
codeyungApr 20, 2018
63
当然有了 //
|
 |
sgissb1Apr 20, 2018
64
没有 review，也没有必要。但是作为学习有必要看别人的代码。
review 很费时，搞不好就成茶话会或者秀技大会了，同时可能代码写好写坏有关系吗？会因为写的好点老板多发点钱？人家现在只看结果，只关心快速抢占市场等等。 |
 |
wweirApr 20, 2018 via Android
65
有必要，主要审两方面东西：
局部的架构设计 代码审查，包括命名风格之类的检查 |
 |
velaApr 20, 2018
66
有。
有必要。 |
 |
jedihyApr 20, 2018 via iPhone
67
不 review 就能 checkin 这不是做宝搞吗
|
 |
pwfApr 20, 2018 via Android
68
用 crucible 的 cr，还有组内投屏...
|
 |
lizz666Apr 20, 2018
69
老板的意思是最短的时间用最快的方法写出来上线给客户拿钱，等模板开发好，谁还管你代码质量好不好，全是复制粘贴。
我要学技术。。。 |
 |
yurikoApr 20, 2018
70
前东家有
用 gerrit 虽然有必要，但是也不能过分 比如合入到 dev 的 master 分支需要三个人加分，感受下…… |
Select Language