首页/开发/代码质量与评审速查/SOLID 自检清单(评审时用)
设计原则与模式62 条

SOLID 自检清单(评审时用)

评审中用五个问题快速扫描:这个类为什么会被改?新增同类能力要改哪里?子类替上去行为还成立吗?有没有人被迫实现不用到的方法?依赖方向指向抽象了吗?

评审五问

评审五问
1. 本 PR 同时回答了几个「变化原因」?
2. 新增一个同类实现,要触碰几处既有代码?
3. 把子类换成父类引用,断言还成立吗?
4. 有无空实现/抛不支持的占位方法?
5. 业务代码里 import 了具体基础设施吗?
清单用于对话与提醒,不用于打分:每条都有合理例外,重点是让权衡显式化。

易错点

  • 把清单当硬门槛:合理例外被强行改写
  • 只在评审时提:设计讨论缺席日常开发,问题积重难返

数据来源与参考

  • Robert C. Martin · SOLID 论述的综合工程化总结
​‌‌​​‌​​​‌‌​‌‌‌‌​‌‌​​​‌‌​‌‌‌​​‌‌​​‌​‌‌‌​​‌‌‌​​​‌​‌‌​‌​​‌​‌‌‌​​​‌​‌‌​‌​​‌​‌‌​​​‌‌​‌‌‌​‌​​​‌‌​‌‌‌‌​​‌​‌‌‌​​‌‌​​​‌‌​‌‌​‌‌‌‌​‌‌​‌‌​‌