代码评审提示词适合一种情况:代码能跑,但你不确定里面有没有埋着问题。直接说“帮我看看这段代码”,AI 常常只夸写得不错,或者一口气重写一遍。把检查范围、严重程度和输出格式写进提示词里,它才会像评审员一样挑问题:先说哪里必须改,再说哪里可以更好。
先复制这段提示词
你现在是一名资深代码评审员,熟悉[编程语言],做过[代码用途]这类代码的评审。
请评审我贴在最后的代码,只看贴出的内容,不要猜测没贴出的部分。
按这个顺序检查:1)正确性:逻辑错误、边界条件、空值和异常处理;2)安全:注入、越权、敏感信息泄露;3)性能:明显多余的循环、重复查询、大对象不必要的复制;4)可读性和可维护性:命名、重复代码、过长的函数;5)测试缺口:哪些分支没有测试覆盖。
输出时先给问题清单,再给修改建议。每个问题写清严重程度、位置、问题和改法,严重程度只用这三种:必须改、建议改、可以先放着。只给“必须改”的问题写修改代码,其他问题只说改法,不重写整段代码。拿不准的地方写“不确定”,并说缺什么信息才能确定。最后问我一个最关键的背景问题。
评审时重点看[评审重点],按[严格程度]把握尺度。
代码如下:
[粘贴代码]5 个方括号这样填
- [编程语言]:写具体语言和版本,比如 Python 3.12,别只写“编程语言”。
- [代码用途]:写这段代码是干什么的,比如订单金额计算、用户登录接口,评审标准跟着用途走。
- [评审重点]:安全、性能、可读性里选一两个,别全选,全选等于没重点。
- [严格程度]:严格是连风格问题也列出来,宽松是只列影响正确性和安全的问题。
- [粘贴代码]:贴完整的函数,别只贴一半;超过 200 行先拆成几段,分段评审。
一个填好的开头长这样
你现在是一名资深代码评审员,熟悉 Python 3.12,做过订单金额计算这类代码的评审。
评审时重点看安全和边界条件,按严格把握尺度。输出不对劲时这样调
- 列的问题太多太细:把严格程度改成宽松,只看“必须改”那几条。
- 改法太笼统:贴上调用这段代码的地方和一个真实输入例子,让它按真实路径检查。
- AI 直接重写了代码:补一句“不要重写代码,只给问题和改法”,重新发一次。
评审完先改“必须改”,跑通测试再看“建议改”,顺序对了,评审才省时间。