Laravel实现仅展示当前认证用户图片的功能咨询

嘿，我帮你捋捋这段代码的问题，然后给你调整后的方案～

原代码的问题点 #

• 你用Photos::all()取出了所有用户的图片，之后尝试通过$images->user->user_id判断，这会直接报错——因为all()返回的是Collection集合对象，不是单个Photos模型，集合没有user这个关联属性哦。
• 先取所有图片再过滤的逻辑效率很低，完全可以在数据库查询阶段就只筛选当前用户的图片。
• 方法里的Photos $photos参数是路由模型绑定，但你这里是要展示当前用户的所有图片，不需要单个图片实例，这个参数可以删掉。

修正后的代码方案 #

首先确保你的模型关联是正确的：
在User模型里添加关联（如果还没加）：

public function photos()
{
    // 假设Photos表的外键是user_id，对应User表的id
    return $this->hasMany(Photos::class);
}

然后修改控制器的show方法，两种方式任选：

方式一：通过用户关联直接获取（推荐，更符合Laravel的关联设计）

public function __construct()
{
    $this->middleware('auth');
}

public function show()
{
    $user = auth()->user();
    // 直接通过用户的关联关系获取属于他的图片，还支持分页哦
    $images = $user->photos()->paginate(10); // 10是每页展示数量，按需修改

    return view('profile', compact('images', 'user'));
}

方式二：用where条件直接查询

public function __construct()
{
    $this->middleware('auth');
}

public function show()
{
    $user = auth()->user();
    // 直接在查询时筛选user_id等于当前用户id的图片
    $images = Photos::where('user_id', $user->id)->paginate(10);

    return view('profile', compact('images', 'user'));
}

这样修改后，你的profile页面就能正确展示当前认证用户的所有图片啦，还保留了分页功能，和你原来的图库分页逻辑保持一致～

内容的提问来源于stack exchange，提问作者Totoro Ghibli