記一次對 Laravel-permission 項目的性能優化
我最近研究分析了在 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 知識社區
記一次對 Laravel-permission 項目的性能優化