先看正确性，边界条件、异常分支、并发下的竞态，这几处最容易漏而且代价最大。
再看安全和数据，参数校验、权限判断、日志里有没有打印敏感字段。
最后看可维护性，命名和分层是否一致、有没有可测性，别在评审里纠结空格和换行。
# 评审清单
# 1 正确性：边界、异常、并发
# 2 安全：参数校验、鉴权、敏感信息
# 3 可维护：命名、分层、单测覆盖THE END
相关推荐
先看正确性，边界条件、异常分支、并发下的竞态，这几处最容易漏而且代价最大。
再看安全和数据，参数校验、权限判断、日志里有没有打印敏感字段。
最后看可维护性，命名和分层是否一致、有没有可测性，别在评审里纠结空格和换行。
# 评审清单
# 1 正确性：边界、异常、并发
# 2 安全：参数校验、鉴权、敏感信息
# 3 可维护：命名、分层、单测覆盖
暂无评论内容