问如何在代码评审中找到积极的东西？

EN

在去年出现了一些严重的质量问题之后，我的公司最近推出了代码评审。代码评审过程被迅速引入，没有指导方针或任何类型的检查表。

另一位开发人员和我选择在系统被合并到主干之前检查对系统所做的所有更改。

我们还被选为“技术领导”。这意味着我们对代码质量负责，但我们没有任何权力来实现流程中的更改、重新分配开发人员或阻止项目。

从技术上讲，我们可以拒绝合并，让它回到开发阶段。在现实中，这几乎总是随着我们的老板要求准时发货而结束。

我们的经理是MBA，他最关心的是为即将到来的项目制定时间表。当他尝试的时候，他几乎不知道我们的软件从业务的角度来做什么，甚至在没有开发人员解释的情况下，他也很难理解最基本的客户需求。

目前，开发是在SVN的开发分支机构进行的，当开发人员认为他已经准备好之后，他会将票务系统中的票重新分配给我们的经理。然后经理把它分配给我们。

代码评审导致了我们团队内部的一些紧张关系。尤其是一些年长的成员对这些变化提出质疑。“我们总是这样做”或者“为什么这个方法应该有一个合理的名字，我知道它是干什么的？”)。

在最初的几周后，我的同事开始听之任之，不给同事制造麻烦(她自己告诉我，在一位客户提交了一份bug报告后，她知道了这个bug，但担心开发人员会因为她指出这个错误而生气)。

另一方面，我现在以指出所提交代码的问题而出名。

我不认为我的标准太高。

我目前的清单是：

• 代码会编译。
• 代码至少有一种工作方式。
• 该代码将适用于大多数正常情况。
• 该代码将适用于大多数边缘情况。
• 如果插入的数据无效，代码将抛出合理的异常。

但我完全接受我给予反馈的方式的责任。我已经给出了一些可操作的要点，解释了为什么某些东西应该改变，有时甚至只是问为什么某些东西是以一种特定的方式实现的。当我觉得不好的时候，我会指出我会用另一种方式来发展它。

我所缺乏的是找到“好”的东西的能力。我读到一个人应该试着把坏消息当成好消息。

但我很难找到好东西。“嘿，这一次你真的做了你所做的每一件事”更多的是谦卑，而不是善意或帮助。

示例代码评审

嘿，Joe，关于您在Library\ACME\ExtractOrderMail类中的更改，我有一些问题。我不明白你为什么把"TempFilesToDelete“标记为静态的？目前，对"GetMails“的第二个调用将引发异常，因为在删除它们之后，可以向其添加文件，但从不删除它们。我知道这个函数每次运行只调用一次，但在将来可能会发生变化。你能不能让它成为一个实例变量，然后我们就可以并行地拥有多个对象。..。(其他一些不起作用的要点)小问题：

• 为什么"GetErrorMailBody“会以异常作为参数？我错过了什么吗？您不是抛出异常，而是传递它并调用"ToString“。为什么会这样呢？
• SaveAndSend不是这个方法的好名字。如果邮件处理出错，此方法将发送错误邮件。你能把它重命名为"SendErrorMail“或类似的东西吗？
• 请不要仅仅评论旧代码，直接删除它。我们还在颠覆中。