#2 kurten 2015年09月28日 本来我在我司想推code review,but 同事懒惯了,搞不起来。我们用gitlab自己搭建了一个服务,用起来的话,虽然code review工具差点,但是还是可以用的。目前策略还是靠自己自律,多写测试,多做测试。但是,人多少总有点惰性在那,这个效果不是很理想。偶尔还是会出点故障。 欣赏 0 反对 0 回复本楼
#4 尼克徐 2015年09月28日 Code Review是个好习惯。 技术强的队伍,也能够通过Code Review,用不同视角审视彼此的代码,从而相互促进。 另外,Code Reiview的过程,还能帮助团队彼此熟悉各自的工作,实际上起到了互为备份的作用。 欣赏 0 反对 0 回复本楼
#8 tinyfool 2015年09月28日 我现在不做Code review当然跟我们人少有关系,谁会在10个人和1000个人的团队用相同的方法做管理啊。 不过我们人少也是因为我不喜欢太重的管理方式,我们可以雇更多的人,不过刻意的少雇,同时非常注意能力和效率而已。 欣赏 0 反对 0 回复本楼
#14 damon 2015年09月28日 不做code review就会像@tinytool 前段时间的 select rand是没有办法为啥这么做的,这个实在是太坑太坑的解决方案。 欣赏 0 反对 0 回复本楼
#15 tinyfool 2015年09月29日 14楼 @damon 这个网站是我一个人业余做着玩的,code review给谁看, 又不是我们公司产品,而且select rand靠review我觉得是不对的,这个东西只在某种负载下有问题,实际上,主要是因为我一直懒得写一个脚本清理mail队列的已发邮件,否则的话,这个方案其实也是可以的。 欣赏 0 反对 0 回复本楼
#17 biiigfish 2015年09月29日 好的执行,还是一定程度上依赖于好的监督的。生产-品保,研发-测试,coding-review。。。 如果没有了监督,你的好坏其实就变成了人家的猜测。当然,你可以说是给了100%的信任,但是,这个信任真的靠得住吗?最好的程序员也会干蠢事,因为他也是个人。回头来讲,一些问题的产生,并不是因为他干了蠢事,但是让他自己去证明显然是困难的。后一个是分谤、分责的需求。 不过,事情要是变成找茬和挑毛病,就走偏了。毕竟,做成事,还是要靠共同信仰的,并不能依靠相互制约。 欣赏 0 反对 0 回复本楼
#19 frank4565 2015年09月29日 我所在的公司用 Crucible 做 Code review,配合 Perforce/Git,JIRA 等工具。要求没过 Code review 不能提交代码。 欣赏 0 反对 0 回复本楼
#22 pinxue 2015年10月01日 你们要是接手一个有10年以上历史不用太大几十万就行的项目,没有熟手来做 code review,哼哼。 要是已经上 xp 了,当我没说。 欣赏 0 反对 0 回复本楼
#23 tinyfool 2015年10月02日 22楼 @pinxue 我们这次做iOS转Android,何止涉及到几十万行代码。自己写的代码几万,涉及到的代码至少百万吧。 欣赏 0 反对 0 回复本楼
#27 tsiannian 2015年10月07日 你的策略是不可行的,至少你的公司不是最好的it公司,为什么一定要雇佣最好的工程师?就管理上而言,过多的依赖员工的个人能力,不仅对员工不利,也对公司不利。 欣赏 0 反对 0 回复本楼