外观
AI 写出的代码,应该按什么顺序审查?
标准回答
先确认它有没有正确解决需求,再看接口和生命周期,随后检查线程、错误路径、权限与资源,最后用构建和测试验证。语法正确只是第一关,尤其要核对它引用的 API 是否真的存在、版本是否匹配。
- 阅读实际 diff,重点看新依赖、外部请求、数据流和被删掉的逻辑。
- 核实线程切换、取消传播、空值、重复回调与权限拒绝,不能只走成功路径。
- 关键接口查官方文档;验证不通过时先找原因,不让 AI 随意删断言或屏蔽报错。
通俗例子
AI 给 Fragment 加了设备监听,正常打开页面能收到数据,但退出页面没有移除监听。审查时追踪“在哪里注册、在哪里释放、回调到来时视图还在不在”,就能发现这个潜在泄漏。
深入解释
**为什么编译通过还不够:**编译器能发现部分语法和类型问题,却不知道需求是不是理解错、页面退出后是否仍更新、硬件超时是否被误报成功。
**验证要覆盖哪几条路径:**先跑正常路径,再检查失败、重复回调、取消、生命周期结束等边界。审查者应能解释每个关键改动为何需要,而不是只复述 AI 生成的说明。
易混淆点
AI 说“已修复”或“测试通过”不是证据,要看到真实的执行结果和对应场景。
参考资料
内容来源
- Android · AI Agent 知识手册 · 原条目
job-C02