在C#/SQL Server中无法使DataReader正常工作的问题排查

兄弟，我一眼就揪出你代码里的问题了，这也是新手操作SQL Server时很容易踩的坑！

为什么SqlDataReader不工作？ #

你的SQL查询语句直接拼接字符串，犯了两个致命错误：

1. 语法错误：因为Username是字符串类型，拼接后生成的SQL语句里没有给用户名加单引号。比如如果Globals.username是"John"，你的SQL会变成：

   SELECT * FROM Main_Information WHERE Username = John
   

   数据库会把John当成列名而非字符串值，直接抛出语法错误，ExecuteReader执行失败，dr.Read()自然不会有任何结果。
2. SQL注入风险：这种拼接字符串的写法会让你的程序暴露在SQL注入攻击下，恶意用户可以通过构造特殊用户名直接篡改你的SQL语句，甚至删除整个数据库。

彻底解决的正确写法：参数化查询 #

这是数据库操作的标准最佳实践，既解决语法问题，又彻底杜绝SQL注入。修改后的代码如下：

string strConnect = "Server=DESKTOP-2Q73COU\\SQLEXPRESS;Database=LoginApp;Trusted_Connection=True;";

// 用using语句自动释放资源，避免连接泄漏
using (SqlConnection conn = new SqlConnection(strConnect))
{
    conn.Open();
    MessageBox.Show("Connected to SSMS.");

    // 带参数占位符的SQL，不要直接拼字符串！
    string loadMainInfo = "SELECT Standing, First_Name FROM Main_Information WHERE Username = @Username";
    
    using (SqlCommand cmd = new SqlCommand(loadMainInfo, conn))
    {
        // 添加参数并赋值，数据库会自动处理字符串引号和类型转换
        cmd.Parameters.AddWithValue("@Username", Globals.username);
        // 更推荐的写法：明确指定参数类型，避免潜在的类型转换问题
        // cmd.Parameters.Add("@Username", SqlDbType.NVarChar, 50).Value = Globals.username;

        using (SqlDataReader dr = cmd.ExecuteReader())
        {
            if (dr.Read())
            {
                lblStanding.Text = dr["Standing"].ToString();
                lblName.Text = dr["First_Name"].ToString(); // 补全你的列名，比如First_Name
                // 其他控件赋值...
            }
            else
            {
                // 没有找到匹配用户时的提示
                MessageBox.Show("No matching user found.");
            }
        }
    }
}

额外的优化建议 #

• 用using包裹所有可释放资源：SqlConnection、SqlCommand、SqlDataReader都实现了IDisposable，using会自动帮你关闭连接、释放资源，避免数据库连接池耗尽。
• 别用SELECT *：明确写出你需要的列名（比如Standing, First_Name），减少数据传输量，也避免因表结构变更导致的意外问题。
• 添加异常处理：用try-catch捕获数据库操作可能出现的异常，方便调试和给用户友好提示：

  try
  {
      // 上面的数据库操作代码
  }
  catch (Exception ex)
  {
      MessageBox.Show($"Oops, something went wrong: {ex.Message}");
      // 这里可以加日志记录，方便排查问题
  }
  

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