Code Review 的目的是发现代码中可能存在的缺陷、漏洞和问题,并提出建设性的意见和建议。在 CodeReview 过程中,通常会关注以下几个方面的问题:
1.代码质量:包括代码的可读性、可维护性、可扩展性、健壮性等方面。检査代码是否符合编码规范、是否存在代码几余、是否使用了不恰当的数据结构或算法等。
2.易于理解: 代码是写给机器执行的,但是也是写给人看的。所以,在CR的时候也需要关注代码是否易于理解、是否清晰易懂、是否有注释等方面,以便其他开发人员可以快速了解代码的实现细节和逻辑。
3.潜在错误: 比如是否有空指针异常、是否有内存泄漏、分支是否完整等问题,以避免出现在生产环境中。
4.代码逻辑:检査代码是否正确实现了预期的业务逻辑,是否存在逻辑错误、边界条件处理不当等问题。
5.安全性:检查代码是否存在安全漏洞,如 SQL 注入、跨站脚本攻击等,是否使用了加密、防护措施等。
6.性能:检査代码是否存在性能问题,如循环嵌套过多、频繁的数据库读写等,是否使用了合适的算法和数据结构以及合理的缓存策略等。
7.注释和文档:检查代码是否有充分的注释和文档,是否符合编码规范、命名规范、文档规范等,以便于其他开发人员能够更好地理解和维护代码。
通过对这些问题的关注,可以帮助开发人员发现并解决代码中存在的问题,提高代码的质量和可维护性,从而保证应用程序的可靠性和稳定性。