Question

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

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

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

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。您也可以在服务类中放置相同的逻辑（可能是最好的解决方案）。

Answer 2

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

注意：但是id将同时是$user1和$user2，所以我们将其设为AND代替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();

顺便说一句，如果您希望使用这些查询来执行此操作，则可以将其放在Engine模型和用户中，或者在需要时重新使用。同样对于助手，它可以在控制器中不可重用的地方重用。

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

2 个答案: