代码评审

大多数KPI指标毫无价值。绝对最佳的程序员所编写的代码少于能力较弱的程序员。最好的衡量标准是编写的代码少，代码越少越好。实际上，删除代码是您可以执行的最有效的操作之一。负生产力反而是一个加号。测量代码行会惩罚您最好的程序员。代码质量（不是错误，而是实际质量）无法衡量（也许您可

在 Tableau，Tableau Mobile团队约有 30 人，分布在 3 个 Scrum 团队中。我们主要在 Tableau Mobil

软件工程师的工作不是编写代码，而是解决问题；我们可通过生成代码解决了大部分问题。但是最终，生成代码也很困难，我们需要帮助。这就是为什么 GitHub 的Copilot

我听过人们无数次吹嘘其他人要了解他们的工作需要多长多长时间，依我看情况正好相反。你应该吹嘘人们能够多快地理解你所写的内容。 当我看到那些能够以让别人可以简单理解的方式创造表达复杂事物的人时，我会对他留

编写可维护的代码很容易。只需保持方法和参数列表简短，名称和注释较长，并遵循样式指南。正如一位著名记者曾经写道：“对于每一个复杂的问题，都有一个清晰、简单和错误的答案。”使代码难以维护的不是样式和形状。这是在缺乏明确的如何代码工作，它代表什么以及为什么它以这种方式被写？

在过去的4-5年中，对程序员的需求增长如此之快，以至于程序员的数量总是每五年翻一番。结果，拥有5年经验的程序员所拥有的行业任职时间比整个行业的一半还多。我现在在这个行业中已经推进了20年。我花了大约10个角色担任我的主要职能是编写代码。其他10个项目涉及管理程序员，指导他们，就如何管

软件系统是容易的积聚一些累赘cruft  : 内部质量不高，导致其比预想更难进行修改和进一步扩展系统。技术债务是沃德坎宁安（Ward Cunningham）创造的一个比喻，它描述了如何考虑处理这种问题，将其视为金融债务，增加新功能所需的额外成本是债务利息。<

作为开发人员，我们都知道代码审查在理论上是一件好事。他们应该帮助我们： 尽早发现错误和安全问题 提高代码的可读性 提供安全网以确保所有任务完全完成 现实情况是，代码审查对于每个参与者来说经常是一种令人不舒服的体验，导致审查变得好斗，无效甚至更糟，根本

在过去的三年中，我已经审查了1000多个拉（合并）请求。在那段时间里，我学到了很多东西–主要是关于如何不审阅代码，如何减轻过程的痛苦，使高质量的代码产生什么等等。 拉取请求只需要做一件事最好的办法是将请求合

Google的代码评审在工程实践中发挥着重要作用，并且早在谷歌就已经采用。直到今天，它们仍然用于保持代码库的清洁，连贯并确保不提交任意代码。尽管代码评审过程与

基于长期经验，本节中的页面包含有关进行代码评审的最佳方式的建议。它们共同代表了一个完整的文档，分为许多单独的部分。你不必全部阅读它们，但很多人发现它对自己和他们的团队阅读整套都很有帮助。

本文指出了所有开发人员在审查其代码或提交拉取请求时可能遇到的特定反模式，并对此进行了谴责。代码作者花了数小时甚至数天的时间来创建他们认为最有效的解决方案。他们考虑了多种设计方案，并采取了最相关的道路。他们考虑了现有应用程序架构，并在适当的位置进行了更改。然后他们将其解决方案作为请求请

尽管编程语言已经发生了巨大的发展，但它们的核心仍​​然有一个主要的共同点：让计算机以最有效和最无错误的方式实现目标。现代语言在许多方面使开发变得更加容易，但是在我们实际检查各个代码行以使它们无错误的方式方面，并没有太大改变。在提高代码质量，提高性能和降低运营成本方面所做的工作甚至更少。

代码质量是软件工程最重要的方面之一。SonaQube 是代码保证工具，它通过收集您的源代码并对其进行分析来确保项目的代码质量。您可以根据此工具的结果将 CI/CD 管道配置为部署或不部署。例如，如果单元测试覆盖率低于 85%，则构建管道可能会失败。在这篇文章中，我们将看到如何在本地机

质量软件取决于许多因素，但开发人员是最重要的因素之一。糟糕的软件往往是我们的错，通过问自己正确的问题，我们也可以为好的软件做出贡献。 这是对的吗？ - 代码是否实现了规范。如果没有明确的规范，你是否做了足够的努力来找出预期的行为。并且这种行为是以某种方式测试的 - 最好通过自