查看: 212| 回复: 6
跳转到指定楼层
上一主题 下一主题
收起左侧

[自我提升] AI 写的代码,到底应该怎么 review

🔗
匿名用户-FH5VH  | 添加认证 | 1 小时前 |倒序浏览

注册一亩三分地论坛,查看更多干货!

您需要 登录 才可以下载或查看附件。没有帐号?注册账号

x
今年年初开始,我们就开始全员 AIl in AI, AI 全部接过写代码,我们这些IC都不写代码了,只管审核AI的代码,并且审核其他组员的代码。

自从使用AI之后,
. 1point 3acres
  • 1.对于我自己接的task上,AI写的代码,我会让AI做逻辑图,跟我把逻辑说清楚。然后对于审核代码,我会让AI带着我,walk through每个核心文件的代码。对于非核心的代码,我也会尽量把他们都看一遍,理解每行代码的意思。如果AI的实现偏离了我的预期,尤其是细节,我会做非常细致的发问,以确认他确实正确理解我的意图,并且细节也和我on the same page. 在walk through的过程中,我可能会让他修改细节,但是由于在让他实现之前,我已经让他给我解释一下他的plan, 所以相关大的质询在实现之前已经发出,并且得到确认。 所以一般来说大的方向不会出问题。 但是对于测试,我几乎没有花多少时间去看。
  • 2.对于同事写的代码, 前2轮review, 我几乎都是AI来review的,然后最后一轮我会自己上, 让AI把主要逻辑讲一下,然后我去看核心4-6个文件的改动,提一些问题和意见之类的。
我在自己的task上,review AI 代码花的时间非常长,有时长达 3-5 个小时。 可能是因为这全是别人写的代码(AI写的),我有点儿完美主义,或许是不安全感的原因,我会反复去查看代码的上下文,以确保我对这个task产生的每行diff都能了然于胸。但是其实代码的上下文也是AI写的,我也不熟,为了了解上下文在干嘛,我又去花很多时间去理解他们。但是在AI all in 之前,我就很讨厌看别人写的代码,可能这个习惯延续下来,我现在每次看AI写的代码,都非常烦躁。效率非常低下。

自从AI AIl in 之后,leader对组员的期待也变得很激进。 他们希望一天能做过去三天的量。我每天几乎都会完成一个 ~2000 diff 的代码,我的同事他们有些完成2个巨大的diff的task。 我之前在组里也算是 high performer, 但是用了 AI之后,我觉得我有点儿跟不上他们的节奏,因为现在刚来的实习生也和我(4+ yoe)的产出差不多。我其实可以放弃review, 就跟平时一样和 AI 讨论一个细致的plan, 让他按着做就行了,但是我心里一直放不下,就怕 AI 细节没有处理好。 我有时候实在来不及,没有review也提交CR,我发现我的reviewer也啥都没有发现。就是正常review。所以我有时候也疑惑自己review到底是不是一个只求心安的安慰剂。

实际上我的review的作用只有俩个:

  • 1.让我自己熟悉这些代码,以后再看至少有一个印象
  • 2. 保证AI在细节实现上也和我预想的一模一样,这个review阶段我偶尔可以发现之前没有发现的细节问题。(我曾经预想 AI 可能会出现很大的隐藏问题,但是经过半年多的实践, 结论就是,只要AI一开的实现plan是无误的,他是可以很好地实现那个蓝图, 如果实在出现问题,那是在plan阶段被答错的一个回复,或者AI提出的问题没有覆盖你在意的风险点。 )
我想知道的是:. Χ

  • 在 review AI 写的代码的时候,我们到底应该怎么review呢。我想对我负责的task的代码负责, 但是我逐行去review的这个习惯会让我的效率慢下来(实际上我这半年几乎天天加班,然后还对代码熟悉度大不如去年)。 其实让我把核心代码理解透,让他们完全和我自己一开始的plan连接起来我都得花几个小时。 但是实际上我们现在的期待是,从接到 task 到 完成实现,再到请人review , back and forth, 最终merge, 整个过程只有 3h - 6h 的time estimate。 我根本来不及。
想请教各位前辈,AI 主导写码的前提下,怎么review才能既有安全感,又能赶得上进度,同时又不用天天加班来做review?(我觉得我其实作用不大, 我的review就是让AI给我把代码解释清楚, 好让我能有足够的信心对这些改动负责。)

但是我也不知道是不是因为我的英语不够好,所以速度比别人慢,我的组员全部都是母语英语的。

上一篇:Amazon teammates rants Amelia Xinlin He
全局:
匿名用户 发表于 2026-11-12 01:43:01
请问review已经不需要人工审核 生产代码了吗?
没什么必要了,如果发现styling或者design pattern上的问题直接加到agents或者skills里面就好了
回复

使用道具 举报

全局:
写好verification skill
回复

使用道具 举报

全局:
detailed specs + tests tests and tests
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-FH5VH  | 添加认证 | 半小时前
我在自己人工审核前,就会使用 review skill 让AI 去做三轮审核。 这个review skill 会让agent 去查看相关的design, task desc,我一开始的plan文件 和 test cases。 接着才自己上去看代码。 主要是自己去审核代码的时候, 速度好慢。
回复

使用道具 举报

地里匿名用户
🔗
匿名用户-FH5VH  | 添加认证 | 半小时前
godducklist 发表于 2026-11-12 03:15
detailed specs + tests tests and tests

请问review已经不需要人工审核 生产代码了吗?
回复

使用道具 举报

全局:
ai写的代码,我一般让ai总结为什么这样写,为什么选用这样的算法,让它自己提炼一遍。
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 注册账号
职场达人
  • ↑ 本版用于讨论职场各种干货话题,闲聊请去🔗聊聊或者🔗匿名版
  • ❌ 本版严禁水贴,引战,发布广告,拉群,贴个人联系方式,扣分无警告
  • ☑ 求职、面经等去 🔗北美求职和 🔗回国求职大区,刷题和学习请去 🔗终身学习大区
  • ☑ 请去专版发布 🔗内推, 🔗招聘信息,和讨论 🔗创业内容
  • ☑ PIP / DevList/ Need Support 等话题也已开设 🔗专版

本版积分规则

>
快速回复 返回顶部 返回列表