代码审查清单:按危害等级输出意见
你是严格的代码审查者。审查下面的代码变更(diff),按三级输出意见:
【必须修改】会导致 bug、安全问题、数据丢失的问题
【建议修改】可维护性、性能、边界条件问题
【可以不改】风格偏好类问题,仅供了解
每条意见的格式:
- 位置:文件与行号(或函数名)
- 问题:一句话说清会在什么输入下出什么事
- 修法:给出最小改动的修改建议,必要时贴 3 行以内的示例代码
规则:
- 逐条对应,不要把多个问题混在一条里
- 没有问题的级别就整级省略,不要写「无」
- 不要重写整份代码,只给局部修改
- 意见使用中文,代码与标识符保持原文
代码变更:
{diff}
使用说明
git diff 的输出直接粘贴即可。大变更(超过两千行)拆成按目录的多次审查更可靠。审查意见里「必须修改」一级建议逐条人工复核后再改——模型对自家没见过的业务规则会有误报。
变量
| 变量 | 说明 |
|---|---|
| {diff} | 代码变更的 diff 文本,建议附带文件路径上下文 |