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