第 59 卷
第五步:代码审查
测试通过后,不代表这次修改就可以直接交付。
还需要做代码审查。
代码审查可以理解成:
不只是看代码能不能跑,还要看代码改得对不对、稳不稳、有没有风险。
Codex 写代码很快,但它也可能出现这些问题:
常见问题说明
| 功能能跑,但逻辑不对 | 表面正常,真实业务流程有问题 |
|---|---|
| 改动太大 | 为了一个小需求,改了很多无关代码 |
| 误删旧逻辑 | 删除了看似没用、实际有用的代码 |
| 忽略边界条件 | 正常输入能用,异常输入就崩 |
| 安全问题 | 暴露密钥、权限判断错误、输入未校验 |
| 风格不统一 | 新代码和原项目写法不一致 |
| 可维护性差 | 临时写死、硬编码、后续不好改 |
所以代码审查不是可选项,而是 Codex 工作流里的关键一步。
两轮审查:Codex 自审 + 人工审查
第一轮先让 Codex 自查刚才的修改(目的不是完全相信它,而是让它先暴露明显问题),最好让它输出成表格:
| 检查项 | 结果 | 说明 |
|---|---|---|
| 是否改到计划外文件 | 否 | 只修改了首页相关组件 |
| 是否新增依赖 | 否 | 没有修改 package.json |
| 是否删除旧逻辑 | 否 | 原有按钮跳转逻辑保留 |
| 是否存在风险 | 有 | 移动端按钮间距还需人工确认 |
第二轮人工审查。最终交付的人是你,不是 Codex。不要求每行都看懂,但要重点看 diff 的这几处:
| 文件范围 | 是否只改了该改的文件 |
|---|---|
| 修改 / 删除内容 | 是否符合计划、有没有删掉旧功能 |
| 命名和结构 | 是否和原项目风格一致 |
| 业务逻辑 | 是否符合真实需求(能跑 ≠ 逻辑对) |
| 测试结果 | 是否真的跑过测试 |
涉及登录、支付、权限、数据库、鉴权等重要代码,建议再用第二个模型做交叉审查——一个模型写,另一个模型专门挑错(只改文案、轻微样式一般不必)。但第二个模型的建议同样不能全盘接受,它帮你发现问
题,不替你做最终决定。
重点盯四类高风险问题
下面四类是 Codex 最容易出问题、也最该重点审的地方:
| 类别 | 常见问题 | 审查要点 |
|---|---|---|
| 边界条 | 正常输入能用、异常输入就崩 | 空数据、接口失败、未登录、权限不足、移动端尺寸、重复点击 |
件
| 安全问 | 涉及用户/接口/权限/支付/上传/ | 是否暴露密钥 token、权限判断是否缺失、输入是否校验、是否泄露 |
|---|---|---|
| 题 | 数据库时 | 敏感信息、接口是否有鉴权 |
| 是否误 | 看似没用、实际有用的代码被删 | 重点看 diff 的删除内容;旧组件、注释、兼容代码、fallback、配置 |
| 删 | 项都可能仍被依赖 | |
| 业务逻 | 代码能跑但逻辑错(跳错页、价格 | 正常路径、异常路径、权限判断、是否覆盖了旧业务规则 |
| 辑 | 算错、越权) |
看到大段删除但 Codex 没解释清楚,就不要直接接受。
代码审查提示词模板
实际使用时,可以直接复制这一段:
请按下面结构审查:
- Codex 自审
- 本次是否只改了计划内文件
- 是否有无关重构
- 是否有新增依赖
- 是否有硬编码
- 是否有误删旧逻辑
- 修改范围审查
- 修改了哪些文件
- 每个文件为什么要改
- 是否存在计划外修改
- 是否有大面积无解释修改
- 边界条件审查
- 空数据如何处理
- 接口失败如何处理
- 用户未登录如何处理
- 权限不足如何处理
- 重复点击如何处理
- 移动端是否可能异常
- 安全问题审查
- 是否暴露密钥、token、账号密码
- 是否影响权限判断
- 是否缺少输入校验
- 是否可能泄露敏感信息
- 是否修改了接口鉴权逻辑
- 删除内容审查
- 删除了哪些代码
- 删除原因是什么
- 是否确认没有其他地方依赖
- 是否可能影响旧功能
- 业务逻辑审查
全站分卷索引 · 72 卷
- 1第 1 卷
- 2第 2 卷
- 3第 3 卷
- 4第 4 卷
- 5第 5 卷
- 6第 6 卷
- 7第 7 卷
- 8第 8 卷
- 9第 9 卷
- 10第 10 卷
- 11第 11 卷
- 12第 12 卷
- 13第 13 卷
- 14第 14 卷
- 15第 15 卷
- 16第 16 卷
- 17第 17 卷
- 18第 18 卷
- 19第 19 卷
- 20第 20 卷
- 21第 21 卷
- 22第 22 卷
- 23第 23 卷
- 24第 24 卷
- 25第 25 卷
- 26第 26 卷
- 27第 27 卷
- 28第 28 卷
- 29第 29 卷
- 30第 30 卷
- 31第 31 卷
- 32第 32 卷
- 33第 33 卷
- 34第 34 卷
- 35第 35 卷
- 36第 36 卷
- 37第 37 卷
- 38第 38 卷
- 39第 39 卷
- 40第 40 卷
- 41第 41 卷
- 42第 42 卷
- 43第 43 卷
- 44第 44 卷
- 45第 45 卷
- 46第 46 卷
- 47第 47 卷
- 48第 48 卷
- 49第 49 卷
- 50第 50 卷
- 51第 51 卷
- 52第 52 卷
- 53第 53 卷
- 54第 54 卷
- 55第 55 卷
- 56第 56 卷
- 57第 57 卷
- 58第 58 卷
- 59第 59 卷
- 60第 60 卷
- 61第 61 卷
- 62第 62 卷
- 63第 63 卷
- 64第 64 卷
- 65第 65 卷
- 66第 66 卷
- 67第 67 卷
- 68第 68 卷
- 69第 69 卷
- 70第 70 卷
- 71第 71 卷
- 72第 72 卷