- 主题:各位是从哪里获取code review的动力的?
问题:如何给同事做code review同时又不会惹人反感?
就像你前面说的,你觉得我说的两个做法都算错,但你楼上认为code review不应该管,也就是说如果是你review他的代码,给了意见说不行,那他一定很不满,心想:”明明不应该管的东西,干嘛非要提,害我没法merge“,这个怎么破?
【 在 PaoloMaldini 的大作中提到: 】
: 看了这个话题下的讨论之后,觉得我对code review的态度还算比较认真的。。
:
--
FROM 120.21.71.*
我至少保证自己team的code review水平让自己满意
要是旁边人水平都比我高那当然更好
【 在 dpblue (deep blue) 的大作中提到: 】
: 问题:如何给同事做code review同时又不会惹人反感?
: 就像你前面说的,你觉得我说的两个做法都算错,但你楼上认为code review不应该管,也就是说如果是你review他的代码,给了意见说不行,那他一定很不满,心想:”明明不应该管的东西,干嘛非要提,害我没法merge“,这个怎么破?
--
FROM 103.107.216.225
楼上诸位对于公司给的code review的任务都如此嫌弃,那为啥会有人认同Linux宣传的“有无数只眼睛帮你检查,很安全”呢?
--
FROM 123.116.198.*
打工让快乐的劳动异化^o^
【 在 leadu (leadu) 的大作中提到: 】
: 楼上诸位对于公司给的code review的任务都如此嫌弃,那为啥会有人认同Linux宣传的“有无数只眼睛帮你检查,很安全”呢?
--
FROM 122.233.189.*
公司是拿钱干活。当然可以拔高,并不妨碍要求高的人往高层次的地方去想,“为了跟公司一起实现自我价值”、“为了服务用户/客户”等等言论都是OK的。但基本面就是一份拿钱的工作,一旦出问题被罚,应该能体会到资本/绩效的无情,这是它的下限。视工作为盖大教堂的,和视工作为砌墙搬砖的,可以区分对待。
linux那个代码review、fuzzy啥的,不同的人有各种目的。
前一阵还有人吐槽华为有人往里面提交低质量的代码,
还有一个老外机构的人为了测试这个代码review到底有效性如何,长期往里面加入有问题的代码,把这事作为研究素材。
有人主动发现、暴露linux的这些问题,说明这个review是有效果的。
【 在 leadu 的大作中提到: 】
: 楼上诸位对于公司给的code review的任务都如此嫌弃,那为啥会有人认同Linux宣传的“有无数只眼睛帮你检查,很安全”呢?
--
修改:z16166 FROM 114.245.195.*
FROM 114.245.195.*
查出来的可以说是review有效的,也可以说还有海恩法则,后面还有很多未暴露的。
取决于以乐观角度还是以悲观角度看。安全问题以乐观角度看,还是太乐观了,哈哈
而且还有Linux heartbleed漏洞呢
所以我觉得Linux所谓的有无数只眼睛帮你看着,也许就是个自欺欺人的说法
全球开发人员负责,其实就是全都不负责。
漏洞可能逃过code review,code review了也查不出来,最后还都不需要负责
【 在 z16166 的大作中提到: 】
: 公司是拿钱干活。当然可以拔高,并不妨碍要求高的人往高层次的地方去想,“为了跟公司一起实现自我价值”、“为了服务用户/客户”等等言论都是OK的。但基本面就是一份拿钱的工作,一旦出问题被罚,应该能体会到资本/绩效的无情,这是它的下限。视工作为盖大教堂的,和视工作为砌墙搬砖的,可以区分对待。
: linux那个代码review、fuzzy啥的,不同的人有各种目的。
: 前一阵还有人吐槽华为有人往里面提交低质量的代码,
: ...................
--
FROM 123.116.198.*
豆大,给钱压kpi都出现如此多的问题的流程,用爱就可以克服么?
【 在 adoal 的大作中提到: 】
: 打工让快乐的劳动异化^o^
:
--
FROM 123.116.198.*
花钱让人看不愿意,但是找代码copy自己用估计会看的
问题是,发现问题了,并不意味着会曝光并修复
【 在 leadu (leadu) 的大作中提到: 】
: 豆大,给钱压kpi都出现如此多的问题的流程,用爱就可以克服么?
--
FROM 119.131.204.*
【 在 iMx 的大作中提到: 】
: 花钱让人看不愿意,但是找代码copy自己用估计会看的
谁用Linux之前会把Linux代码看一遍,增量的也不会看
: 问题是,发现问题了,并不意味着会曝光并修复
对,这个也是个问题,对于黑客来说,自己偷着用获利可能更高
:
: ...................
--
FROM 123.116.198.*
linux的代码从来没人说是全球的开发负责吧,肯定coding和review的人负责,其他人只是有条件一起看。
有个办法可以对比一下,就是把某个公司的私有代码拿出来一定数量,和linux同样规模的代码比,看看哪个问题多一些、严重一些。不过这个也涉及到coding/review的人的水平的对齐。
【 在 leadu 的大作中提到: 】
: 查出来的可以说是review有效的,也可以说还有海恩法则,后面还有很多未暴露的。
: 取决于以乐观角度还是以悲观角度看。安全问题以乐观角度看,还是太乐观了,哈哈
: 而且还有Linux heartbleed漏洞呢
: ...................
--
修改:z16166 FROM 114.245.195.*
FROM 114.245.195.*