記一次對 Laravel-permission 項目的性能優化

時間 2019-12-10

標籤一次 laravel permission 項目性能優化欄目系統性能简体版

原文原文鏈接

我最近研究分析了在 SWIS上面建立的項目的性能。使人驚訝的是，最耗費性能的方法之一是優秀的 spatie/laravel-permission 包形成的。laravel

通過查閱更多資料和研究，發現一個可能明顯改善的性能問題。既然解決方案已明確闡述，就很容易編寫代碼改善，提交請求。git

如今這個解決方案已被合併和發佈，下面是這個性能問題的分析和如何在本身的項目避免這類問題。github

TL;DR: 跳轉到結論部分.數據庫

性能瓶頸

若是咱們抽象的看 spatie/laravel-permission 它主要作兩件事：緩存

保持一個屬於某個模型的權限清單。
檢查某個模型是否具備權限。

第一點說是性能瓶頸有點牽強。這裏的權限數據存放在數據庫中，須要的時候將會被讀取出來。這個過程是有點慢但也只是執行一次。結果會被緩存下來，後續的請求能夠直接使用。性能

第二點在性能瓶頸的觀點上來看確實是一個瓶頸。這個瓶頸取決於權限的性質和項目的大小，由於權限會被頻繁的檢查。在這個檢查的過程當中任何的遲鈍都會成爲整個項目的性能瓶頸。測試

過濾集合類

過濾權限集合的方法被認爲是形成低性能的緣由。它作了以下事情：spa

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

修改後：code

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

這兩個代碼段實現了同一件事情，但第二個更快。orm

性能測試

我正在開發的應用中大約有 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();
}