Auth::user() 踩坑

AegisforPhP 的个人博客 / 1 / 9 / 创建于 2年前 / 更新于 2年前

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

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

AegisforPhP

课程读者 49 声望

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

0 人点赞

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

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

讨论数量: 9

Marden

308 声望

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

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

2年前评论

GeorgeKing

L5.7 译者 761 声望

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

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

2年前评论

lddtime

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

GeorgeKing （作者）

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

Marden

308 声望

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

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

2年前评论

JerryBool

283 声望

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

2年前评论

nff93

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

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

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

2年前评论

勇敢的心

见习助教 845 声望

第一次见这么用的~

2年前评论

largezhou

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

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

2年前评论

吃鱼不吐刺

课程读者 126 声望

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

2年前评论

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

帮助

未填写

私信

所有博文

文章归档

1 篇 2023 年 1 月

2年前 Auth::user() 踩坑

0 Auth::user() 踩坑

成为赞助商

Auth::user() 踩坑

高认可度评论：

社区赞助商

关于 LearnKu

资源推荐

服务提供商

其他信息

Auth::user() 踩坑

高认可度评论：

社区赞助商

关于 LearnKu

资源推荐

服务提供商

其他信息

请登录