启动代码审查过程，而以前没有吗？

Question

对于需要开始代码评审的组，有什么建议吗？我们目前没有真正的软件工程流程。我已经做了7年的软件工程师，从来没有见过在我工作过的不同地方实施过任何真正的过程，尽管很多书都支持各种工程方法。管理部门支持启用新过程的想法，尽管具有20年以上软件工程经验的高级工程师对此存在一定的阻力。

我们是一组两名程序员，没有软件工程经理。我们直接向公司总经理汇报。但我们在许多软件项目上工作。我们必须开始执行代码评审的真正原因是因为PA的需求。因此，我推动我们做的代码审查四面八方-不只是在支付应用软件。

Answer 1

因为只有你们两个人，理论上你并不需要一个正式的审查过程。应该发生的是，当你们中的一个完成一段代码时，他应该简单地对另一个说：“这是我所做的，请看一下。”

当然，这是假设你们俩都是理性的，有足够的纪律，并且你们都对自己的自我有足够的掌控能力。如果这些假设是错误的，那么您确实需要从顶部强制执行一个正式的过程。最简单的做法是制定一条规则，所有提交或合并到主干都必须由其他人或至少一个其他人查看和批准，以防您的团队增长。

这一点还不适用于您，但根据我的经验，即使是一个由4人组成的团队进行的正式代码评审最终也会花费太多的时间，个别评审人员的评论90%是相同的。这就是为什么我们换了一个审查系统。

Answer 2

您希望代码评审是一个积极的练习！以下是一些建议：

• 代码审查，早对编程！即使这一天只有30分钟。
• 使用静态代码分析工具、样式检查器和模板来消除模糊。然后，您可以专注于更重要的问题(SRP、设计模式、异常处理、接口编码等)，这正是代码评审的真正意义(IMHO)。
• 在你们之间得到一份关于什么是重要的和什么不重要的商定清单
• 建议采用某种测试驱动的方法(更好的前期设计)。
• 有一些非常好的代码审查工具-我个人使用坩埚。
• 确保‘改进代码’的态度，而不是‘简化编码器’。这对于保持每个人的自我快乐都是至关重要的。
• 要有规律地做，不要吝啬。无论是在一组承诺或1/天之后，还是在你同意的任何事情之后。

祝好运!

Answer 3

启动代码评审的最简单方法:只需使用一个开源审查工具就可以了。http://ostatic.com/blog/open-source-code-review-tools您可以从简单的过程开始:每天都有一个开发人员检查第二个开发人员的代码，反之亦然。如果复习可以帮助您改进代码，那么您可能会想到“真正的技术过程”。