英语轻松读发新版了,欢迎下载、更新

我们公司从来不做code review,我们的策略是只雇佣最好的工程师

tinyfool 发布于 2015年09月28日
无人欣赏。

你们呢?你们的策略是啥?

共27条回复
tinyfool 回复于 2015年09月28日

问这个是因为,刚才有人问我我公司在用什么东西Code review,我一想,我们也没Code review啊

kurten 回复于 2015年09月28日

本来我在我司想推code review,but 同事懒惯了,搞不起来。我们用gitlab自己搭建了一个服务,用起来的话,虽然code review工具差点,但是还是可以用的。目前策略还是靠自己自律,多写测试,多做测试。但是,人多少总有点惰性在那,这个效果不是很理想。偶尔还是会出点故障。

baichunpeng 回复于 2015年09月28日

贵司的Heart Review甩CodeReview几条街

尼克徐 回复于 2015年09月28日

Code Review是个好习惯。

技术强的队伍,也能够通过Code Review,用不同视角审视彼此的代码,从而相互促进。

另外,Code Reiview的过程,还能帮助团队彼此熟悉各自的工作,实际上起到了互为备份的作用。

jasonim 回复于 2015年09月28日

@tinyfool 那是你团队还没成规模, 还有现在的代码还在你的可控范围内(这个可控只是你的标准),哈哈!

tinyfool 回复于 2015年09月28日

5楼 @jasonim 对啊,你说的都是显而易见的啊

tinyfool 回复于 2015年09月28日

3楼 @baichunpeng Heart Review是啥?

tinyfool 回复于 2015年09月28日

我现在不做Code review当然跟我们人少有关系,谁会在10个人和1000个人的团队用相同的方法做管理啊。

不过我们人少也是因为我不喜欢太重的管理方式,我们可以雇更多的人,不过刻意的少雇,同时非常注意能力和效率而已。

pinxue 回复于 2015年09月28日

你就是懒得看吧

tinyfool 回复于 2015年09月28日

9楼 @pinxue 不是啊,现在Code review就算要做,也是CTO的事情,不是我的事情了

minddriven 回复于 2015年09月28日

Code review 会伤感情,碰上无药可救的就是浪费时间。

能请合适的人是最好的。

tinyfool 回复于 2015年09月28日

11楼 @minddriven 哈哈哈

shininguang 回复于 2015年09月28日

测试通过就可以,也是不做code review

damon 回复于 2015年09月28日

不做code review就会像@tinytool 前段时间的 select rand是没有办法为啥这么做的,这个实在是太坑太坑的解决方案。

tinyfool 回复于 2015年09月29日

14楼 @damon 这个网站是我一个人业余做着玩的,code review给谁看, 又不是我们公司产品,而且select rand靠review我觉得是不对的,这个东西只在某种负载下有问题,实际上,主要是因为我一直懒得写一个脚本清理mail队列的已发邮件,否则的话,这个方案其实也是可以的。

东岸往事 回复于 2015年09月29日

code review 是撒子唉。。。

biiigfish 回复于 2015年09月29日

好的执行,还是一定程度上依赖于好的监督的。生产-品保,研发-测试,coding-review。。。

如果没有了监督,你的好坏其实就变成了人家的猜测。当然,你可以说是给了100%的信任,但是,这个信任真的靠得住吗?最好的程序员也会干蠢事,因为他也是个人。回头来讲,一些问题的产生,并不是因为他干了蠢事,但是让他自己去证明显然是困难的。后一个是分谤、分责的需求。

不过,事情要是变成找茬和挑毛病,就走偏了。毕竟,做成事,还是要靠共同信仰的,并不能依靠相互制约。

zhiyuan 回复于 2015年09月29日

人少项目复杂度低的前提下可行;

frank4565 回复于 2015年09月29日

我所在的公司用 Crucible 做 Code review,配合 Perforce/Git,JIRA 等工具。要求没过 Code review 不能提交代码。

whueric 回复于 2015年09月29日

在之前公司做电信方面的软件,感觉code review确实可以发现不少问题。

梦中醒不过来 回复于 2015年09月30日

盘子大了要培养各梯队人才,对于新手才有必要code review把把关吧

pinxue 回复于 2015年10月01日

你们要是接手一个有10年以上历史不用太大几十万就行的项目,没有熟手来做 code review,哼哼。

要是已经上 xp 了,当我没说。

tinyfool 回复于 2015年10月02日

22楼 @pinxue 我们这次做iOS转Android,何止涉及到几十万行代码。自己写的代码几万,涉及到的代码至少百万吧。

pinxue 回复于 2015年10月04日

23楼 @tinyfool 几年?

tinyfool 回复于 2015年10月04日

24楼 @pinxue 我们的项目你不是知道么,到现在折腾了1年了吧

pinxue 回复于 2015年10月04日

25楼 @tinyfool 我当然知道呀,我说的是维护历史长的大项目没 code review 真不行呀。

tsiannian 回复于 2015年10月07日

你的策略是不可行的,至少你的公司不是最好的it公司,为什么一定要雇佣最好的工程师?就管理上而言,过多的依赖员工的个人能力,不仅对员工不利,也对公司不利。

本帖有27个回复,因为您没有注册或者登录本站,所以,只能看到本帖的10条回复。如果想看到全部回复,请注册或者登录本站。

登录 或者 注册
[顶 楼]
|
|
[底 楼]
|
|
[首 页]