Laravel 9中集合分页报错问题求助

核心错误原因 #

你这里的问题出在get()->paginate(10)这行代码：get()方法会直接把符合条件的所有数据查询出来并转换成集合，而paginate()是查询构建器的专属方法，不能在已经获取的集合上调用，这就是导致报错的直接原因。

另外你在循环里逐个查询每个商品的图片，会产生N+1查询问题，严重影响性能，也需要一起优化。

修复步骤 #

1. 修正分页逻辑：直接在查询构建器上调用paginate()，不要先执行get()
2. 预加载关联数据：通过Eloquent的with()方法提前加载商品对应的图片，避免循环查询

第一步：在Product模型中定义关联

先确保你的Product模型里已经定义了和Image的关联关系：

// app/Models/Product.php
class Product extends Model
{
    // ... 其他代码

    public function images()
    {
        return $this->hasMany(Image::class, 'product_id');
    }
}

第二步：修正控制器代码

public function city_listing($city)
{
    // 直接在查询构建器上调用paginate，同时预加载图片关联
    $products = Product::where('city', $city)->with('images')->paginate(10);

    // 不需要手动循环处理，预加载后每个product已经自带images属性
    return view('home.search_file')->with('products', $products);
}

第三步：视图中渲染分页链接

在你的search_file.blade.php视图里，循环展示商品后，添加分页链接：

@foreach($products as $product)
    <!-- 展示商品信息 -->
    <div>商品名称：{{ $product->name }}</div>
    <!-- 展示关联图片 -->
    @foreach($product->images as $image)
        <img src="{{ $image->url }}" alt="商品图片">
    @endforeach
@endforeach

<!-- 渲染Laravel自带的分页链接 -->
{{ $products->links() }}

额外说明 #

如果确实需要对已有的集合进行分页（比如数据不是直接从数据库查询的），可以使用Illuminate\Pagination\LengthAwarePaginator手动创建分页实例，但你的场景完全不需要，直接用查询构建器的paginate更高效。

内容的提问来源于stack exchange，提问作者Saviour Mkpong