← 返回目录

第 59 卷

第 59 / 72 卷日期:

第五步:代码审查

测试通过后,不代表这次修改就可以直接交付。

还需要做代码审查。

代码审查可以理解成:

不只是看代码能不能跑,还要看代码改得对不对、稳不稳、有没有风险。

Codex 写代码很快,但它也可能出现这些问题:

常见问题说明

功能能跑,但逻辑不对表面正常,真实业务流程有问题
改动太大为了一个小需求,改了很多无关代码
误删旧逻辑删除了看似没用、实际有用的代码
忽略边界条件正常输入能用,异常输入就崩
安全问题暴露密钥、权限判断错误、输入未校验
风格不统一新代码和原项目写法不一致
可维护性差临时写死、硬编码、后续不好改

所以代码审查不是可选项,而是 Codex 工作流里的关键一步。

两轮审查:Codex 自审 + 人工审查

第一轮先让 Codex 自查刚才的修改(目的不是完全相信它,而是让它先暴露明显问题),最好让它输出成表格:

检查项结果说明
是否改到计划外文件否只修改了首页相关组件
是否新增依赖否没有修改 package.json
是否删除旧逻辑否原有按钮跳转逻辑保留
是否存在风险有移动端按钮间距还需人工确认

第二轮人工审查。最终交付的人是你,不是 Codex。不要求每行都看懂,但要重点看 diff 的这几处:

文件范围是否只改了该改的文件
修改 / 删除内容是否符合计划、有没有删掉旧功能
命名和结构是否和原项目风格一致
业务逻辑是否符合真实需求(能跑 ≠ 逻辑对)
测试结果是否真的跑过测试

涉及登录、支付、权限、数据库、鉴权等重要代码,建议再用第二个模型做交叉审查——一个模型写,另一个模型专门挑错(只改文案、轻微样式一般不必)。但第二个模型的建议同样不能全盘接受,它帮你发现问

题,不替你做最终决定。

重点盯四类高风险问题

下面四类是 Codex 最容易出问题、也最该重点审的地方:

类别常见问题审查要点
边界条正常输入能用、异常输入就崩空数据、接口失败、未登录、权限不足、移动端尺寸、重复点击

件

安全问涉及用户/接口/权限/支付/上传/是否暴露密钥 token、权限判断是否缺失、输入是否校验、是否泄露
题数据库时敏感信息、接口是否有鉴权
是否误看似没用、实际有用的代码被删重点看 diff 的删除内容;旧组件、注释、兼容代码、fallback、配置
删项都可能仍被依赖
业务逻代码能跑但逻辑错(跳错页、价格正常路径、异常路径、权限判断、是否覆盖了旧业务规则
辑算错、越权)

看到大段删除但 Codex 没解释清楚,就不要直接接受。

代码审查提示词模板

实际使用时,可以直接复制这一段:

请按下面结构审查:

  1. Codex 自审
  • 本次是否只改了计划内文件
  • 是否有无关重构
  • 是否有新增依赖
  • 是否有硬编码
  • 是否有误删旧逻辑
  1. 修改范围审查
  • 修改了哪些文件
  • 每个文件为什么要改
  • 是否存在计划外修改
  • 是否有大面积无解释修改
  1. 边界条件审查
  • 空数据如何处理
  • 接口失败如何处理
  • 用户未登录如何处理
  • 权限不足如何处理
  • 重复点击如何处理
  • 移动端是否可能异常
  1. 安全问题审查
  • 是否暴露密钥、token、账号密码
  • 是否影响权限判断
  • 是否缺少输入校验
  • 是否可能泄露敏感信息
  • 是否修改了接口鉴权逻辑
  1. 删除内容审查
  • 删除了哪些代码
  • 删除原因是什么
  • 是否确认没有其他地方依赖
  • 是否可能影响旧功能
  1. 业务逻辑审查
全站分卷索引 · 72 卷
  1. 1第 1 卷
  2. 2第 2 卷
  3. 3第 3 卷
  4. 4第 4 卷
  5. 5第 5 卷
  6. 6第 6 卷
  7. 7第 7 卷
  8. 8第 8 卷
  9. 9第 9 卷
  10. 10第 10 卷
  11. 11第 11 卷
  12. 12第 12 卷
  13. 13第 13 卷
  14. 14第 14 卷
  15. 15第 15 卷
  16. 16第 16 卷
  17. 17第 17 卷
  18. 18第 18 卷
  19. 19第 19 卷
  20. 20第 20 卷
  21. 21第 21 卷
  22. 22第 22 卷
  23. 23第 23 卷
  24. 24第 24 卷
  25. 25第 25 卷
  26. 26第 26 卷
  27. 27第 27 卷
  28. 28第 28 卷
  29. 29第 29 卷
  30. 30第 30 卷
  31. 31第 31 卷
  32. 32第 32 卷
  33. 33第 33 卷
  34. 34第 34 卷
  35. 35第 35 卷
  36. 36第 36 卷
  37. 37第 37 卷
  38. 38第 38 卷
  39. 39第 39 卷
  40. 40第 40 卷
  41. 41第 41 卷
  42. 42第 42 卷
  43. 43第 43 卷
  44. 44第 44 卷
  45. 45第 45 卷
  46. 46第 46 卷
  47. 47第 47 卷
  48. 48第 48 卷
  49. 49第 49 卷
  50. 50第 50 卷
  51. 51第 51 卷
  52. 52第 52 卷
  53. 53第 53 卷
  54. 54第 54 卷
  55. 55第 55 卷
  56. 56第 56 卷
  57. 57第 57 卷
  58. 58第 58 卷
  59. 59第 59 卷
  60. 60第 60 卷
  61. 61第 61 卷
  62. 62第 62 卷
  63. 63第 63 卷
  64. 64第 64 卷
  65. 65第 65 卷
  66. 66第 66 卷
  67. 67第 67 卷
  68. 68第 68 卷
  69. 69第 69 卷
  70. 70第 70 卷
  71. 71第 71 卷
  72. 72第 72 卷