ASP.NET实现不同用户类型显示对应头部菜单的问题求助

咱们先梳理下你现有代码里的几个关键问题，然后一步步修正：

现有代码的核心错误 #

1. 初始代码逻辑完全偏离需求：你查询了整个Users表的数据，直接取第一行的第5列作为用户类型，这拿到的是数据库里第一个用户的类型，和当前登录的活跃会话用户完全无关，肯定不对。
2. 最新修改的SQL语句逻辑错误：select Usertype from Users where usertype = @usertype这个查询毫无意义——你用会话里的用户类型去查找相同类型的用户，最后再取这个类型，相当于绕了一圈又回到原点。而且你只查询了Usertype一列，代码里用dt.Rows[0][5]会直接抛出索引越界异常，因为结果集只有1列，索引只能是0。
3. 缺少Session为空的判断：如果用户没登录，Session["usertype"]会是空值，直接调用ToString()会触发空引用错误。

正确的实现方案 #

其实最高效的方式是用户登录时就把用户类型存入Session，之后直接从Session读取判断就行，不用每次都查数据库。如果担心Session被篡改，再通过用户ID去数据库验证。

方案1：直接使用Session（推荐，高效） #

首先，在用户登录验证成功后，把用户类型存入Session：

// 登录逻辑示例（验证用户名密码通过后）
string userType = // 从数据库查询当前用户的Usertype字段值
int userId = // 同时存入用户ID，方便后续其他操作
Session["UserType"] = userType;
Session["UserId"] = userId;

然后在页面的Page_Load方法中调用判断逻辑：

protected void Page_Load(object sender, EventArgs e)
{
    if (!IsPostBack)
    {
        CheckUserType();
    }
}

private void CheckUserType()
{
    // 先判断用户是否已登录
    if (Session["UserType"] == null)
    {
        // 未登录时隐藏所有菜单，跳转到登录页
        admin.Visible = false;
        user.Visible = false;
        Response.Redirect("Login.aspx");
        return;
    }

    string userType = Session["UserType"].ToString().Trim();

    // 根据类型显示对应菜单
    if (userType == "U")
    {
        user.Visible = true;
        admin.Visible = false;
    }
    else if (userType == "A")
    {
        admin.Visible = true;
        user.Visible = false;
    }
    else
    {
        // 处理未知用户类型的情况
        admin.Visible = false;
        user.Visible = false;
    }
}

方案2：从数据库验证（更安全，适合敏感场景） #

如果担心Session被篡改，可以通过登录时存入的用户ID，去数据库查询当前用户的真实类型：

protected void Page_Load(object sender, EventArgs e)
{
    if (!IsPostBack)
    {
        CheckUserType();
    }
}

private void CheckUserType()
{
    if (Session["UserId"] == null)
    {
        admin.Visible = false;
        user.Visible = false;
        Response.Redirect("Login.aspx");
        return;
    }

    string CS = ConfigurationManager.ConnectionStrings["BoothsConnectionString1"].ConnectionString;
    using (SqlConnection con = new SqlConnection(CS))
    {
        // 通过用户ID查询真实类型，这才是正确的查询逻辑
        SqlCommand cmd = new SqlCommand("SELECT Usertype FROM Users WHERE UserId = @UserId", con);
        cmd.Parameters.Add("@UserId", SqlDbType.Int).Value = Convert.ToInt32(Session["UserId"]);
        con.Open();

        // 用ExecuteScalar直接获取单个值，比DataAdapter更高效
        object result = cmd.ExecuteScalar();
        if (result != null)
        {
            string userType = result.ToString().Trim();
            if (userType == "U")
            {
                user.Visible = true;
                admin.Visible = false;
            }
            else if (userType == "A")
            {
                admin.Visible = true;
                user.Visible = false;
            }
            else
            {
                admin.Visible = false;
                user.Visible = false;
            }
        }
        else
        {
            // 未找到对应用户，跳登录页
            Response.Redirect("Login.aspx");
        }
    }
}

补充说明 #

• 你的HTML部分的控件ID要和后台代码里的一致，比如HTML里是id="admin"和id="user"，后台就要用admin.Visible和user.Visible，这点你最新代码里已经改对了，没问题。
• 记得给Users表的UserId（或者你的用户唯一标识字段）加索引，提升查询效率。

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