罪魁祸首
因为 $batch_id 为 null。
新建的字段,默认值为 null,因此之前的数据,全被设置为 null。
代码 数据库创建全是 ai 写的,运行下,逻辑没问题,通过!
结果昨天 boss 说就删了一条对账,对账全不见了...
后续
问题也不大,单独恢复那个表数据,运行下校验：
我擦,怎么对不上。
找了几小时,才发现在我下载备份后,boss 又撤销了两个单 内心...
1.用 ai 还是要动下脑子的,ai 可不知道你有老数据的存在
2.备份真的非常重要,有备份 问题都不大。
第 1 条附言 · 2 天前
还有 delete,也有 bug 的,因为 delete 批量操作不会触发`observer`。
 |
1
xdzhang 2 天前
生成的代码敢直接用？
|
 |
2
TimePPT PRO  1
经验之谈，跟是不是 AI 写代码没关系：所有涉及数据库写改的代码变更，上线前都先在测试环境连测试数据库全部跑一遍测试。生造数据怕漏的，可以把生产环境 Sample 一部分数据灌进去（公司有隐私要求的可以做下脱敏替换啥的），全流程走一遍 crud ，没问题了再上线。
|
 |
3
fuzzsh 2 天前 via Android
……
直接 push & release? |
 |
4
wxw752 2 天前
 |
 |
5
Borch 2 天前
不写测试的吗
|
 |
6
gransh 2 天前
所以测试环境、开发环境、生产环境是同一个是吧。
|
 |
9
cmdOptionKana 2 天前 via Android 3
这事儿 AI 不背锅，流程不规范，就算没有 AI 也早晚出事。
|
 |
10
quqivo 2 天前
什么编辑器
|
 |
12
lyxxxh2 OP @cmdOptionKana
人为写,不会出这种严重的 bug 的。 因为写的过程,会考虑各种情况。 例如 where,肯定会加各种限制条件,如超市 id 对账单 id,而不是单纯一个操作次数完事。 但是用 ai,没时间去考虑。 当然,锅肯定不是 ai 的,因为 ai 只是工具。 |
 |
14
snitfk 2 天前 1
这关 AI 啥事。人一样会写出这种代码。锅在没有规范的发布流程。
|
 |
15
liuliuliuliu 2 天前
我说句题外话，这 php 代码是真难看啊……有人同意吗……
|
 |
16
earthyan 2 天前
数据不做软删除吗
|
 |
17
sunny2580839896 2 天前 2
@liuliuliuliu php 惹你什么了，再说楼主的代码也没有什么问题
|
 |
18
HENQIGUAI 2 天前
涉及到删除的肯定要慎之又慎。
|
 |
19
66450146 2 天前 via iPhone
就你一个人，真没时间不搞测试。出事是迟早的事，有充分的测试覆盖才能把 AI 高效用起来
|
|
20
lyxxxh2 OP |
 |
21
Immortal 2 天前
以前是脚本小子,现在是 AI 小子了
|
 |
22
qiaobeier 2 天前
都不 review 的吗。。。
|
 |
23
mb4555 2 天前
搞了半天 你还是不清楚问题在哪 那就等着下次继续暴雷吧
|
Select Language