评审十问(摘)
评审十问(摘)
1. 依赖方向:核心是否 import 了框架/存储细节?
2. 扩展性:新增一个同类实现要碰哪些文件?
3. YAGNI:哪个机制目前无人使用?
4. 组合/继承:继承是否只为复用?
5. 封装:外部能绕过校验改内部状态吗?
6. 兼容:这个变更对调用方是破坏性的吗?清单的目标是让权衡过程可讨论:原则冲突时(如 DRY vs KISS),把取舍写在 PR 描述里。
把原则变成评审问题:这段代码的依赖方向如何?新增同类能力要改几处?有没有为假想需求预留的机制?模块内是否只讲同一层的事?回答时不求一致,要求显式。
1. 依赖方向:核心是否 import 了框架/存储细节?
2. 扩展性:新增一个同类实现要碰哪些文件?
3. YAGNI:哪个机制目前无人使用?
4. 组合/继承:继承是否只为复用?
5. 封装:外部能绕过校验改内部状态吗?
6. 兼容:这个变更对调用方是破坏性的吗?