Laravel-Excel 导出查询时重复JOIN问题分析与解决方案

问题背景

在使用Laravel-Excel进行数据导出时，开发者可能会遇到一个特殊问题：当通过构造函数传递查询构建器对象并在query()方法中添加JOIN语句时，这些JOIN会被执行两次。这个问题主要出现在实现了FromQuery接口的导出类中。

问题重现

假设我们有以下导出类结构：

class ExcelReviewExport implements FromQuery, WithHeadings
{
    use Exportable;

    public $query;

    public function __construct($query)
    {
        $this->query = $query;
    }

    public function query(){
        return $this->query
            ->select(['id'])
            ->join("products", "reviews.product_id", '=', "products.id");
    }
}

当这样调用时：

$query = Review::query();
return (new ExcelReviewExport($query))->download('reviews.xlsx');

生成的SQL会包含重复的JOIN语句。

问题原因

经过分析，这个问题源于Laravel-Excel内部的工作机制：

1. 导出过程会多次调用query()方法
2. Laravel的查询构建器是基于对象引用的
3. 每次调用query()都会在原查询对象上添加新的JOIN条件
4. 最终生成的SQL就包含了重复的JOIN语句

解决方案

方案一：克隆查询对象

最简单的解决方案是在query()方法中克隆查询对象：

public function query(){
    return $this->query
        ->clone()
        ->select(['id'])
        ->join("products", "reviews.product_id", '=', "products.id");
}

这种方法确保每次调用query()都操作一个新的查询对象实例。

方案二：静态变量控制

对于更复杂的情况，可以使用静态变量确保query()方法只执行一次修改：

public function query(){
    static $done = false;
    if ($done) {
        return $this->query;
    }
    $done = true;
    
    return $this->query
        ->select(['id'])
        ->join("products", "reviews.product_id", '=', "products.id");
}

方案三：预构建查询

将JOIN操作放在构造函数调用之前：

$query = Review::query()
    ->join("products", "reviews.product_id", '=', "products.id");
return (new ExcelReviewExport($query))->download('reviews.xlsx');

然后在导出类中只做选择操作：

public function query(){
    return $this->query->select(['id']);
}

最佳实践建议

1. 对于简单查询，推荐使用方案三，将查询构建完全放在导出类外部
2. 对于需要动态修改的复杂查询，使用方案一的克隆方法
3. 只有在特殊情况下才考虑使用方案二的静态变量方法
4. 避免在query()方法中进行会改变查询状态的操作

总结

这个问题揭示了Laravel查询构建器与Laravel-Excel交互时的一个微妙之处。理解查询构建器是基于对象引用这一特性，对于避免此类问题非常重要。选择哪种解决方案取决于具体的应用场景和查询复杂度，但所有方案都能有效解决重复JOIN的问题。