首页 > 代码库 > 代码审查

代码审查

代码审查:

互相帮助,测试中遇到了问题,领导或是资深开发就会审查。聘请外部专家进行安全代码审查。
系统发布后,基于风险的审查,编写一些风险较高的代码,框架与安全代码、核心业务逻辑或是之前曾经出现过问题的地方。审查体现出了她的价值,收获颇丰。更进一步,让代码审查成为一个标准的实践。

改变人们的工作方式就不是简单的事情,足够的时间进行代码审查,理解并对反馈做出相应。设计高效的代码审查流程需要花时间。
容易通过:商量好,好说话,不知道要花费多少时间才能完成,拖得很久,代码完成测试甚至发布后才完成。
不确定应该看什么,有意义的反馈信息,负面批评
主开发这会对需求有着更好的理解,对代码的行为有着清晰的认识,更有可能发现代码中的错误

该看什么,看到的结果
确保代码能够正常工作,间接好处:如何编写更好的代码,如何在团队内分享知识。
代码行为是否与预期一致,其逻辑是否是正确无误的
被审查代码是否与其他代码拥有类似的结构和功能

话在更加重要的设计错误上,并发Bug、潜在的思索

理解,而不是批评
理解代码,确保代码能够正常工作
终点关注:“我徐璈完全理解这部分代码才能确保她能够正常工作,如果由我来修复代码中的问题,我是不会这么写的,因此希望你也不要这么来写”

代码审查的价值 收益递减