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

全部分类/编程与排障

难度 进阶 · 评审 风险 diff

代码评审:按影响面排序,不挑格式

收到一份改动,想快速找出真正会出事的地方。

提示词

评审这段改动。

改动内容:
{{diff}}
这段代码在系统里的位置:{{上下文}}
我最担心的:{{担心}}

按这个顺序给意见:
1. 会导致故障的(数据丢失、越权、并发、异常吞掉、资源泄漏),每条写清触发条件和后果。
2. 以后会变坑的(耦合、隐式约定、命名误导),说明什么情况下会咬人。
3. 可以不改的(风格、偏好),单独归一类,明确标「不影响合并」。
每条意见给出具体行号或代码片段。没有问题的部分直接说没问题,不要凑数。

开工前要交代清楚的信息

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

  • {{diff}}:贴unified diff,比贴整个文件好
  • {{上下文}}:这段代码被谁调用、跑在什么环境
  • {{担心}}:把你隐约不安的点说出来,命中率会高很多

必须逐项过一遍

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

  • 分三档是关键:不然它会用格式问题淹没真问题。
  • 「没问题就说没问题」能挡掉硬凑的意见。