跳到正文
MyDSH我的 DSH 实战笔记
English
独立维护,非DeepSeek官方;与mydsh.dev等其他MyDSH站点无关联。

全部分类/编程与排障

排障与定位 · 难度 进阶 · 排障 重构 测试

重构:先加测试再动结构

一个函数 800 行,要重构成可维护的,但不能改变行为。

提示词

你排查问题从不先改代码。你先说清「什么情况下必现、什么情况下不出现」,然后用最小改动去验证一个假设,一次只动一个地方。

任务:一个函数 800 行,要重构成可维护的,但不能改变行为。

【动手前先做三件事】
1. 用一句话复述这件事的目标:成果最终给谁看、用来做什么决定。
2. 下面这些信息如果我没交代,直接问我,不要自己假设:
   - 这个函数的所有调用方与输入范围
   - 已有测试
   - 最常改动的分支
3. 先说清你打算怎么做(不超过 5 步),等我确认再往下做。

【必须逐项过一遍】
- 先补覆盖主要分支的测试,再动代码
- 给出拆分方案:按输入校验、业务计算、输出组装
- 每一步重构后能独立验证
- 明确说明哪些边界行为保持不变

【工序】
1. 先复述现象:什么操作、什么输入、看到什么、期望什么
2. 给出 3 个假设,按可能性排序,每个写「如果成立应该观察到什么」
3. 设计最小验证:一条命令、一行日志或一个只跑一次的测试
4. 根据结果排除或确认,不要一次改多处
5. 定位后说明根因是「为什么会写错」而不只是「哪行错」

【输出】
现象 → 假设(带预测)→ 最小验证 → 结论 → 下一步。

【不要】
- 不要在没有日志或复现的情况下给结论
- 不要一次改三个地方,否则不知道哪个起作用
- 不要用「试试看」代替验证
- 不确定就说不确定,并给验证方法

开工前要交代清楚的信息

这几项没交代,模型就会自己假设一个填进去——大多数「答得不对」都是从这里开始的。

  • 这个函数的所有调用方与输入范围
  • 已有测试
  • 最常改动的分支

必须逐项过一遍

这一行活最容易漏的地方。已经写进提示词了,不用你再记。

  • 先补覆盖主要分支的测试,再动代码
  • 给出拆分方案:按输入校验、业务计算、输出组装
  • 每一步重构后能独立验证
  • 明确说明哪些边界行为保持不变