由买买提看人间百态

boards

本页内容为未名空间相应帖子的节选和存档,一周内的贴子最多显示50字,超过一周显示500字 访问原贴
Programming版 - Code review代码审查怎么做
进入Programming版参与讨论
1 (共1页)
w*****g
发帖数: 4298
1
发现有个组员老是不喜欢别人提comments, 立即用自己的观点反驳并把comment列为已
解决并关闭,尽管巳经多次说过要测试代码并把测试结果和数据列出来作为解决
comment的依据,但别人就是不听, 仍然固执地用自己的观点
h****e
发帖数: 2125
2
你如果是reviewer的话,他不得求着你approve吗?他不合作,你不approve不就完了
如果他敢闹的话,整理相关证据找老板

【在 w*****g 的大作中提到】
: 发现有个组员老是不喜欢别人提comments, 立即用自己的观点反驳并把comment列为已
: 解决并关闭,尽管巳经多次说过要测试代码并把测试结果和数据列出来作为解决
: comment的依据,但别人就是不听, 仍然固执地用自己的观点

d*******n
发帖数: 43
3
有具体例子吗?
我最烦自己的代码被挑刺,明明两种方法都行,ta非要说用另一种。和他辩论吧浪费时
间,听他的吧下次更甚。
最后不得不微信成立个互相approve小组,把pr直接发微信里互相approve,唉。后面那
人走了pr里面才清净。
s********d
发帖数: 162
4
Profile 一下,谁的cpu 快,mem 少,用谁的?
n******t
发帖数: 4406
5
你这个想法看起来有道理,在实际工作中不work,因为你打交道的是人不是机器。
如果你已经知道他的comment make sense,但是还是要别人浪费时间来证明,这属于
asshole。当然有时候这么做是必须的,但是每次这么干都是有相应代价的,你必须要
谨慎使用,说白了,know what you are doing.

【在 w*****g 的大作中提到】
: 发现有个组员老是不喜欢别人提comments, 立即用自己的观点反驳并把comment列为已
: 解决并关闭,尽管巳经多次说过要测试代码并把测试结果和数据列出来作为解决
: comment的依据,但别人就是不听, 仍然固执地用自己的观点

n******t
发帖数: 4406
6
所以很多人抱怨公司同事不好相处,但是从来不从别人角度考虑问题。
公司做事,你写几行comment就要别人花几十分钟乃至一小时做测试来证明别这个
comment没用,如果他真的这么搞, 最后绩效不行的肯定是他。长此以往,别人和你能
关系好才有鬼了。

【在 d*******n 的大作中提到】
: 有具体例子吗?
: 我最烦自己的代码被挑刺,明明两种方法都行,ta非要说用另一种。和他辩论吧浪费时
: 间,听他的吧下次更甚。
: 最后不得不微信成立个互相approve小组,把pr直接发微信里互相approve,唉。后面那
: 人走了pr里面才清净。

w*****g
发帖数: 4298
7
码工码农脾气暴,不喜欢别人提不同意见,当然别人都绕过走,也有人根本就不看代码
直接approve. 久而久之,代码库的质量越来越差。这就涉及到了要维持人与人的关系
,还是要保产品质量的问题,这两个之间出现了矛盾。作为领导,怎么解决这个矛盾?
n*******0
发帖数: 2002
8
多简单个事儿,把该提的comments 放进去,然后当场accept。
丫要是要点脸就自己照着comments改了之后commit。
丫要是不要脸就原样commit。
出了事情都是丫的锅。完美。

【在 w*****g 的大作中提到】
: 发现有个组员老是不喜欢别人提comments, 立即用自己的观点反驳并把comment列为已
: 解决并关闭,尽管巳经多次说过要测试代码并把测试结果和数据列出来作为解决
: comment的依据,但别人就是不听, 仍然固执地用自己的观点

w**s
发帖数: 1911
9
你出左脚,他非得他喜欢出右脚

【在 d*******n 的大作中提到】
: 有具体例子吗?
: 我最烦自己的代码被挑刺,明明两种方法都行,ta非要说用另一种。和他辩论吧浪费时
: 间,听他的吧下次更甚。
: 最后不得不微信成立个互相approve小组,把pr直接发微信里互相approve,唉。后面那
: 人走了pr里面才清净。

h****e
发帖数: 2125
10
如果他水平比较牛就是听不进别人的意见,大部分也都是cosmetic小问题的话,没必要
过多纠缠。如果水平很烂但是脾气还大,而且和老板关系也一般般的话,这种人就得往
死里整,PR死活不approve,打到他服为止
最难办的是水平不行但是和老板关系铁,那就得你自己掂量着办了

【在 w*****g 的大作中提到】
: 码工码农脾气暴,不喜欢别人提不同意见,当然别人都绕过走,也有人根本就不看代码
: 直接approve. 久而久之,代码库的质量越来越差。这就涉及到了要维持人与人的关系
: ,还是要保产品质量的问题,这两个之间出现了矛盾。作为领导,怎么解决这个矛盾?

w*****g
发帖数: 4298
11
呵呵,技术大牛和老板铁哥们的对决,老板该怎么办?

【在 h****e 的大作中提到】
: 如果他水平比较牛就是听不进别人的意见,大部分也都是cosmetic小问题的话,没必要
: 过多纠缠。如果水平很烂但是脾气还大,而且和老板关系也一般般的话,这种人就得往
: 死里整,PR死活不approve,打到他服为止
: 最难办的是水平不行但是和老板关系铁,那就得你自己掂量着办了

h****e
发帖数: 2125
12
如果是我肯定对事不对人咯,谁更有道理就听谁的

【在 w*****g 的大作中提到】
: 呵呵,技术大牛和老板铁哥们的对决,老板该怎么办?
m*****n
发帖数: 3575
13
你需要考虑一下老板多重视这个code review
如果老板重视,就报告老板,大不了他走人。
我最烦用烂代码充数的,要我是老板,肯定占你。
C*****l
发帖数: 1
14
如果你们组是必须approve才能check in,那你不approve就是了。如果不是,那你就睁
一只眼闭一只眼好了

【在 w*****g 的大作中提到】
: 发现有个组员老是不喜欢别人提comments, 立即用自己的观点反驳并把comment列为已
: 解决并关闭,尽管巳经多次说过要测试代码并把测试结果和数据列出来作为解决
: comment的依据,但别人就是不听, 仍然固执地用自己的观点

C*****l
发帖数: 1
15
不是老板的话笑笑走开最好,是老板的话,那必须有点action

【在 m*****n 的大作中提到】
: 你需要考虑一下老板多重视这个code review
: 如果老板重视,就报告老板,大不了他走人。
: 我最烦用烂代码充数的,要我是老板,肯定占你。

y****w
发帖数: 3747
16
就是这样,不懂换位思考。很多时候最不好处的反而不是脾气差的,而是懒且喜欢麻烦
人的,大家都很忙好不好。

【在 n******t 的大作中提到】
: 所以很多人抱怨公司同事不好相处,但是从来不从别人角度考虑问题。
: 公司做事,你写几行comment就要别人花几十分钟乃至一小时做测试来证明别这个
: comment没用,如果他真的这么搞, 最后绩效不行的肯定是他。长此以往,别人和你能
: 关系好才有鬼了。

y****w
发帖数: 3747
17
当领导是定规矩的 你得根据你的人设定相应的规矩。
我一般对不是代码错误的方面推两条,
1. 对于不同解法,不同style,不同技术路线等,那么由owner自己决定,非否定即肯
定,owner也负责代码质量。
2. 如果reviewer认为有必要讨论或者有不清楚的地方,除非你们两个关系好,底下商
商量量就搞定了,那就都到email group里解决。请提意见的负责写清楚背景问题方案
,数据不对齐的请拿出你做过的research来 -- 比如人家统计是30,你觉得是29,至少
你找出那个1先吧。 有些人自己不先搞清楚,上来什么问题不说,能不能耽误你五分钟
打个电话,结果拉着人家一小时不放,烦死。
工作场合的不合很多都是因为不尊重别人的时间或者做事情敷衍引起的,本质上都是因
为一些人的问题引发别人反感。 开始的时候我关注点在态度,后来就变成关注做事的
方式,而且越来越觉得很多对respect方面抱怨的真正问题反而出在抱怨的人身上。

【在 w*****g 的大作中提到】
: 码工码农脾气暴,不喜欢别人提不同意见,当然别人都绕过走,也有人根本就不看代码
: 直接approve. 久而久之,代码库的质量越来越差。这就涉及到了要维持人与人的关系
: ,还是要保产品质量的问题,这两个之间出现了矛盾。作为领导,怎么解决这个矛盾?

w*****g
发帖数: 4298
18
工作场合的不合很多都是因为不尊重别人的时间或者做事情敷衍引起的,本质上都是因
为一些人的问题引发别人反感。 开始的时候我关注点在态度,后来就变成关注做事的
方式,而且越来越觉得很多对respect方面抱怨的真正问题反而出在抱怨的人身上。
关系重要还是质量重要? 码工拒绝写单元测试,一段五十行的function, 一个月来回
改过八遍,还在production出问题。为何? 码工脾气暴,不喜欢别人提意见,别人都
绕过走,根本就不看代码直接approve.
h****e
发帖数: 2125
19
你这啥公司,老板是干吗吃的?

【在 w*****g 的大作中提到】
: 工作场合的不合很多都是因为不尊重别人的时间或者做事情敷衍引起的,本质上都是因
: 为一些人的问题引发别人反感。 开始的时候我关注点在态度,后来就变成关注做事的
: 方式,而且越来越觉得很多对respect方面抱怨的真正问题反而出在抱怨的人身上。
: 关系重要还是质量重要? 码工拒绝写单元测试,一段五十行的function, 一个月来回
: 改过八遍,还在production出问题。为何? 码工脾气暴,不喜欢别人提意见,别人都
: 绕过走,根本就不看代码直接approve.

w*****g
发帖数: 4298
20
发现有个组员老是不喜欢别人提comments, 立即用自己的观点反驳并把comment列为已
解决并关闭,尽管巳经多次说过要测试代码并把测试结果和数据列出来作为解决
comment的依据,但别人就是不听, 仍然固执地用自己的观点
h****e
发帖数: 2125
21
你如果是reviewer的话,他不得求着你approve吗?他不合作,你不approve不就完了
如果他敢闹的话,整理相关证据找老板

【在 w*****g 的大作中提到】
: 发现有个组员老是不喜欢别人提comments, 立即用自己的观点反驳并把comment列为已
: 解决并关闭,尽管巳经多次说过要测试代码并把测试结果和数据列出来作为解决
: comment的依据,但别人就是不听, 仍然固执地用自己的观点

d*******n
发帖数: 43
22
有具体例子吗?
我最烦自己的代码被挑刺,明明两种方法都行,ta非要说用另一种。和他辩论吧浪费时
间,听他的吧下次更甚。
最后不得不微信成立个互相approve小组,把pr直接发微信里互相approve,唉。后面那
人走了pr里面才清净。
s********d
发帖数: 162
23
Profile 一下,谁的cpu 快,mem 少,用谁的?
n******t
发帖数: 4406
24
你这个想法看起来有道理,在实际工作中不work,因为你打交道的是人不是机器。
如果你已经知道他的comment make sense,但是还是要别人浪费时间来证明,这属于
asshole。当然有时候这么做是必须的,但是每次这么干都是有相应代价的,你必须要
谨慎使用,说白了,know what you are doing.

【在 w*****g 的大作中提到】
: 发现有个组员老是不喜欢别人提comments, 立即用自己的观点反驳并把comment列为已
: 解决并关闭,尽管巳经多次说过要测试代码并把测试结果和数据列出来作为解决
: comment的依据,但别人就是不听, 仍然固执地用自己的观点

n******t
发帖数: 4406
25
所以很多人抱怨公司同事不好相处,但是从来不从别人角度考虑问题。
公司做事,你写几行comment就要别人花几十分钟乃至一小时做测试来证明别这个
comment没用,如果他真的这么搞, 最后绩效不行的肯定是他。长此以往,别人和你能
关系好才有鬼了。

【在 d*******n 的大作中提到】
: 有具体例子吗?
: 我最烦自己的代码被挑刺,明明两种方法都行,ta非要说用另一种。和他辩论吧浪费时
: 间,听他的吧下次更甚。
: 最后不得不微信成立个互相approve小组,把pr直接发微信里互相approve,唉。后面那
: 人走了pr里面才清净。

w*****g
发帖数: 4298
26
码工码农脾气暴,不喜欢别人提不同意见,当然别人都绕过走,也有人根本就不看代码
直接approve. 久而久之,代码库的质量越来越差。这就涉及到了要维持人与人的关系
,还是要保产品质量的问题,这两个之间出现了矛盾。作为领导,怎么解决这个矛盾?
n*******0
发帖数: 2002
27
多简单个事儿,把该提的comments 放进去,然后当场accept。
丫要是要点脸就自己照着comments改了之后commit。
丫要是不要脸就原样commit。
出了事情都是丫的锅。完美。

【在 w*****g 的大作中提到】
: 发现有个组员老是不喜欢别人提comments, 立即用自己的观点反驳并把comment列为已
: 解决并关闭,尽管巳经多次说过要测试代码并把测试结果和数据列出来作为解决
: comment的依据,但别人就是不听, 仍然固执地用自己的观点

h****e
发帖数: 2125
28
如果他水平比较牛就是听不进别人的意见,大部分也都是cosmetic小问题的话,没必要
过多纠缠。如果水平很烂但是脾气还大,而且和老板关系也一般般的话,这种人就得往
死里整,PR死活不approve,打到他服为止
最难办的是水平不行但是和老板关系铁,那就得你自己掂量着办了

【在 w*****g 的大作中提到】
: 码工码农脾气暴,不喜欢别人提不同意见,当然别人都绕过走,也有人根本就不看代码
: 直接approve. 久而久之,代码库的质量越来越差。这就涉及到了要维持人与人的关系
: ,还是要保产品质量的问题,这两个之间出现了矛盾。作为领导,怎么解决这个矛盾?

w*****g
发帖数: 4298
29
呵呵,技术大牛和老板铁哥们的对决,老板该怎么办?

【在 h****e 的大作中提到】
: 如果他水平比较牛就是听不进别人的意见,大部分也都是cosmetic小问题的话,没必要
: 过多纠缠。如果水平很烂但是脾气还大,而且和老板关系也一般般的话,这种人就得往
: 死里整,PR死活不approve,打到他服为止
: 最难办的是水平不行但是和老板关系铁,那就得你自己掂量着办了

h****e
发帖数: 2125
30
如果是我肯定对事不对人咯,谁更有道理就听谁的

【在 w*****g 的大作中提到】
: 呵呵,技术大牛和老板铁哥们的对决,老板该怎么办?
m*****n
发帖数: 3575
31
你需要考虑一下老板多重视这个code review
如果老板重视,就报告老板,大不了他走人。
我最烦用烂代码充数的,要我是老板,肯定占你。
C*****l
发帖数: 1
32
如果你们组是必须approve才能check in,那你不approve就是了。如果不是,那你就睁
一只眼闭一只眼好了

【在 w*****g 的大作中提到】
: 发现有个组员老是不喜欢别人提comments, 立即用自己的观点反驳并把comment列为已
: 解决并关闭,尽管巳经多次说过要测试代码并把测试结果和数据列出来作为解决
: comment的依据,但别人就是不听, 仍然固执地用自己的观点

C*****l
发帖数: 1
33
不是老板的话笑笑走开最好,是老板的话,那必须有点action

【在 m*****n 的大作中提到】
: 你需要考虑一下老板多重视这个code review
: 如果老板重视,就报告老板,大不了他走人。
: 我最烦用烂代码充数的,要我是老板,肯定占你。

y****w
发帖数: 3747
34
就是这样,不懂换位思考。很多时候最不好处的反而不是脾气差的,而是懒且喜欢麻烦
人的,大家都很忙好不好。

【在 n******t 的大作中提到】
: 所以很多人抱怨公司同事不好相处,但是从来不从别人角度考虑问题。
: 公司做事,你写几行comment就要别人花几十分钟乃至一小时做测试来证明别这个
: comment没用,如果他真的这么搞, 最后绩效不行的肯定是他。长此以往,别人和你能
: 关系好才有鬼了。

y****w
发帖数: 3747
35
当领导是定规矩的 你得根据你的人设定相应的规矩。
我一般对不是代码错误的方面推两条,
1. 对于不同解法,不同style,不同技术路线等,那么由owner自己决定,非否定即肯
定,owner也负责代码质量。
2. 如果reviewer认为有必要讨论或者有不清楚的地方,除非你们两个关系好,底下商
商量量就搞定了,那就都到email group里解决。请提意见的负责写清楚背景问题方案
,数据不对齐的请拿出你做过的research来 -- 比如人家统计是30,你觉得是29,至少
你找出那个1先吧。 有些人自己不先搞清楚,上来什么问题不说,能不能耽误你五分钟
打个电话,结果拉着人家一小时不放,烦死。
工作场合的不合很多都是因为不尊重别人的时间或者做事情敷衍引起的,本质上都是因
为一些人的问题引发别人反感。 开始的时候我关注点在态度,后来就变成关注做事的
方式,而且越来越觉得很多对respect方面抱怨的真正问题反而出在抱怨的人身上。

【在 w*****g 的大作中提到】
: 码工码农脾气暴,不喜欢别人提不同意见,当然别人都绕过走,也有人根本就不看代码
: 直接approve. 久而久之,代码库的质量越来越差。这就涉及到了要维持人与人的关系
: ,还是要保产品质量的问题,这两个之间出现了矛盾。作为领导,怎么解决这个矛盾?

w*****g
发帖数: 4298
36
工作场合的不合很多都是因为不尊重别人的时间或者做事情敷衍引起的,本质上都是因
为一些人的问题引发别人反感。 开始的时候我关注点在态度,后来就变成关注做事的
方式,而且越来越觉得很多对respect方面抱怨的真正问题反而出在抱怨的人身上。
关系重要还是质量重要? 码工拒绝写单元测试,一段五十行的function, 一个月来回
改过八遍,还在production出问题。为何? 码工脾气暴,不喜欢别人提意见,别人都
绕过走,根本就不看代码直接approve.
h****e
发帖数: 2125
37
你这啥公司,老板是干吗吃的?

【在 w*****g 的大作中提到】
: 工作场合的不合很多都是因为不尊重别人的时间或者做事情敷衍引起的,本质上都是因
: 为一些人的问题引发别人反感。 开始的时候我关注点在态度,后来就变成关注做事的
: 方式,而且越来越觉得很多对respect方面抱怨的真正问题反而出在抱怨的人身上。
: 关系重要还是质量重要? 码工拒绝写单元测试,一段五十行的function, 一个月来回
: 改过八遍,还在production出问题。为何? 码工脾气暴,不喜欢别人提意见,别人都
: 绕过走,根本就不看代码直接approve.

x***l
发帖数: 315
38
Management is the least efficient activity in your organization. Usually
when workers escalate to the appropriate manager to address the issue,
manager transforms the performance issue into a relationship issue: it’s
not that the peer isn’t performing, it’s just that the two of you don’t
like each other. Instead of being the manager’s responsibility to resolve
the performance issue, it’s now the main character’s responsibility.
The appropriate manager is almost always aware of the underlying issue, and
for some reason they’re simply unwilling to confront it. You think that you
’re bringing a new problem to that manager to solve, but what you’re
actually doing is trying to hold that manager accountable for not solving a
known problem.
y****w
发帖数: 3747
39
对你这个情况,如果你是reviewer, 就稍麻烦一点,comment里面放上你的test cases
。我一般就这么干,这个list通过就行,比较明白。 我从来不介意comment长点,能想
到的尽量都说清楚。另外style具体实现这些不要指手画脚。
如果你是老板,看看这个交流过程。coder可能经验不足,可能水平不够,也可能只是
因为关注点就不在这里只是顺便搞一把,还有根本不是owner只是过来帮忙了一下。 提
意见的是怎么提出问题的也得关注下,有时候要求不是一次说清楚,今天一条明天一条
,这个过程其实很拱火,有时候会让人想是不是在故意给人下套。



【在 w*****g 的大作中提到】
: 工作场合的不合很多都是因为不尊重别人的时间或者做事情敷衍引起的,本质上都是因
: 为一些人的问题引发别人反感。 开始的时候我关注点在态度,后来就变成关注做事的
: 方式,而且越来越觉得很多对respect方面抱怨的真正问题反而出在抱怨的人身上。
: 关系重要还是质量重要? 码工拒绝写单元测试,一段五十行的function, 一个月来回
: 改过八遍,还在production出问题。为何? 码工脾气暴,不喜欢别人提意见,别人都
: 绕过走,根本就不看代码直接approve.

f*****4
发帖数: 1
40
那也是创造工作机会,bug多,对TA自己没好处,但某种意义上对大家的饭碗有好处
改变别人是很难的事情

【在 w*****g 的大作中提到】
: 工作场合的不合很多都是因为不尊重别人的时间或者做事情敷衍引起的,本质上都是因
: 为一些人的问题引发别人反感。 开始的时候我关注点在态度,后来就变成关注做事的
: 方式,而且越来越觉得很多对respect方面抱怨的真正问题反而出在抱怨的人身上。
: 关系重要还是质量重要? 码工拒绝写单元测试,一段五十行的function, 一个月来回
: 改过八遍,还在production出问题。为何? 码工脾气暴,不喜欢别人提意见,别人都
: 绕过走,根本就不看代码直接approve.

1 (共1页)
进入Programming版参与讨论