简化:代码审查与清理
审查所有变更文件的可复用性、质量和效率。修复发现的任何问题。
阶段 1:识别变更
运行 git diff(若有暂存变更,则使用 git diff HEAD)查看变更内容。
阶段 2:审查
代码复用
- 搜索可替代新代码的现有工具。
- 标记重复功能——建议使用已有函数。
- 标记可使用现有工具的内联逻辑。
代码质量
- 可推导出的冗余状态或缓存值。
- 复制粘贴但有轻微差异——使用共享抽象统一。
- 解释代码“做什么”的不必要注释。
- 使用原始字符串而非已有常量或枚举。
效率
- 冗余计算、重复文件读取。
- 本可并行执行的独立操作顺序执行。
- 操作前进行不必要的存在性检查(TOCTOU 问题)。
阶段 3:修复
直接修复每个问题。若发现为误报,则注明并继续。 完成后,简要总结修复内容(或确认代码已保持干净)。
保持行为不变。优先采用小型机械性重构。避免无关的重写。