资讯专栏INFORMATION COLUMN

记一次对 Laravel-permission 项目的性能优化

Nino / 2955人阅读

摘要:我最近研究分析了在上面创建的项目的性能。经过查阅更多资料和研究,发现一个可能明显改善的性能问题。在这个检查的过程中任何的迟钝都会成为整个项目的性能瓶颈。过滤集合类过滤权限集合的方法被认为是造成低性能的原因。使用代替可以提高的性能。

我最近研究分析了在 SWIS上面创建的项目的性能。令人惊讶的是,最耗费性能的方法之一是优秀的  spatie/laravel-permission 包造成的。

经过查阅更多资料和研究,发现一个可能明显改善的性能问题 。既然解决方案已明确阐述,就很容易编写代码改善,提交请求。

现在这个解决方案已被合并和发布,下面是这个性能问题的分析和如何在自己的项目避免这类问题。

TL;DR: 跳转到结论部分.

性能瓶颈

如果我们抽象的看 spatie/laravel-permission 它主要做两件事:

保持一个属于某个模型的权限清单。

检查某个模型是否具有权限。

第一点说是性能瓶颈有点牵强。这里的权限数据存放在数据库中,需要的时候将会被读取出来。这个过程是有点慢但也只是执行一次。结果会被缓存下来,后续的请求可以直接使用。

第二点在性能瓶颈的观点上来看确实是一个瓶颈。 这个瓶颈取决于权限的性质和项目的大小, 因为权限会被频繁的检查。 在这个检查的过程中任何的迟钝都会成为整个项目的性能瓶颈。

过滤集合类

过滤权限集合的方法被认为是造成低性能的原因。 它做了如下事情:

$permission = $permissions
    ->where("id", $id)
    ->where("guard_name", $guardName)
    ->first();

修改后:

$permission = $permissions
    ->filter(function ($permission) use ($id, $guardName) {
        return $permission->id === $id && $permission->guard_name === $guardName;
    })
    ->first();

这两个代码段实现了同一件事情,但第二个更快。

性能测试

我正在开发的应用中大约有 150 个不同的权限。 在一个普通的请求中, 大约有 50 个权限需要用  hasPermissionTo 这个方法去检查,当然,有些页面可能需要检查大约 200 个权限。

以下是用来做性能测试的一些设置。

$users = factory(User::class, 150)->make();
$searchForTheseUsers = $users->shuffle()->take(50);

# 方法 1: where
foreach($searchForTheseUsers as $user) {
    $result = $users->where("id", "=", $user->id)->first();
}

# 方法 2: 过滤,传递一个模型作为回调
foreach($searchForTheseUsers as $searchUser) {
    $result = $users->filter(function($user) use ($searchUser) {
        return $user->id === $searchUser->id;
    })->first();
}

# 方法 3: 过滤,传递属性作为回调
foreach($searchForTheseUsers as $user) {
    $searchId = $user->id;
    $result = $users->filter(function($user) use ($searchId) {
        return $user->id === $searchId;
    })->first();
}

以上三个方法都会被用来测试过滤 1 个属性,2 个属性,3 个属性,所以,用方法 1 过滤三个属性就会是这样:

foreach($searchForTheseUsers as $user) {
    $result = $users
        ->where("id", "=", $user->id)
        ->where("firstname", "=", $user->firstname)
        ->where("lastname", "=", $user->lastname)->first();
}
结果
方法 #1 方法 #2 方法 #3
1个属性 0.190 0.139 (-27%) 0.072 (-62%)
2个属性 0.499 0.372 (-25%) 0.196 (-61%)
3个属性 0.488 0.603 (+25%) 0.198 (-59%)
结论

我们可以得出结论:对一个项目而言,重复的过滤一个大集合会引发严重性能瓶颈。

多属性的过滤明显增加计算成本。

使用 Collection::filter() 代替 Collection::where() 可以提高60%的性能。

警告:传递完整的模型给过滤器回调是很耗费性能的,最好是传递多带带的属性。

致谢

感谢 Spatie 和 spatie/laravel-permissions 的贡献者创建如此优秀的包,我非常喜欢使用!感谢 Andru Beldie 指出这些性能问题,我才有机会对其进行调查和纠正。

链接

spatie/laravel-permission package.

Issue #550 addressing performance problem.

Pull request #710 that fixes problems.

更多现代化 PHP 知识,请前往 Laravel / PHP 知识社区

文章版权归作者所有,未经允许请勿转载,若此文章存在违规行为,您可以联系管理员删除。

转载请注明本文地址:https://www.ucloud.cn/yun/28808.html

相关文章

  • 【 Laravel 工具包推荐--角色/权限管理】

    摘要:在大多数的开发中,角色和权限的管理都是非常重要的一部分。上关于角色和权限管理的包有很多,今天就为大家介绍几个好用的包。缓存在中,为了提高应用的性能,或自动的存储角色和权限数据。 showImg(https://segmentfault.com/img/bVTEb3?w=2200&h=1125); 在大多数的web开发中,角色和权限的管理都是非常重要的一部分。Laravel上关于角色和权...

    xiaoxiaozi 评论0 收藏0
  • 一次tornado QPS 优化

    摘要:初步分析提升可从两方面入手,一个是增加并发数,其二是减少平均响应时间。大部分的时间花在系统与数据库的交互上,到这,便有了一个优化的主题思路最大限度的降低平均响应时间。不要轻易否定一项公认的技术真理,要拿数据说话。 本文最早发表于个人博客:PylixmWiki 应项目的需求,我们使用tornado开发了一个api系统,系统开发完后,在8核16G的虚机上经过压测qps只有200+。与我们当...

    Doyle 评论0 收藏0
  • 一次 webpack 打包体积优化

    摘要:手头做的项目开发得差不多了,而打包配置是一开始粗略配置的,不大的项目打包出来得,所以现在必须进行优化。用于生产环境的打包,设置其为后,这些库会提供最小体积的文件。这种情况打包后的体积要更小一些。最后打包结果的体积开销主要就是以上几项。 手头做的项目开发得差不多了,而打包配置是一开始粗略配置的,不大的项目打包出来得6MB+,所以现在必须进行优化。 打包结果分析 执行命令 webpack ...

    tomlingtm 评论0 收藏0
  • 一次Vue优化及思路

    摘要:记录一个前端项目优化的路程,效果如上图。第二步优化结果再次运行查看项目打包情况可以看到项目体积已经优化到,中也看不到的踪影了。本文主要想提供一些优化思路及手段,即如何定位通过,查看页面加载时间问题,然后再解决这些问题。 showImg(https://segmentfault.com/img/bVbq282?w=381&h=384); 记录一个前端项目优化的路程,效果如上图。 接下来我...

    keithxiaoy 评论0 收藏0
  • 一次Vue优化及思路

    摘要:记录一个前端项目优化的路程,效果如上图。第二步优化结果再次运行查看项目打包情况可以看到项目体积已经优化到,中也看不到的踪影了。本文主要想提供一些优化思路及手段,即如何定位通过,查看页面加载时间问题,然后再解决这些问题。 showImg(https://segmentfault.com/img/bVbq282?w=381&h=384); 记录一个前端项目优化的路程,效果如上图。 接下来我...

    nanchen2251 评论0 收藏0

发表评论

0条评论

Nino

|高级讲师

TA的文章

阅读更多
最新活动
阅读需要支付1元查看
<