- 主题:各位是从哪里获取code review的动力的?
未经code review批准的代码无法合并到主分支,系统设定,人工合并可以,找其他部门的人有权限的人帮你操作。
然后很快就会出现,你不review别人的代码,等你要review的时候,你就找不到人。就算你是超级大牛,别人不diao你,你也没招。
【 在 dpblue 的大作中提到: 】
: 我们组里谁都可以review代码,头儿貌似也不太关心你是否review同事们的代码,所以我平时都只顾着自己手上的活,懒得去review。但感觉这样不太好,请问各位是从哪里获取code review的动力的?
--
FROM 219.142.54.*
虽然我不懂,但是感觉你这个法子不靠谱啊,万一这个id以后被人弄出来呢?
【 在 dpblue 的大作中提到: 】
: 关于你说的“code review是个挑错性质的东西,不怎么受人待见,而且容易起争执”,这个我也想知道大家是怎么解决的。如果代码真的错了还好说,那种改善型的意见,真不知道是该给还是不该给。
: 举一个例子,我同事写了一个client,需要实现isAvailable(),就是去连一下对端服务器,连上了就返回true。
: 他的方法是调用一个取groups的API,这个API会返回属于某个ID的一大堆groups,但问题是这堆groups根本用不上。
: ...................
--
FROM 219.142.54.*
那你们直接检查一个已经存在的id不就好了么。省的有人拿不存在当一个条件干了些不该干的事情,结果有一天突然存在了。
【 在 dpblue 的大作中提到: 】
: 看来大家提到过好几次这个id被人弄出来的情况啊
: 那我就多解释一下:如果那个id存在,那会返回200 ok,也是连接成功了。检查的时候检查的是200 or 404,如果是其他exception,说明连接失败。
:
--
FROM 219.142.54.*