代码审查是开发人员编写的代码由另一个人检查以查找缺陷和改进的过程。换句话说，开发人员大部分都是独立编写代码的，当代码完成之后，他们会召集1次评审。下面，和郑州华软教育一起探讨下可以做哪些事情，可以让代码审查为所有参与者带来愉快的体验。

  1对管理层的建议：营造健康文化

  有效的代码审查，需要一个重视质量和卓越的健康文化。如果团队不以提供高质量的产品为信仰，代码审查将不会给您所期望的结果。你需要一个人人参与的积极文化—立足于建设性批评，智者胜。

  除了创造一个健康文化，并允许花时间和资源进行评审，管理者在代码审查中应保持低姿态。大多数人不想在上司面前暴露自己的秘密，这已是一种文化。代码审查由同行进行，管理层不应该询问可以用来评价人的细节。的确有一些管理人员索要检查表和成绩，以便他们可以“衡量”并评价人。

  可能你已经有一个健康的文化（算你幸运）。这还不够，营造健康的文化取决于许多因素（团队和组织内部）。这是非常具有挑战性的，没有灵丹妙药。没有正确的文化，代码审查不会带来期望的收获，甚至在极端情况下可能会适得其反。

  2对个人的建议：换位思考

  KarlWiegers在他的《PeerReviewsinSoftware:APracticalGuide》中写道：

  产品的作者与评审者之间的互动至关重要。作者必须足够信任和尊重评审者才能接受他们的意见。同样，评审者必须尊重作者的才华和辛勤。评审者应谨慎选择他们用来提出问题的词汇，重点关注他们对产品的观察。说『我没有看到这些变量被初始化』可能会引发建设性的反馈，而『你没有初始化这些变量』可能会让作者非常不爽。

  关注代码很容易，但不要忘记，桌子（或计算机）的另一端有一个人。他有主见有“自我”。请记住，解决问题的方式有很多。

  要谦虚。我既见过非常高效的评审，也见过因为吹毛求疵而非常低效的评审。不要吹毛求疵！！！

  确保您有编码标准。编码标准是在组织中共享的人人都认同的一套准则。如果你没有编码标准，那么不要让讨论变成一个比较编码风格的比赛（大括号{在同一行还是下一行！）如果你遇到这样的情况，请在编码标准论坛上离线讨论。

  良好地沟通。你必须能够清楚地表达想法和理由。

  编程策略是一个仁者见仁智者见智的问题。评审者和开发人员应该寻求理解彼此的观点，而不应该成为哲学辩论

  3对评审者的建议：谦虚

  开发者不是冤大头。评审的目的不是证明谁是更出色的程序员而是查找缺陷，并确保代码是简单和可维护的。

  问问题。不要提出可能听起来带指责的要求或言论。例如，不要说：『你没有遵循标准XYZ』。更好的方式是真正寻求理解开发者的观点：『你对标准XYZ有什么看法，它是否适用于这里？』，这可以引到我的下一个观点。

  避免『你为什么』，『你为什么不』风格的问题。它会使人对立。『为什么把它声明为全局变量？』可以更好地表达为『我不明白为什么这里用一个全局变量』。寻找方法来简化代码。代码审查的目标之一是创建“可维护”软件。

  记住要欣赏并感谢对方。人们经常忘记一句简单的『干得好』或『它看起来很棒』的影响有多大。

  有些事情，如果看起来像排练过的或以讽刺的语气说出来，就不会奏效。像正常对话一样对待代码审查。你正在聆听他人，应该真正寻求理解他们的观点。需要时提供建议和提示。如果代码很棒，不要强迫找一些消极的来说。

  4对开发者的建议：它不是个人的事情

  不要感情用事。记住，别人说的不是，是代码中的缺陷或不足，而不是你。

  意识到你和你的代码是捆绑在一起是正常的事情。如果你为自己的感到自豪，那是一个很好的迹象，说明你是一个关心作品的人。

  有适当的自我。足够信任和捍卫自己的观点，但又不至于盲目拒绝对方的好建议和意见。

  人非圣贤孰能无过。评审者作为第二双眼，可以指出你可能忽略的事情。问题与具体建议一样有价值。

  提问题要有针对性。『将所有这些类纳入它们自己的软件包中是否更有意义』

  感谢审稿人的时间和他们可能提供的任何反馈。

尊重原创文章，转载请注明出处与链接：http://huaruan.5zix.com/news/132307/ 违者必究！ 以上就是郑州华软教育 小编为您整理怎样让代码审查更有效的全部内容。

---

• ← Java在互联网中起到什么作用
• 如何成为一名合格的程序员 →