- 主题:各位是从哪里获取code review的动力的？
我们组里谁都可以review代码，头儿貌似也不太关心你是否review同事们的代码，所以我平时都只顾着自己手上的活，懒得去review。但感觉这样不太好，请问各位是从哪里获取code review的动力的？
--
FROM 120.21.44.*
关于你说的“code review是个挑错性质的东西，不怎么受人待见，而且容易起争执”，这个我也想知道大家是怎么解决的。如果代码真的错了还好说，那种改善型的意见，真不知道是该给还是不该给。
举一个例子，我同事写了一个client，需要实现isAvailable()，就是去连一下对端服务器，连上了就返回true。
他的方法是调用一个取groups的API，这个API会返回属于某个ID的一大堆groups，但问题是这堆groups根本用不上。
我说你随便传一个不存在的ID就行，因为不存在人家会返回404，也是说明连上了。但这哥们就是不改，估计他心里是一大堆草泥马，想着反正这个代码能用，干嘛非要改。
【 在 z16166 的大作中提到: 】
: 动力主要来自KPI。。。KPI要求流程里必须有这步，发布出去出了问题可能要担连带责任（主要责任还是写这个代码的）
: code review是个挑错性质的东西，不怎么受人待见，而且容易起争执
: 极少数情况下是为了学习别人的好代码，那种已经不算review了，算learning。
--
FROM 120.21.44.*
这个微服务倒是有返回200的接口，叫deepcheck，表示自己可用并且自己可以访问到依赖的所有资源
不过我们只用到了这个微服务的groups api，所以就没有用deepcheck
【 在 PaoloMaldini 的大作中提到: 】
: 你们俩的实现都很奇葩。。且不说这玩意儿有用没用
: 即便要做，也应该是让服务端直接写个返回200的空接口专供测试用吧
: 哪能用业务接口测试去
: ...................
--
FROM 120.21.44.*
嗯，看来以后我要改自己的review习惯了
因为以前是C++码农，眼里容不下任何效率不高的实现
【 在 xunery 的大作中提到: 】
: 个人感觉得原则上这不属于code review该管的范围，当然也分情况。说说我的观点
: 1.code review的目的找错误，明显的不明显的错误。你举例这种如果没有重大影响，编码人员有自己实现的自由。
: 2.如果成员水平差距挺大，应该制定详尽的编码规范，能避免很多类似这样扯皮
: ...................
--
FROM 120.21.44.*
这个方法就一个简单目的：看对方那个微服务是否可用，返回Boolean
【 在 xiaoju 的大作中提到: 】
: 这两个方法不等价啊
:
--
FROM 120.21.44.*
我们组6个人，只要一个人review通过就可以merge，像我这种刚来一个多月的整个组最烂的仔review通过也可以
【 在 toutouqi 的大作中提到: 】
: 所以，review必须有约束机制，比如不修改不让merge。可以提前约定好，5个人的组必须3个以上同意才可以merge代码，提意见以功能和潜在问题为主，这些意见必须改，coding style意见仅供参考等。
--
FROM 120.21.71.*
问题：如何给同事做code review同时又不会惹人反感？
就像你前面说的，你觉得我说的两个做法都算错，但你楼上认为code review不应该管，也就是说如果是你review他的代码，给了意见说不行，那他一定很不满，心想：”明明不应该管的东西，干嘛非要提，害我没法merge“，这个怎么破？
【 在 PaoloMaldini 的大作中提到: 】
: 看了这个话题下的讨论之后，觉得我对code review的态度还算比较认真的。。
:
--
FROM 120.21.71.*
然而code review也没法被认定是自己的功劳，反而特别容易惹人不快
【 在 here080 的大作中提到: 】
: 谁提意见谁实现，实现了能认定多大功劳呢？
: 因为这个操作不好的话很容易变成能者多劳，不管不劳。
:
--
FROM 120.21.57.*
能理解。改了500行那个太复杂，看不懂，没法review
我现在review那种大的PR，都要下载到本地，实际跑起来debug跟一下，才能稍微摸到里面的逻辑，很费时间
问题就在于，manager往往看不到你花的这些时间，他只会看到你做了几个item
【 在 eGust 的大作中提到: 】
: 很好奇是怎么做到的？
: code review 是 law of triviality（自行车棚效应）最好的体现：reddit 上面也经常有关于这方面的 meme：一个 PR 只改了2行代码，各种改动意见；另外一个改了500行，于是所有人都 LGTM
:
--
FROM 120.21.57.*
看来大家提到过好几次这个id被人弄出来的情况啊
那我就多解释一下：如果那个id存在，那会返回200 ok，也是连接成功了。检查的时候检查的是200 or 404，如果是其他exception，说明连接失败。
【 在 zhbzhang 的大作中提到: 】
: 虽然我不懂，但是感觉你这个法子不靠谱啊，万一这个id以后被人弄出来呢？
--
FROM 120.21.57.*