求助 observer 的 updated 内再次 save 导致死循环

问答 / 225 / 9 / 创建于 10个月前 / 更新于 10个月前

1. 系统环境

CentOS + PHP8.2 + MySQL8.0

2.代码框架

wenman框架使用的 illuminate/database 11.33

3.问题

在observer的updated方法内在此掉用save方法会导致死循环，如下代码：

class Device
{
    public function updated($device) {
        if ($device->isDirty('balance') && $device->balance < 0) {
            $device->state = -1;
            $device->save()
        }
    }
}

执行save后再次触发的updated后$device->isDirty('balance')依然为真，这种情况是否有解决方案？
我当前的解决方案是在查一下这条数据在save

4 声望

暂无个人描述~

《L05 电商实战》

从零开发一个电商项目，功能包括电商后台、商品 & SKU 管理、购物车、订单管理、支付宝支付、微信支付、订单退款流程、优惠券等

《L01 基础入门》

我们将带你从零开发一个项目并部署到线上，本课程教授 Web 开发中专业、实用的技能，如 Git 工作流、Laravel Mix 前端工作流等。

推荐文章：

更多推荐...

Dcat-Plus-Admin：dcat-admin框架的超级增强版，开发效率提升200%！ 15 / 16 |

用 Laravel12 Startkit 做了一个 composer 私有包托管平台 😂 点赞超过 20 个开源，看看需要的人多不多 28 / 19 |

如何打造令后端面试官印象深刻的简历？ 16 / 10 |

花了四个月打磨的 Laravel Plus 开源 34 / 101 |

冯老师的困惑 —— 一个跑了两年的 BUG 22 / 18 |

在laravel下实现全双工的websocket开发 21 / 10 |

还不出来

Laravel 8.x 译者 156 声望

最佳答案

在if中复制一份然后同步一下，这样既不干扰其他changed事件触发，也能保证当前括号内业务继续触发changed事件

if($device->isDirty('balance') && $device->balance < 0){
    $newModel = $device->fresh();
    $newModel->state = -1;
    $newModel->save();
}

10个月前评论

RainLee1990 （楼主）

还是不可以，根据以上代码执行后save并不是更新而是会创建一条新的数据，删除replicate()方法后会更新，但是很奇怪的是save没触发observer

还不出来（作者）

@RainLee1990 答案重新更新了一下，可以使用fresh

讨论数量: 9

王小大

Laravel 9.x 译者 157 声望

静默保存模型 $device->saveQuietly();

快速入门《Laravel 10 中文文档》

10个月前评论

RainLee1990 （楼主）

主要是 observer 内有 state 字段变动的判断根据变更状态在做相应的操作

王小大（作者）

@RainLee1990 在观察者内更新当前模型数据理应静默事件

Imuyu

课程读者 878 声望

使用saveing，在更新前处理，或者强制忽略观察者

10个月前评论

deatil

见习助教 779 声望

updated 通常作为后续日志这种记录的

如果要这样用，换成DB直接改

10个月前评论

保安

Laravel 9.x 译者 418 声望 / 保密 @ 保密

//伪代码

if($device->isDirty('balance')  &&  $device->balance  <  0){ 
    Device::query()->where('id', $device->id)->update(['status'=>-1]);
}

10个月前评论

还不出来

Laravel 8.x 译者 156 声望

在if中复制一份然后同步一下，这样既不干扰其他changed事件触发，也能保证当前括号内业务继续触发changed事件

if($device->isDirty('balance') && $device->balance < 0){
    $newModel = $device->fresh();
    $newModel->state = -1;
    $newModel->save();
}

10个月前评论

RainLee1990 （楼主）

还是不可以，根据以上代码执行后save并不是更新而是会创建一条新的数据，删除replicate()方法后会更新，但是很奇怪的是save没触发observer

还不出来（作者）

@RainLee1990 答案重新更新了一下，可以使用fresh

讨论应以学习和精进为目的。请勿发布不友善或者负能量的内容，与人为善，比聪明更重要！

帮助