看到标题就知道我要被政治正确人士怼了但不吐不快。
组里有个人还不错（看注释）的同事我以前从没合作过。最近开始给他审查代码才发现他能力不行而且非常粗心。
一个小功能需要他反反复复修改多次后才能入目，他经常做的事是 push 了一个修改立马叫我说好了，然后我刚开始看，他立马追加一个 push 继续修复问题。
这两天给他审查一个稍微大点的功能后我发现了很多问题。给他提出一些问题和建议后他昨天又告诉我说好了。
我到现在还没回复他，根本不想看他的代码，想反映给经理但想想大家出来混口饭吃都不容易。
现在我就把他晾在那儿，希望他能审核审核自己的代码，不要把我当人肉 debug 机用。

说他人不错是因为我给他提出问题和建议后能很明显的发现他自尊被伤了，但他基本能忍住不怼我。

举个最近的例子：他写的数据层代码找东西，找到了返回，找不到东西直接抛出异常。我说异常不是这么用的。如果没找到你直接返回 null。谁用你的方法到数据库找东西时需要考虑得到 null 的可能。说白了就是数据层里别加商业逻辑。他听了后看了我 3 秒钟后说：好，改。我觉得我修养还行，基本都是就事论事讨论公事不带感情色彩，从来没说过：这你都不知道啊！这么写太蠢了！这类的话。但在那 3 秒里我随时可能破戒。
举个最近的例子：他写的数据层代码找东西，找到了返回，找不到东西直接抛出异常。我说异常不是这么用的。如果没找到你直接返回 null。谁用你的方法到数据库找东西时需要考虑得到 null 的可能。说白了就是数据层里别加商业逻辑。他听了后看了我 3 秒钟后说：好，改。我觉得我修养还行，基本都是就事论事讨论公事不带感情色彩，从来没说过：这你都不知道啊！这么写太蠢了！这类的话。但在那 3 秒里我随时可能破戒。