Question

我有以下代码：

$users = User::with('profile')->paginate($count);

和

$roles = Role::withCount('users')->get();

现在使用laravel-debugbar，当我打开我的用户页面时，我看到的是107个查询正在执行，使得页面加载速度非常慢，我感觉当我添加更多用户时，执行的查询总数是会增加。我查看了代码，现在我无法分辨它有什么问题，或者如何减少生成的查询数量。

这些是我的模特：

user.php的

<?php

namespace App\Models;

use Illuminate\Notifications\Notifiable;
use Illuminate\Foundation\Auth\User as Authenticatable;
use Kodeine\Acl\Traits\HasRole;

class User extends Authenticatable
{
    use HasRole, Notifiable;
    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'name', 'email', 'password',
    ];

   protected $table = "users";
    protected $appends = ['role'];

   public function profile()
    {
        return $this->hasOne('App\Models\Profile');
    }

   public function getRoleAttribute() {
        return $this->getRoles();
    }
}

Role.php

<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Model;
use Kodeine\Acl\Traits\HasPermission;
use Cviebrock\EloquentSluggable\Sluggable;

class Role extends Model
{
    use HasPermission, Sluggable;

   protected $table = "roles";

   /**
     * The attributes that are fillable via mass assignment.
     *
     * @var array
     */
    protected $fillable = ['name', 'slug', 'description'];

   /**
     * Roles can belong to many users.
     *
     * @return Model
     */
    public function users()
    {
        return $this->belongsToMany('App\Models\User', 'role_user', 'role_id', 'user_id');
    }

   /**
     * List all permissions
     *
     * @return mixed
     */
    public function getPermissions()
    {
        return $this->getPermissionsInherited();
    }

   /**
     * Checks if the role has the given permission.
     *
     * @param string $permission
     * @param string $operator
     * @param array  $mergePermissions
     * @return bool
     */
    public function can($permission, $operator = null, $mergePermissions = [])
    {
        $operator = is_null($operator) ? $this->parseOperator($permission) : $operator;

       $permission = $this->hasDelimiterToArray($permission);
        $permissions = $this->getPermissions() + $mergePermissions;

       // make permissions to dot notation.
        // create.user, delete.admin etc.
        $permissions = $this->toDotPermissions($permissions);

       // validate permissions array
        if ( is_array($permission) ) {

           if ( ! in_array($operator, ['and', 'or']) ) {
                $e = 'Invalid operator, available operators are "and", "or".';
                throw new \InvalidArgumentException($e);
            }

           $call = 'canWith' . ucwords($operator);

           return $this->$call($permission, $permissions);
        }

       // validate single permission
        return isset($permissions[$permission]) && $permissions[$permission] == true;
    }

   /**
     * @param $permission
     * @param $permissions
     * @return bool
     */
    protected function canWithAnd($permission, $permissions)
    {
        foreach ($permission as $check) {
            if ( ! in_array($check, $permissions) || ! isset($permissions[$check]) || $permissions[$check] != true ) {
                return false;
            }
        }

       return true;
    }

   /**
     * @param $permission
     * @param $permissions
     * @return bool
     */
    protected function canWithOr($permission, $permissions)
    {
        foreach ($permission as $check) {
            if ( in_array($check, $permissions) && isset($permissions[$check]) && $permissions[$check] == true ) {
                return true;
            }
        }

       return false;
    }


   /**
     * Return the sluggable configuration array for this model.
     *
     * @return array
     */
    public function sluggable()
    {
        return [
            'slug' => [
                'source' => 'name'
            ]
        ];
    }
}

Profile.php

<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Model;

class Profile extends Model
{
    protected $table = "profiles";

   public function user()
    {
        return $this->belongsTo('App\Models\User');
    }
}

这似乎是重复的查询：

select `roles`.*, `role_user`.`user_id` as `pivot_user_id`, `role_user`.`role_id` as `pivot_role_id`, `role_user`.`created_at` as `pivot_created_at`, `role_user`.`updated_at` as `pivot_updated_at` from `roles` inner join `role_user` on `roles`.`id` = `role_user`.`role_id` where `role_user`.`user_id` = '4'

修改问题似乎来自我的角色，我正在查看控制器和模型中的代码，我不知道出了什么问题。我将$roles = Role::withCount('users')->get();更改为$roles = Role::with('users')->withCount('users')->get();，现在我生成的查询更少（71仍然很多）。

Answer 1

它是导致这么多过多查询的用户模型上的appends role属性。作为测试注释，附加角色并测试运行的查询数量。您可能需要延迟加载角色以最大限度地减少查询数量。

渴望加载Eloquent

1 个答案: