laravel 5.* 中的助手与控制器性能

Question

假设我有一个名为 engine 的 helper。

if( ! function_exists('engine') )
{
    function engine($user_id_1, $user_id_2, $league, $log = true)
    {
        /*
         * variables
         */
        $questionsLevel = 1;

        $user1 = \App\User::where('id', $user_id_1)->first();
        $user2 = \App\User::where('id', $user_id_2)->first();

        $league = \App\Models\League::where('id', $league)->first();

        $users = $league->all_users;

        /*
         * check if users joined to league correctly
         */
         $user1_ok = $user2_ok = false;
         foreach($users as $user)
         {

             if( $user->id == $user_id_1 )
             {
                 $user1_ok = true;
             }
             if( $user->id == $user_id_2)
             {
                 $user2_ok = true;
             }

             $check_users = [
                 $user1_ok,
                 $user2_ok
             ];
         }

        if( in_array(false, $check_users) )
        {
            return [
                'errcode' => 404,
                'errmessage' => 'one ro both user/s did not joined to league'
            ];
        }

       //DO SOME STUFF

    }//function engine
}

如你所知，我可以写一个controller 来做同样的事情。

有谁知道基于 Laravel 架构哪个更快？以及如何在这两种情况下测试性能？

我正在使用laravel 5.7.*

Answer 1

关于性能，我认为不会有任何变化，因为将执行相同的查询以及执行相同数量的操作，例如foreach()循环在这两种情况下都是O(n)，那么会有什么不同，但是您可能希望将代码更改为如下所示的内容，或者也可以使用三元运算符。不同之处在于代码的组织。在控制器中测试会更容易。

注意：但是id 将如何同时是 $user1 和 $user2，所以让我们用 OR 代替 AND

foreach($users as $user)
 {
     if( $user->id == $user_id_1 )
     {
         $user1_ok = true;
     }elseif( $user->id == $user_id_2)
     {
         $user2_ok = true;
     }

     $check_users = [
         $user1_ok,
         $user2_ok
    ];
}

到

foreach($users as $user)
{
    if( in_array($user->id, [$user_id_1, $user_id_2])
    {
        $user1_ok = true;
    }

    $check_users = [
        $user1_ok,
        $user2_ok
    ];
}

或

foreach($users as $user)
{
    if( $user->id == $user_id_1 )
    {
        $check_users['user1_ok'] = $user->id == $user_id_1 ? true : false;
    }
    if( $user->id == $user_id_2)
    {
        $check_users['user2_ok'] = true;
    }
}

如果您想获得user1_ok、user2_ok 的工程师，您可能希望按照@Namoshek 的建议运行查询。

或者您可能想将查询更改为类似

League::query()->where('id', $league)
        ->whereHas('users', function($query) use ($user1, $user2) {
            $query->where('id', $user1)->where('id', $user2);
    })->exists();

但是，$user1 和 $user2 的 id 将如何？所以，如果它是 OR 而不是 AND。

League::query()->where('id', $league)
    ->whereHas('users', function($query) use ($user1, $user2) {
        $query->whereIn('id', [$user1, $user2]);
})->exists();

甚至$users = [$user1, $user2];和

League::query()->where('id', $league)
    ->whereHas('users', function($query) use ($users) {
        $query->whereIn('id', $users);
})->exists();

顺便说一句，如果你想使用这些查询，你可以把它放在引擎模型和用户中，或者在需要时重用它。同样对于助手，它可以在控制器中不可重复使用的地方重复使用。

Answer 2

最快的解决方案是不加载不必要的数据：

if(! function_exists('engine'))
{
    function engine(int $user1, int $user2, int $league)
    {
        return League::query()
            ->where('id', $league)
            ->whereHas('users', function($query) use ($user1) {
                $query->where('id', $user1);
            })
            ->whereHas('users', function($query) use ($user2) {
                $query->where('id', $user2);
            })
            ->exists();
    }
}

一般来说，我不认为这样的函数应该返回复杂的结果，比如错误消息。对于这个特定的工作，您应该使用控制器操作。在我看来，辅助方法应该只返回true 或false。不过，您也可以将相同的逻辑放在服务类中（可能是最好的解决方案）。