首页/开发/代码质量与评审速查/评审检查项:错误处理
代码评审64 条

评审检查项:错误处理

错误处理检查:失败路径是否被显式处理、错误是否被吞(空 catch)、错误信息能否支撑排查、资源在异常路径上是否仍释放。

  • 空 catch 或忽略返回值:错误被静默吞掉是高频问题
  • 错误信息带上下文(参数、ID),能用于定位
  • 资源释放:文件、连接、锁在异常路径是否仍释放
  • 重试与降级的语义:是否幂等、会不会放大故障
见到 catch 后只打日志继续跑,要追问:这里继续执行是否安全?
吞错排查空 catch、忽略返回码、默认值掩盖失败
上下文要求错误信息包含操作、对象 ID 与原始原因

易错点

  • catch 抹平一切异常:故障被掩盖,排障只能靠猜
  • 错误信息只有失败两个字:线上无法定位根因

数据来源与参考

  • Google Engineering Practices · What to look for in a code review(Functionality)
​‌‌​​‌​​​‌‌​‌‌‌‌​‌‌​​​‌‌​‌‌‌​​‌‌​​‌​‌‌‌​​‌‌‌​​​‌​‌‌​‌​​‌​‌‌‌​​​‌​‌‌​‌​​‌​‌‌​​​‌‌​‌‌‌​‌​​​‌‌​‌‌‌‌​​‌​‌‌‌​​‌‌​​​‌‌​‌‌​‌‌‌‌​‌‌​‌‌​‌