- 主题:各位是从哪里获取code review的动力的？
这坨傻逼代码早晚还得我来擦屁股
每次这么一想就有动力了
【 在 dpblue (deep blue) 的大作中提到: 】
: 我们组里谁都可以review代码，头儿貌似也不太关心你是否review同事们的代码，所以我平时都只顾着自己手上的活，懒得去review。但感觉这样不太好，请问各位是从哪里获取code review的动力的？
--
FROM 111.193.175.3
你们俩的实现都很奇葩。。且不说这玩意儿有用没用
即便要做，也应该是让服务端直接写个返回200的空接口专供测试用吧
哪能用业务接口测试去
【 在 dpblue (deep blue) 的大作中提到: 】
: 关于你说的“code review是个挑错性质的东西，不怎么受人待见，而且容易起争执”，这个我也想知道大家是怎么解决的。如果代码真的错了还好说，那种改善型的意见，真不知道是该给还是不该给。
: 举一个例子，我同事写了一个client，需要实现isAvailable()，就是去连一下对端服务器，连上了就返回true。
: 他的方法是调用一个取groups的API，这个API会返回属于某个ID的一大堆groups，但问题是这堆groups根本用不上。
: ...................
--
FROM 111.193.175.3
anyway你们的做法。。如果让我review，两个都算错
【 在 dpblue (deep blue) 的大作中提到: 】
: 这个微服务倒是有返回200的接口，叫deepcheck，表示自己可用并且自己可以访问到依赖的所有资源
: 不过我们只用到了这个微服务的groups api，所以就没有用deepcheck
--
FROM 111.193.175.3
看了这个话题下的讨论之后，觉得我对code review的态度还算比较认真的。。
【 在 dpblue (deep blue) 的大作中提到: 】
: 我们组里谁都可以review代码，头儿貌似也不太关心你是否review同事们的代码，所以我平时都只顾着自己手上的活，懒得去review。但感觉这样不太好，请问各位是从哪里获取code review的动力的？
--
FROM 103.107.216.225
我至少保证自己team的code review水平让自己满意
要是旁边人水平都比我高那当然更好
【 在 dpblue (deep blue) 的大作中提到: 】
: 问题：如何给同事做code review同时又不会惹人反感？
: 就像你前面说的，你觉得我说的两个做法都算错，但你楼上认为code review不应该管，也就是说如果是你review他的代码，给了意见说不行，那他一定很不满，心想：”明明不应该管的东西，干嘛非要提，害我没法merge“，这个怎么破？
--
FROM 103.107.216.225