git 协同开发代码审查_现代开发人员，第4部分：代码审查和质量保证

git 协同开发代码审查

作为软件开发人员，您的工作不仅是编写代码。它旨在为复杂的问题提供高质量的解决方案。

您编写的代码是否高质量，可以使软件完成应有的工作当软件遇到错误时，是否可以正常处理该软件是否安全且性能足够

有两个过程可用来衡量和提高软件项目的质量：代码审查和质量保证。让我们检查它们中的每一个，看看如何在工作流中实现它们以提供更好的结果。

更常见的代码审查方法是由团队中的某人执行。流程本身取决于公司，并且可能会有所不同，但是重要的是必须提供反馈。

关于审稿人，他们必须提供建设性反馈。审阅者应该知道代码应该做什么。走廊代码检查（当您从随机同事那里获得关于一段代码的反馈时）是一个例外，但这很少使用。

当涉及到您时，受审者总是要善意。您和审阅者都可能希望提供最高质量的代码。不要亲自批评您的代码-您不是您的代码，也不是您的工作。

不要忘记，代码的目的是解决问题。如果您不同意审稿人，请与他们联系并进行简短讨论。我建议您跳过电子邮件，除非这是公司政策，否则在解决这类误解时，口头交流会更好。

代码审查软件

有一些工具可以自动进行代码审查。尽管它们可以捕获一些错误并遵循诸如“函数名称不能长于XY符 ”之类的规则，但是它们并没有那么有用……。

对我来说，它们是扩展的代码和样式短毛绒。即使其中一些可以指出圈复杂性问题，也无法捕获业务错误。可以随意使用它们，但是它们不能代替自我代码审查或团队代码审查。

不要跳过代码评论

尽管时间问题和尽快发布的压力可能会诱使您跳过代码审查，但这始终不是一个好主意。

软件是团队的共同努力，作为一个团队，您有责任提供最高质量的软件。代码审查是提高工作质量和验证所发布功能是否成功的最简单方法之一。

请记住，即使是10分钟的自我代码检查也可能最终捕获到将被带入生产环境的关键业务错误。不要妥协质量。做代码审查。

质量保证

质量保证（QA）经历了许多趋势。业界正朝着自动化测试迈进，但有趣的是，仍然有很多职位可供手动测试人员使用。

让我们分解一些最受欢迎的质量检查方法，看看每种方法如何提高软件的质量。

开发人员完成的质量保证

这是开发人员（通常是团队中的某人）作为最终用户测试功能是否正确的时候。这是最古老的方法之一，部分原因是管理人员不想雇用手动质量检查人员。最终结果通常很差。

大多数情况下，开发人员已经有了足够的想法，不会将测试的细节与开发任务同等地对待。从理论上讲，这不应该发生，因为它不专业，但是实际上这就是发生的情况。

如今，开发人员进行测试的唯一合法方法是在开发人员内部进行测试，并且通常由团队负责人进行测试，然后再进行功能以进行真正的质量检查。

手动质量检查

尽管很想跳过单元测试和集成测试，但从长远来看，大多数情况下，跳过它们会降低您的速度。不要试图偷工减料-做正确的事情，这样，在项目发布后，您将获得非常轻便的维护，而不会出现使您烦恼的错误，并将生产率降低到下一两个季度。

另一方面，非功能测试是检查系统应如何执行某些任务的测试：

性能测试 –检查系统在正常负载下的运行速度。
压力测试 –检查有多个并发用户时系统的行为。
恢复测试 –揭示系统是否可以从崩溃中恢复。通常，进程开始运行时不会出现很多问题，但是存储（文件系统和数据库）将进入恢复模式。有时，使系统正常运行的唯一方法是使用备份。

在大多数组织中，功能测试往往优先于非功能测试。这是非常不幸的，因为每种类型的测试都是为了捕获特定的错误，并且项目通常迟早会遇到各种错误。

即使分配给质量检查的资源不足，也应在系统投入生产之前执行所有主要类型的测试。

不要忘记功能测试

功能不能有错误，不能快速运行，并且对用户友好，但仍然不是很好的功能。更重要的是诸如以下问题：它是否执行了应该执行的任务对系统有用吗它会增加价值吗

通常，如果功能与目标用户进行了几次反馈循环，则功能所带来的价值将不如所能提供的那么多。通过代码审查或出色的QA工程师无法解决此问题。这是公司文化层面上存在的东西。

软件质量是一个旅程，而不是目标

只要软件发生变化，保持质量不变就是一场持续的战斗。它并不总是那么简单，也不总是那么容易，但这是使项目保持活力并减少技术债务的必要条件。

进行自我代码审查是交付更高质量软件的最便宜，最有效的方法之一。同样，使用自动棉绒和样式检查器将帮助您专注于业务逻辑。

但是，甚至除此之外，让您的同事参与将有助于您提供更好的体系结构代码，并且进行手动和自动的质量保证将减少导致生产的错误数量。

作为软件开发人员，请始终提醒自己，保持高质量的代码与添加新功能一样重要。

翻译自: https://www.javacodegeeks.com/2019/10/the-modern-developer-code-review-and-quality-assurance.html