Laravel查询users表关联screenshots仅返回单条用户ID问题咨询

问题根因 #

• 第一个循环foreach ($data as $ll)完全冗余，$data是固定键LoggedUserInfo的数组，直接取对应值即可，循环遍历没有实际意义。
• 第二个循环foreach ($ff as $llo2) {$ll2 = $llo2->uniqueid;}存在逻辑缺陷：每次循环都会覆盖$ll2的取值，循环结束后$ll2只会保留最后一次遍历到的用户的uniqueid，仅能匹配单个用户。
• 最终查询screenshots表时使用=匹配单个$ll2，自然只能返回单个用户对应的截图数据。

参考数据表 #

• users用户表
  
• screenshots截图表
  

修复后代码 #

// 获取当前登录管理员信息，主键查询直接用find更简洁
$currentAdmin = Admin::find(session('LoggedUser'));
// 提取当前管理员关联的admin_id
$adminId = $currentAdmin->admin_id;

// 直接取出当前管理员名下所有用户的uniqueid数组，无需循环遍历
$userUniqueIds = \DB::table('users')
    ->where('admin_id', $adminId)
    ->pluck('uniqueid')
    ->toArray();

// 使用whereIn匹配所有用户的uniqueid，查询对应截图
$screenshots = \DB::table('screenshots')
    ->whereIn('userid', $userUniqueIds)
    ->paginate(15);

业务逻辑优化建议 #

• 配置ORM关联关系：给Admin模型添加hasMany(User::class, 'admin_id')关联，给User模型添加hasMany(Screenshot::class, 'userid', 'uniqueid')关联，后续可直接通过链式调用$currentAdmin->users()->with('screenshots')->paginate(15)获取数据，代码可读性和可维护性更高。
• 增加边界校验：补充登录态失效、管理员无关联用户等场景的空值判断，避免代码抛出异常。
• 优化查询性能：给users.admin_id、screenshots.userid字段添加普通索引，数据量较大时可大幅提升查询效率。

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