Question

我正在尝试使用laravel查询构建器来复制连接：

LEFT JOIN content_userdata
ON content_id = content.id
AND user_id = $user_id

我发现我可以在我的模型中使用以下函数执行额外的“ons”，扩展了Eloquent

public function scopeJoinUserData($query, $user_id)
{
    return $query->leftJoin('content_userdata', function($join)
    {
        $join->on('content_userdata_content_id', '=', 'content.content_id')->on('content_userdata_user_id', '=', 10);
    });
}

但这会产生两个问题。首先，我无法将$ user_id变量放入函数中，其次，即使我将其硬编码用于测试目的，如上所述（对于“10”），Laravel将其包含在`意味着它应该被解释为列名时不会这样：

left join `content_userdata`
on `content_id` = `content`.`id`
and `user_id` = `10`

所以现在我有两个问题。

使用查询范围时，我无法将$ user_id放入连接函数
即使我可以，我也无法向连接发送变量，因为它始终将其解释为列名称

我为什么要这样做？ 我意识到一个回应可能是把它放在哪里。但是我试图这样做，因为连接可能不一定返回任何结果（因此左连接），因为content_userdata表包含诸如用户对一段内容的评级之类的内容。如果我使用where，那么content_userdata表中没有任何内容的结果将不会被返回，就好像我可以将它放入连接中那样，由于左连接，它们将被返回。

无论如何都要在Laravel中实现这一目标，如果没有替代方案，显然完全改变了放弃Laravel是最重要的，但我能想到的唯一选择是在单独的查询中获取用户数据。

Answer 1

您需要使用use关键字将变量传递给闭包 - 它将变量导入范围。例如：

public function scopeJoinUserData($query, $user_id)
{
    return $query->leftJoin('content_userdata', function($join) use ($user_id)
    {
        $join->on('content_userdata_content_id', '=', 'content.content_id')
             ->on('content_userdata_user_id',    '=', DB::raw($user_id));
    });
}

这是与PHP语法相关的问题，而不是Laravel限制！

Answer 2

在接受的答案中，仅在查询的DB::raw部分添加引号将无法完全保护它免受sql注入。只需传递user_id中的一些引号即可。要进行参数化，您可以执行以下操作：

public function scopeJoinUserData($query, $user_id)
{
    return $query->leftJoin('content_userdata', function($join)
        {
            $join->on('content_userdata_content_id', '=', 'content.content_id')
                 ->on('content_userdata_user_id',    '=', DB::raw('?'));
        }
    ->setBindings(array_merge($query->getBindings(),array($user_id)));
}

请注意，在此示例中，您不必将变量传递给闭包。或者，您可以尝试编写此部分completely raw。

更新：Taylor added joinWhere，leftJoinWhere ...如果您有功能加入，只需使用->where和->orWhere来自闭幕。

Answer 3

我自己设法解决了这个问题，底部有一个注释，说明为什么它不是完全最优的，但无论如何都要做到这一点：

public function scopeJoinUserData($query, $user_id)
{
    return $query->leftJoin('content_userdata', function($join) use ($user_id)
    {
        $join->on('content_userdata_content_id', '=', 'content.content_id')->on('content_userdata_user_id', '=', DB::raw('"'.$user_id.'"'));
    });
}

请注意@Half Crazed建议使用“use（$ user_id）”。

DB :: raw（）用于将$ user_id包装在引号中，即使它是整数而不是字符串。这将使用`自动停止Laravel，这使得MySQL将其解释为列名。

性能：有一点需要注意的是，当使用整数而不是字符串时，MySQL查询会快得多，并且如果用引号括起来将它解释为字符串。我现在并不担心这一点，但我认为如果其他人使用它作为解决方案，我应该提一下。

Answer 4

为什么不用关系？这是ORM like Eloquent的全部内容？

像这样;

class User extends Eloquent {
    public function userdata()
    {
        return $this->hasOne('Userdata');
    }
}

$result= User::find(1)->userdata();

编辑以显示您可以通过关系做任何您想做的事

选项1：

$place = new Place;

$array = $place->with(array('users' => function($query)
{
    $query->where('user_id', $user_id);
}))->get();

var_dump($array->toArray());

或选项2：

$place = new Place;

$array = $place->with('users')->where('user_id', $user_id)->get();

var_dump($array->toArray());

两者都有不同的结果 - 但你明白了

Answer 5

您的第一个问题：您应该使用PHP语法进行闭包作为Half的答案。关于你的第二个问题，我认为查询的AND user_id = $user_id部分不属于JOIN子句而是属于WHERE子句，因为它只依赖于一个表，而不是这两个表中的连接关系。我认为你应该使用这样的子查询：

public function scopeJoinUserData($query, $user_id)
{
    return $query->leftJoin(\DB:raw("(SELECT * FROM content_userdata WHERE user_id = {$user_id}) AS t"), function($join)
    {
        $join->on('t.content_id', '=', 'content.content_id');
    });
}

但是，如您所见，请确保$user_id变量是安全的，因为我们使用\DB:raw方法。

Laravel 4 Eloquent Query Builder - 复杂的变量连接

5 个答案: