代码评审CheckList
代码评审CheckList
Author: histonevon@zohomail.com
Date: 2023/10/24
- 此博客为笔者在工作中总结的经验，适用于笔者所在的工作，具体情况还需各位自己分析
- 以下的分类并不规范，有好的建议可以给我Email
- 值此1024祝全世界的开发者：天天无Bug，年年有加薪
- 原文地址：https://histonevon.top/archives/code-review-checklist
代码风格
- 大量使用另一开发者读的懂的注释
- 名字要望文生义
- 名字不要太长，使用公司或项目组规定或普遍认同的缩写
- 拒绝魔法数，擅用枚举
- 枚举的使用是为了便于代码阅读，绝不是简化处理可能出现的问题（比如使用枚举的错误类型，要在日志中写入可识别可定位的具体问题，例如流水号、时间、错误描述等）
- 拒绝未使用警告
- 减少代码层级
- 慎用循环
- 在无漏洞的情况下使用封装的轮子，包括可以减少机械化代码的工具（如：Lombok）
- 使用CheckStyle、SonarLint、ESLint等规范待提交的代码
业务处理
- 一个方法仅实现一个功能，且此功能可以从函数名中迅速理解
- 业务处理与非业务处理逻辑要分开，以便复用
- 时刻思考我能为别人实现哪些原子功能，如何拆分这些功能
- 时刻思考如果别人要使用此功能我需要怎么让别人无痛使用
- 时刻考虑在生产环境中如果出错要在日志中写什么才能快速定位问题
- 注意非空判断、超时处理、降级处理
- 注意处理多次操作下的情况（多次重复点击、网络问题、幂等处理）
架构守护
- 架构依赖及命名规范要在项目初期做到人尽皆知
- 使用架构设计最佳实践：如DDD、MVC、MTV、MVVM等
- 使用ArchUnit监控不规范的架构使用
数据处理
- 数据库用什么查什么
- 注意不同DBMS的操作特性
- 能用SQL就用SQL