关于你说的“code review是个挑错性质的东西，不怎么受人待见，而且容易起争执”，这个我也想知道大家是怎么解决的。如果代码真的错了还好说，那种改善型的意见，真不知道是该给还是不该给。

举一个例子，我同事写了一个client，需要实现isAvailable()，就是去连一下对端服务器，连上了就返回true。

他的方法是调用一个取groups的API，这个API会返回属于某个ID的一大堆groups，但问题是这堆groups根本用不上。

我说你随便传一个不存在的ID就行，因为不存在人家会返回404，也是说明连上了。但这哥们就是不改，估计他心里是一大堆草泥马，想着反正这个代码能用，干嘛非要改。

【 在 z16166 的大作中提到: 】
: 动力主要来自KPI。。。KPI要求流程里必须有这步，发布出去出了问题可能要担连带责任（主要责任还是写这个代码的）
: code review是个挑错性质的东西，不怎么受人待见，而且容易起争执
: 极少数情况下是为了学习别人的好代码，那种已经不算review了，算learning。
--
FROM 120.21.44.*