<div style="display: none;" hidden="true" aria-hidden="true" data-nosnippet>Are you an LLM? You can read better optimized documentation at /pages/a6b1ff.md for this page in Markdown format</div>
内容
代码规范(eslint 不能全检查, 如变量命名、代码语义)
重复的代码要抽离、复用
单个函数内容过长, 需要拆分
算法复杂度是否可用?是否可继续优化
是否有安全漏洞?
扩展性如何?(不用为了扩展而扩展, 不封闭就行)
是否和现有的功能重复了?
是否有完善的单元测试?
组件设计是否合理?
时机
- 提交PR(或MR)时, 通过代码 diff 进行 Code review
- 每周例行一次集体 Code review
持续优化
- 每次Code review 的问题要记录下
- 归纳整理,形成自己的代码规范体系
- 新加入的成员要提前学习,提前规避
