从数据库单列取值并在C#中设置多复选框状态的实现方案

一、存储过程设计 #

假设你的用户表名为Users，存储权限的列名为Roles（格式为逗号分隔的字符串，比如"Add Staff,Update Staff,Delete Staff"），存储过程只需根据用户ID查询对应权限值：

CREATE PROCEDURE SPD_GetUserRoles
    @id NVARCHAR(50)
AS
BEGIN
    SET NOCOUNT ON;
    SELECT Roles FROM Users WHERE UserId = @id;
END

二、优化后的C#代码 #

你的现有代码存在逻辑错误（混淆列名与行数据、重复使用数据读取方式），以下是修正后的实现：

private void SetRoles()
{
    using (SqlCommand cmd = new SqlCommand("SPD_GetUserRoles", con))
    {
        cmd.CommandType = CommandType.StoredProcedure;
        cmd.Parameters.AddWithValue("@id", "ABC01");

        try
        {
            con.Open();
            // 读取用户权限字符串，处理空值情况
            string userRoles = cmd.ExecuteScalar()?.ToString() ?? string.Empty;

            // 拆分权限为集合，避免部分匹配的误判
            HashSet<string> roleSet = new HashSet<string>(
                userRoles.Split(new[] { ',' }, StringSplitOptions.RemoveEmptyEntries)
            );

            // 逐个设置Checkbox勾选状态
            CBAddStaff.Checked = roleSet.Contains("Add Staff");
            CBAUpdateStaff.Checked = roleSet.Contains("Update Staff");
            // 其他Checkbox可按此格式扩展
            // CBDeleteStaff.Checked = roleSet.Contains("Delete Staff");
        }
        catch (Exception ex)
        {
            // 可添加异常日志或提示逻辑
            MessageBox.Show($"加载权限失败：{ex.Message}");
        }
        finally
        {
            if (con.State == ConnectionState.Open)
                con.Close();
        }
    }
}

关键说明 #

• 存储过程简化：仅返回目标用户的权限列，减少不必要的数据传输。
• 准确匹配：用HashSet存储拆分后的权限，既提升判断效率，又避免直接用字符串Contains导致的部分匹配错误（比如区分"Add"和"Add Staff"）。
• 资源安全：通过using自动释放命令资源，finally块确保数据库连接必关，避免连接泄漏。
• 空值处理：用?? string.Empty处理数据库返回null的情况，避免空引用异常。

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