Auth::user() 踩坑

AegisforPhP 的个人博客 / 7 / 9 / 创建于 3年前 / 更新于 3年前

Auth::user() 获得当前登录用户实例
本以为Auth::user()->where(‘money’,’>=’,’1’)->decrement(‘money’,$amount);是对当前登录用户的减少金额，结果会把数据库所有余额大于等于1的用户都执行一遍。
只能老老实实用where再判断一下id再进行操作

本作品采用《CC 协议》，转载必须注明作者和本文链接

AegisforPhP

课程读者 49 声望

Java，PHP，Laravel 小白初学者，大佬们多多关照 O(∩_∩)O

0 人点赞

以构建论坛项目 LaraBBS 为线索，展开对 Laravel 框架的全面学习。应用程序架构思路贴近 Laravel 框架的设计哲学。

从零开始带你一步步开发一个 Go 博客项目，让你在最短的时间内学会使用 Go 进行编码。项目结构很大程度上参考了 Laravel。

讨论数量: 9

Marden

315 声望

Auth::user() 返回的是当前登录的用户实例。根据你的描述，代码可以这么组织：

$user = Auth::user();
if($user->money >= 1){
     $user->money = $amount;
     $user->save();
}

3年前评论

GeorgeKing

L5.7 译者 763 声望

第一次见 Auth::user() 能像你这么用，Auth::user() 返回的要么是 Illuminate\Contracts\Auth\Authenticatable 要么是 null。

绝对不可能返回一个 QueryBuilder，你这还能 where，大概率用的不是 Illuminate\Support\Facades\Auth 这个 Facade！

3年前评论

lddtime

不对，Auth::user() 返回的 User 实现了 Authenticatable 但继承自 Model，而 Model 中的 query 恰恰就是 QueryBuilder，所以可以直接调 where

GeorgeKing （作者）

@lddtime 试了一下，确实是可以调用 query，并且 query() 返回的是 Illuminate\Database\Eloquent\Builder。

Marden

315 声望

Auth::user() 返回的是当前登录的用户实例。根据你的描述，代码可以这么组织：

$user = Auth::user();
if($user->money >= 1){
     $user->money = $amount;
     $user->save();
}

3年前评论

JerryBool

293 声望

更新单行记录用orm吧，看你的写法是thinkphp

3年前评论

nff93

课程读者 157 声望 / 搬砖工 @ 工地

看了下源码，在一个模型实例上调用 where 会走 newQuery ，这里只会添加之前的 with 和 withCount ，除此之外没有其它条件。

另外，我也是第一次见有人这么用。

3年前评论

勇敢的心

见习助教 854 声望

第一次见这么用的~

3年前评论

largezhou

课程读者 682 声望 / Ctrl+CV工 @ LZzzz

首先模型实例是可以直接 where 的，但是不会自动把模型的主键加入到查询条件中，，，我以前也因为这么想，这么写，搞过 bug，，，

3年前评论

吃鱼不吐刺

课程读者 126 声望

第一次见到这种写法 :joy:

3年前评论

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

帮助

未填写

私信

所有博文

文章归档

1 篇 2023 年 1 月

3年前 Auth::user() 踩坑

0 Auth::user() 踩坑

成为赞助商

Auth::user() 踩坑

高认可度评论：

社区赞助商

关于 LearnKu

资源推荐

服务提供商

其他信息

Auth::user() 踩坑

高认可度评论：

社区赞助商

关于 LearnKu

资源推荐

服务提供商

其他信息

请登录