如何在循环中正确使用SqlDataReader？解决循环中断报错问题

你遇到的问题根源是：同一个SqlConnection在打开状态下，同一时间只能有一个SqlDataReader处于活跃状态。第一次循环创建的Reader未关闭，第二次执行cmd.ExecuteReader()时连接被占用，就会触发“需关闭SqlDataReader”的错误。以下是几种正确的解决方式：

方案1：用using语句自动释放Reader（基础规范写法） #

using语句会在代码块结束时自动调用Dispose()，帮你关闭Reader并释放资源，无需手动调用Close()。同时必须改用参数化查询——你的原代码用字符串拼接存在严重SQL注入风险，这是必须修正的问题。

代码示例：

List<string> days = new List<string>() { "Monday", "Tuesday", "Wednesday", "Thursday", "Friday" };
List<string> data = new List<string>();

// 连接也用using自动释放，避免资源泄漏
using (SqlConnection cnn = new SqlConnection(@"Con"))
{
    cnn.Open();
    // 提前定义带参数占位符的命令文本
    using (SqlCommand cmd = new SqlCommand("select * from Table where Day = @Day", cnn))
    {
        // 预先定义参数，避免重复创建
        cmd.Parameters.Add("@Day", SqlDbType.VarChar);
        
        foreach (string day in days)
        {
            cmd.Parameters["@Day"].Value = day;
            
            // Reader用using包裹，自动关闭
            using (SqlDataReader reader = cmd.ExecuteReader())
            {
                while (reader.Read())
                {
                    data.Add(reader[0].ToString());
                }
            }
        }
    }
}

方案2：启用MultipleActiveResultSets（MARS，特殊场景用） #

如果业务需要同时打开多个Reader，可以在连接字符串中添加MultipleActiveResultSets=True，允许一个连接上同时存在多个活跃结果集。但这种方式会增加连接资源占用，仅适用于特殊场景，不推荐作为常规方案。

修改后的连接字符串示例：

Server=myServerAddress;Database=myDataBase;User Id=myUsername;Password=myPassword;MultipleActiveResultSets=True;

即使启用MARS，依然要用using释放Reader资源，避免内存泄漏。

方案3：合并查询（推荐，性能更优） #

当days列表较长时，多次循环查询会增加数据库往返次数，效率低下。可以把所有查询条件合并成一个IN语句，只执行一次数据库操作，大幅提升性能。

代码示例：

List<string> days = new List<string>() { "Monday", "Tuesday", "Wednesday", "Thursday", "Friday" };
List<string> data = new List<string>();

using (SqlConnection cnn = new SqlConnection(@"Con"))
{
    cnn.Open();
    // 构建参数化IN查询，彻底避免注入风险
    string paramNames = string.Join(", ", days.Select((_, index) => $"@Day{index}"));
    string sql = $"select * from Table where Day in ({paramNames})";
    
    using (SqlCommand cmd = new SqlCommand(sql, cnn))
    {
        // 逐个添加参数
        for (int i = 0; i < days.Count; i++)
        {
            cmd.Parameters.Add($"@Day{i}", SqlDbType.VarChar).Value = days[i];
        }
        
        using (SqlDataReader reader = cmd.ExecuteReader())
        {
            while (reader.Read())
            {
                data.Add(reader[0].ToString());
            }
        }
    }
}

内容的提问来源于stack exchange，提问作者Chārry