模态框中Checkbox无法正确显示Checked状态的技术问题

你的代码存在两个核心问题：

1. 嵌套循环导致重复渲染：外层遍历所有权限，内层遍历角色权限，会让每个权限被渲染多次（次数等于角色权限的数量），页面出现大量重复复选框。
2. 未过滤当前角色的权限：role_permissions查询没有指定当前角色，拿到的是所有角色关联的权限，导致checked状态判断完全错误。

第一步：优化后端查询（Controller） #

不需要关联permissions表，只获取当前角色对应的permission_id集合即可，用pluck直接得到纯ID集合，判断效率更高：

$permissions = DB::table('permissions')->get();
// 仅查询当前角色的关联权限ID，必须通过where指定角色
$role_permission_ids = DB::table('role_has_permissions')
    ->where('role_id', $role->id)
    ->pluck('permission_id');

return view('role.index', compact('role', 'permissions', 'role_permission_ids'));

第二步：修正视图渲染逻辑 #

去掉嵌套循环，只遍历一次所有权限，用集合的contains方法快速判断当前权限是否属于该角色：

@foreach ($permissions as $permission)
    <div class="form-check ps-1 mt-2">
        <input 
            {{ $role_permission_ids->contains($permission->id) ? 'checked' : '' }} 
            disabled 
            id="permission-{{ $permission->id }}" 
            name="permissions_id[]" 
            class="form-check-input form-disabled" 
            type="checkbox" 
            value="{{ $permission->id }}"
        >
        <label class="form-check-label" for="permission-{{ $permission->id }}">{{ $permission->name }}</label>
    </div>
@endforeach

优化说明 #

• pluck('permission_id')返回Laravel集合，contains方法内部做了优化，判断ID是否存在的效率远高于手动循环。
• 单循环确保每个权限只渲染一次，页面结构清晰，不会出现重复元素。
• 给复选框ID加上permission-前缀，避免和页面其他元素ID冲突，更符合HTML规范。

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