评审五问
评审五问
1. 本 PR 同时回答了几个「变化原因」?
2. 新增一个同类实现,要触碰几处既有代码?
3. 把子类换成父类引用,断言还成立吗?
4. 有无空实现/抛不支持的占位方法?
5. 业务代码里 import 了具体基础设施吗?清单用于对话与提醒,不用于打分:每条都有合理例外,重点是让权衡显式化。
评审中用五个问题快速扫描:这个类为什么会被改?新增同类能力要改哪里?子类替上去行为还成立吗?有没有人被迫实现不用到的方法?依赖方向指向抽象了吗?
1. 本 PR 同时回答了几个「变化原因」?
2. 新增一个同类实现,要触碰几处既有代码?
3. 把子类换成父类引用,断言还成立吗?
4. 有无空实现/抛不支持的占位方法?
5. 业务代码里 import 了具体基础设施吗?