- 主题:各位是从哪里获取code review的动力的?
别人家的狗遍地拉屎,你也可能哪天踩到吧
【 在 dpblue (deep blue) 的大作中提到: 】
: 我们组里谁都可以review代码,头儿貌似也不太关心你是否review同事们的代码,所以我平时都只顾着自己手上的活,懒得去review。但感觉这样不太好,请问各位是从哪里获取code review的动力的?
--
FROM 27.91.71.*
很多时候看pr的感觉是,自己坐飞机时外面有两个鬼在锯飞机翅膀
【 在 allegro (静水流深) 的大作中提到: 】
: 我只review别人改我代码的地方。
--
FROM 27.91.71.*
连带责任是贵司想出来的高招?
【 在 z16166 (Netguy) 的大作中提到: 】
: 动力主要来自KPI。。。KPI要求流程里必须有这步,发布出去出了问题可能要担连带责任(主要责任还是写这个代码的)
: code review是个挑错性质的东西,不怎么受人待见,而且容易起争执
: 极少数情况下是为了学习别人的好代码,那种已经不算review了,算learning。
: ...................
--
FROM 27.91.71.*
违反敏捷开发思想了
【 在 z16166 (Netguy) 的大作中提到: 】
: 开发(包括写代码的、看代码的)、测试,一起为bug负责,不是流行惯例?
--
FROM 27.91.71.*
bug不是个人问题,是系统性问题
【 在 z16166 (Netguy) 的大作中提到: 】
: 敏捷也得找责任人,出了问题都得有人背锅。
: 企业是功利组织,让企业利润受到影响的,都得受罚。让企业赚大钱的,受奖。
--
FROM 27.91.71.*
这两个方法不等价啊
【 在 dpblue (deep blue) 的大作中提到: 】
: 关于你说的“code review是个挑错性质的东西,不怎么受人待见,而且容易起争执”,这个我也想知道大家是怎么解决的。如果代码真的错了还好说,那种改善型的意见,真不知道是该给还是不该给。
: 举一个例子,我同事写了一个client,需要实现isAvailable(),就是去连一下对端服务器,连上了就返回true。
: 他的方法是调用一个取groups的API,这个API会返回属于某个ID的一大堆groups,但问题是这堆groups根本用不上。
: ...................
--
FROM 27.91.71.*
一般都是事先安排好小弟,无脑approve自己的pr
【 在 keygen (失落灵魂之囚) 的大作中提到: 】
: 不是自己到处求别人review自己的pr吗?
: 不然没法merge啊
--
FROM 27.91.71.*
这两个不矛盾
【 在 beep (菜M.喵星耗子) 的大作中提到: 】
: 建议,不要搞时空不同步的code review,而要搞时空同步的结对编程,俩个人凑在一起同时写一段东西,效果和成本平均下来比code review好些
--
FROM 27.91.71.*
pairing就两人啊,review可以N个人
【 在 beep (菜M.喵星耗子) 的大作中提到: 】
: 标 题: Re: 各位是从哪里获取code review的动力的?
: 发信站: 水木社区 (Tue Nov 30 12:06:26 2021), 站内
:
: 矛盾的啊,pairing成本本来就比review高,如果pairing出来的东西还要在review,成本就上天了
:
: 【 在 xiaoju (可爱的龙猫) 的大作中提到: 】
: : 标 题: Re: 各位是从哪里获取code review的动力的?
: : 发信站: 水木社区 (Mon Nov 29 18:27:38 2021), 站内
: :
: : 这两个不矛盾
: :
: : 【 在 beep (菜M.喵星耗子) 的大作中提到: 】
: : : 建议,不要搞时空不同步的code review,而要搞时空同步的结对编程,俩个人凑在一起同时写一段东西,效果和成本平均下来比code review好些
: :
: :
: : --
: :
: : ※ 来源:·水木社区 mysmth.net·[FROM: 27.91.71.*]
:
:
: --
:
: ※ 来源:·水木社区 mysmth.net·[FROM: 123.120.160.*]
--
FROM 27.91.71.*
很多公司QA不测任何bug,只掌管大局,测试的人叫tester
【 在 irreallich (lich) 的大作中提到: 】
: 现在很多互联网公司的qa是质量监察部门
: 什么叫监察,我想你肯定明白
: 架子大得很
: ...................
--
FROM 27.91.71.*