谁应该做代码评审？

Question

在我的公司里，架构师主要负责代码评审。他是一个非常有经验和聪明的软件人，所以他非常擅长它。当开发人员进行代码评审时，他们也不会进行一半的代码评审。我们试图让开发人员进行更多的代码评审，但是代码评审的质量并不好。我们使用Scrum作为开发方法。

然而，就现行制度而言，有两个问题：

1. 建筑师成了瓶颈
2. 开发人员不对代码和体系结构的质量负责(这会导致各种问题)。

我们如何处理这些问题？我们应该改变谁进行代码评审吗？

Answer 1

开发人员应该进行代码评审。他们应该进行代码评审，因为他们应该了解代码、公司风格的标准和实践。通过让其他人进行代码评审，您正在告诉您的开发人员，确保代码符合公司标准并不是他们的责任。

如果你认为他们需要进行代码评审方面的培训，那就给他们看看。考虑到您当前的情况，您可以让开发人员对代码进行评审，然后让开发人员在将其发送给提交者之前将评审提交给架构师以供批准。

Answer 2

在这种情况下，您需要的是这位经验丰富的开发人员的知识，以帮助团队的其他成员成长。团队的质量不是由最好的开发人员的技能决定的，而是由最糟糕的技能决定的。你可以尝试这样的方法：

• 合作审查。这在我的上一支队伍里真的很成功。我们把整个团队都放在一个有投影仪的房间里，然后开始复习一些东西。也许在一开始，架构师就是指导评审的人，但几周后(我们每周五预留一到两个小时)，整个团队开始谈论和理解目前只有架构师才知道的关键概念。
• 对编程。对我来说，这是在团队中传播知识的最好工具。

Answer 3

虽然我明白让系统/软件架构师签署所有更改/提交的重要性，但是软件开发人员应该能够在不涉及架构师的情况下进行评审--除了仲裁之外。

我喜欢的*评审过程是：

• 按需求/问题分组更改。
• 邀请所有开发人员、软件架构师和需求/问题的作者进行评审。(他们并不都需要做一次审查。)
• 当：时，考虑完成一次评审
  • 两位开发人员对此进行了回顾。
  • 所有的评论都会得到回应。(可能是让软件架构师做出决定。)
  • 一个工作日过去了，但未作进一步讨论(或所有受邀缔约方都进行了审查)。

因此，我对您的问题的简短回答是:开发人员应该进行更改评审。

*不幸的是，我参与的项目并不总是这样运作的。