CodeReview的挑战

《CodeReview的挑战》这篇在博客园热度很高，讲的正是大家天天碰到的事。下面帮你把要点捋出来，结尾有能直接抄的结论。

保证CodeReview质量的前提条件 有良性的社交压力 保证CodeReview质量的先决条件在于建立一个良性、有效的社交压力机制。这种机制始于招聘过程，我们需要吸引那些拥有基础专业素养的开发者，其中包括能够承受并积极响应CodeReview中社交压力的能力。 设想一下，当你紧张地编写代码，而交付时间又步步逼近时，面对组织对单元测试的严格要求，你可能会考虑降低标准，甚至为了应付工具的覆盖率要求而编写一些不那么有用但能提高覆盖率的测试。然而，一想到你的代码即将接受同事们的严格审查，那种压力便会促使你避免选择那些短期收益高但长期损失大的“投机”行为。

保证CodeReview质量的前提条件 有良性的社交压力 保证CodeReview质量的先决条件在于建立一个良性、有效的社交压力机制。这种机制始于招聘过程，我们需要吸引那些拥有基础专业素养的开发者，其中包括能够承受并积极响应CodeReview中社交压力的能力。 设想一下，当你紧张地编写代码，而交付时间又步步逼近时，面对组织对单元测试的严格要求，你可能会考虑降低标准，甚至为了应付工具的覆盖率要求而编写一些不那么有用但能提高覆盖率的测试。然而，一想到你的代码即将接受同事们的严格审查，那种压力便会促使你避免选择那些短期收益高但长期损失大的“投机”行为。 CodeReview是对前面所有工作的检查 下面是我总结的CodeReview的关注点： 基础功能方面 修改范围：我在CR时第一关注点是修改范围与需求是否一致，如果修改范围扩大，会造成变更不可控，是极为危险的。

保证CodeReview质量的前提条件 有良性的社交压力 保证CodeReview质量的先决条件在于建立一个良性、有效的社交压力机制。这种机制始于招聘过程，我们需要吸引那些拥有基础专业素养的开发者，其中包括能够承受并积极响应CodeReview中社交压力的能力。 设想一下，当你紧张地编写代码，而交付时间又步步逼近时，面对组织对单元测试的严格要求，你可能会考虑降低标准，甚至为了应付工具的覆盖率要求而编写一些不那么有用但能提高覆盖率的测试。然而，一想到你的代码即将接受同事们的严格审查，那种压力便会促使你避免选择那些短期收益高但长期损失大的“投机”行为。 CodeReview是对前面所有工作的检查 下面是我总结的CodeReview的关注点： 基础功能方面 修改范围：我在CR时第一关注点是修改范围与需求是否一致，如果修改范围扩大，会造成变更不可控，是极为危险的。 安全性：是否有严格的准入校验，检查是否有可能导致安全问题的地方，如SQL注入、跨站脚本（XSS）问题等。

保证CodeReview质量的前提条件 有良性的社交压力 保证CodeReview质量的先决条件在于建立一个良性、有效的社交压力机制。这种机制始于招聘过程，我们需要吸引那些拥有基础专业素养的开发者，其中包括能够承受并积极响应CodeReview中社交压力的能力。 设想一下，当你紧张地编写代码，而交付时间又步步逼近时，面对组织对单元测试的严格要求，你可能会考虑降低标准，甚至为了应付工具的覆盖率要求而编写一些不那么有用但能提高覆盖率的测试。然而，一想到你的代码即将接受同事们的严格审查，那种压力便会促使你避免选择那些短期收益高但长期损失大的“投机”行为。 CodeReview是对前面所有工作的检查 下面是我总结的CodeReview的关注点： 基础功能方面 修改范围：我在CR时第一关注点是修改范围与需求是否一致，如果修改范围扩大，会造成变更不可控，是极为危险的。 安全性：是否有严格的准入校验，检查是否有可能导致安全问题的地方，如SQL注入、跨站脚本（XSS）问题等。对于敏感数据，要确保其在使用、传输和存储过程中的安全性。

保证CodeReview质量的前提条件 有良性的社交压力 保证CodeReview质量的先决条件在于建立一个良性、有效的社交压力机制。这种机制始于招聘过程，我们需要吸引那些拥有基础专业素养的开发者，其中包括能够承受并积极响应CodeReview中社交压力的能力。 设想一下，当你紧张地编写代码，而交付时间又步步逼近时，面对组织对单元测试的严格要求，你可能会考虑降低标准，甚至为了应付工具的覆盖率要求而编写一些不那么有用但能提高覆盖率的测试。然而，一想到你的代码即将接受同事们的严格审查，那种压力便会促使你避免选择那些短期收益高但长期损失大的“投机”行为。 CodeReview是对前面所有工作的检查 下面是我总结的CodeReview的关注点： 基础功能方面 修改范围：我在CR时第一关注点是修改范围与需求是否一致，如果修改范围扩大，会造成变更不可控，是极为危险的。 安全性：是否有严格的准入校验，检查是否有可能导致安全问题的地方，如SQL注入、跨站脚本（XSS）问题等。对于敏感数据，要确保其在使用、传输和存储过程中的安全性。 功能实现：代码是否满足功能需求或修复了相应的Bug，代码实现的逻辑是否正确？

行动建议：收藏这篇，下次碰到同类问题先翻出来对照做一遍。好经验的价值，在于用起来。你最近被这类问题卡过吗？ 来源｜博客园《CodeReview的挑战》，https://www.cnblogs.com/xiexj/p/22961980.html