Skip to content

Code review

仲灏2022-06-21约 1 分钟

<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 的问题要记录下
  • 归纳整理,形成自己的代码规范体系
  • 新加入的成员要提前学习,提前规避