C#操作SQL Server报错‘必须声明标量变量@thisuser’求解决

Hey George, let's work through your issues one by one to get this code running correctly!

核心问题分析 #

你遇到的Must declare the scalar variable "@thisuser"报错，根源是你只在SQL语句里定义了参数@thisuser，但没有给SqlCommand对象添加对应的参数值。除此之外，代码里还有DataReader使用顺序错误、字符串拼接的问题，我会一起帮你修正。

具体修复步骤 #

1. 添加SQL参数
   要让SQL Server识别@thisuser变量，必须通过SqlCommand.Parameters集合把变量值传递进去，这不仅能解决未声明的问题，还能避免SQL注入风险。

2. 修正DataReader读取逻辑
   SqlDataReader初始化后并没有指向任何数据行，必须先调用da.Read()移动到第一行，才能调用GetValue()获取数据。你原代码里在Read()之前就调用GetValue(1)会直接触发错误。

3. 修复MessageBox的字符串显示
   你原代码里直接把sda.GetValue(3).ToString()写在字符串里，会显示这段代码文本而不是实际的分数值，需要用字符串插值或者拼接来获取真实数据。

修正后的完整代码 #

private void btnPersonalInfo_Click(object sender, EventArgs e)
{
    // 用using语句自动释放资源，避免数据库连接泄漏
    using (SqlConnection sqlcon = new SqlConnection("你的数据库连接字符串"))
    {
        sqlcon.Open();
        string query = "SELECT score1, score2, score3 FROM tblLogin WHERE username = @thisuser";
        SqlCommand sda = new SqlCommand(query, sqlcon);
        
        // 添加参数，替换成你实际的thisuser变量值
        sda.Parameters.AddWithValue("@thisuser", thisuser); 
        
        using (SqlDataReader da = sda.ExecuteReader())
        {
            // 先判断是否查询到数据行，再读取内容
            if (da.Read())
            {
                // DataReader索引从0开始，对应SELECT语句的列顺序
                string score1 = da.GetValue(0).ToString();
                string score2 = da.GetValue(1).ToString();
                string score3 = da.GetValue(2).ToString();
                
                // 用字符串插值显示真实的分数数据
                MessageBox.Show($"Your Personal Info are: {score1}, {score2}, {score3}");
            }
            else
            {
                MessageBox.Show("No matching user found!");
            }
        }
        // using语句会自动关闭连接，无需手动调用sqlcon.Close()
    }
}

额外优化建议 #

• 尽量避免使用AddWithValue，如果知道参数类型，推荐用Add方法指定类型（比如sda.Parameters.Add("@thisuser", SqlDbType.NVarChar, 50).Value = thisuser;），这样能避免潜在的类型转换问题。
• 始终用using语句包裹数据库相关对象（连接、命令、DataReader），确保资源被正确释放，防止连接池耗尽。

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