说实话：代码审查并不是开发人员待办事项列表中最令人兴奋的任务。

问题是，良好的代码审查需要哪些要素？而且，我们怎样才能让它少一点呢？

首先，让我们澄清一下什么是代码审查。这不仅仅是挑剔别人的代码或指出每个缺少的分号。这是一个学习和成长、提高代码质量和建立团队团结的机会。这就像一个读书俱乐部，但是是为了代码。我们来这里是为了欣赏这个故事，讨论情节的曲折，也许，只是也许，让它变得更好一点。

良好的代码审查是一门艺术。审阅者就像一位雕塑家，削掉原始代码块以揭示下面的杰作。但这也是一门科学。它需要有条不紊的方法、对代码用途的清晰理解以及对细节的关注。

“优秀的代码审查员不仅仅会发现错误；他们还会促进学习、维护代码质量并促进团队团结。”

那么，我们如何才能实现艺术与科学的微妙平衡呢？

学习：代码审查是互相学习的机会。乐于接受反馈并准备分享您的知识。
质量：保持代码质量应该是重中之重。寻找错误、风格不一致和潜在的改进。
Unity：代码审查是一项团队运动。支持、有建设性并尊重你的队友。
如何不进行代码审查

1. 吹毛求疵的人
   代码审查并不是玩“找出错字”或“找到丢失的分号”游戏。是的，干净的代码很重要，但请记住，我们是程序员，而不是语法老师。这是关于代码的功能和可维护性，而不是轻微的样式不一致。

“代码审查不是为了挑剔；而是为了培养质量和协作。”

Linting 应该能够在没有 The Nitpicker 的情况下在本地完成。

2. 小说家
   曾经收到过比代码本身还长的评论吗？你不是一个人。代码审查应该简明扼要。这不是要展示你的词汇能力或在评论中发表“战争与和平”。

公关上的公关？没有。乐于助人，不要为他们做事。

3、沉默型
代码审查是一条双向路。如果您要进行评论，请务必提供建设性的反馈。如果你是接收端呢？不要用沉默来回应。提出问题、提供解释、参与！

LGTM 没有帮助。在 PR 问题日志上拖拖拉拉也是不合适的。

4. 独裁者
   代码审查不是权力游戏。这不是要强制执行您的个人风格或偏好，而是要保持代码质量和一致性。请记住，代码审查是民主，而不是独裁。

linting 和质量指标可以标准化并在管道中应用是有原因的。是什么让代码既时尚又优质？它被定义了。想改变它吗？提交 PR。独裁者会适得其反。

5.“总是负面”的审稿人
虽然强调问题很重要，但不要忘记承认做得正确的事情。好的评审不仅仅是指出错误，而是促进学习和促进团队团结。这是一种平衡。

让我们将其提取到接口并注入它。走向依赖倒置。

… 与 … 搭配得很好

利斯科夫运用得很好！

为什么代码审查很重要
在深入研究细微差别之前，让我们先了解一下基础知识。代码审查是软件开发过程的关键组成部分。它们有多种用途：

错误识别：第二双眼睛通常会发现第一双眼睛错过的东西。
知识共享：评论传播有关代码库更改的知识，确保没有一个人成为信息孤岛。
代码一致性：他们确保代码符合团队的编码标准和实践。伟大的代码看起来就像是一个人写的。
团队凝聚力：不管你信不信，评论可以成为一种凝聚力的体验，通过协作和相互学习使开发人员更加紧密。
科学：评论中的技术掌握
代码审查不仅仅是发现错误。它们是为了确保代码具有最高质量。以下是一些需要记住的技术方面：

理解上下文：确保您理解代码试图解决的问题。有时，问题不在于代码本身，而在于其与需求的一致性。
检查性能问题：检查是否存在可能影响性能的任何潜在瓶颈或低效循环。
安全审查：密切关注潜在的安全漏洞，例如暴露的敏感数据或未经检查的用户输入。
代码模块化和整洁：代码不应该只是工作；它应该是干净的和模块化的，以便将来的可维护性。
艺术：培养协作评论文化
除了冰冷的代码逻辑之外，代码审查还有更微妙、更巧妙的一面。这涉及人的因素，对于成功的审核过程至关重要。

反馈框架：始终建设性地构建反馈。不要说“这行不通”，而要考虑“这种方法可以进一步优化吗？” 那个苏格拉底家伙有话要说。
鼓励提问：促进一种欢迎和鼓励在评审期间提出问题的文化。
认可和欣赏：认可好的代码和创新的解决方案。一点积极的强化会有很大帮助。
平衡批评与同理心：了解每段代码背后都付出了努力。尊重您的反馈，强调改进而不是批评。
工具和最佳实践
现代开发人员不仅拥有知识，还拥有一套帮助代码审查的工具。

版本控制平台：GitHub 和 Bitbucket 等工具提供集成的代码审查工具，使评论、讨论和跟踪更改变得更加容易。
自动化测试：在深入审查之前，自动化测试可以帮助识别明显的问题。Jenkins 或 Travis CI 等工具可以在这方面提供帮助。
Linters 和 Formatters：像 ESLint 或 Prettier 这样的工具可以自动捕获并修复样式不一致的地方，从而减少审查中的手动开销。
自动化质量和安全分析：像 SonarQube 这样的工具可以突出显示圈和认知复杂性的领域。除此之外，它们还可以配置为使管道失败，从而使反馈非个性化，并确保所有团队成员的贡献都符合相同的标准。
代码审查交响乐
代码审查是技术精度和人类协作的交响乐。如果做得正确，它们不仅仅是开发过程中的一个步骤，而且是一种丰富的经验，可以提高整个团队的技能和凝聚力。

要真正掌握代码审查，我们必须在审查代码的科学性和理解人类协作的艺术之间找到微妙的界限。