Ошибка «Существует уже открытая DataReader, связанная с этой командой, которая должна быть закрыта первой» при использовании 2 различных команд

Question

У меня есть этот унаследованный код:

private void conecta() 
{ 
    if (conexao.State == ConnectionState.Closed) 
     conexao.Open(); 
} 

public List<string[]> get_dados_historico_verificacao_email_WEB(string email) 
{ 
    List<string[]> historicos = new List<string[]>(); 
    conecta(); 

    sql = 
     @"SELECT * 
     FROM historico_verificacao_email 
     WHERE nm_email = '" + email + @"' 
     ORDER BY dt_verificacao_email DESC, hr_verificacao_email DESC"; 

    com = new SqlCommand(sql, conexao); 
    SqlDataReader dr = com.ExecuteReader(); 

    if (dr.HasRows) 
    { 
     while (dr.Read()) 
     { 
      string[] dados_historico = new string[6]; 
      dados_historico[0] = dr["nm_email"].ToString(); 
      dados_historico[1] = dr["dt_verificacao_email"].ToString(); 
      dados_historico[1] = dados_historico[1].Substring(0, 10); 
      dados_historico[2] = dr["hr_verificacao_email"].ToString(); 
      dados_historico[3] = dr["ds_tipo_verificacao"].ToString(); 

      sql = 
       @"SELECT COUNT(e.cd_historico_verificacao_email) QT 
       FROM emails_lidos e 
       WHERE e.cd_historico_verificacao_email = 
        '" + dr["cd_historico_verificacao_email"].ToString() + "'"; 

      tipo_sql = "seleção"; 
      conecta(); 
      com2 = new SqlCommand(sql, conexao); 

      SqlDataReader dr3 = com2.ExecuteReader(); 
      while (dr3.Read()) 
      { 
       //quantidade de emails lidos naquela verificação 
       dados_historico[4] = dr3["QT"].ToString(); 
      } 
      dr3.Close(); 
      conexao.Close(); 

      //login 
      dados_historico[5] = dr["cd_login_usuario"].ToString(); 
      historicos.Add(dados_historico); 
     } 
     dr.Close(); 
    } 
    else 
    { 
     dr.Close(); 
    } 

    conexao.Close(); 
    return historicos; 
} 

я создал два отделяет команды, чтобы исправить но он по-прежнему продолжается: «Существует уже открытый DataReader, связанный с этой Командой, который должен быть закрыт первым».

Дополнительная информация: тот же код работает в другом приложении.

Answer 1

Предлагаю создать дополнительное соединение для второй команды, решив ее. Попробуйте объединить оба запроса в одном запросе. Создайте подзапрос для подсчета.

while (dr3.Read()) 
{ 
    dados_historico[4] = dr3["QT"].ToString(); //quantidade de emails lidos naquela verificação 
} 

Зачем переопределять одно и то же значение снова и снова?

if (dr3.Read()) 
{ 
    dados_historico[4] = dr3["QT"].ToString(); //quantidade de emails lidos naquela verificação 
} 

Будет достаточно.

Answer 2

небось проблема показывается в этой строке

SqlDataReader dr3 = com2.ExecuteReader(); 

Я предлагаю вам выполнить первый читатель и сделать dr.Close(); и итерацию historicos, с другой петлей, выполняя com2.ExecuteReader().

public List<string[]> get_dados_historico_verificacao_email_WEB(string email) 
    { 

     List<string[]> historicos = new List<string[]>(); 
     conecta(); 
     sql = "SELECT * FROM historico_verificacao_email WHERE nm_email = '" + email + "' ORDER BY dt_verificacao_email DESC, hr_verificacao_email DESC"; 
     com = new SqlCommand(sql, conexao); 
     SqlDataReader dr = com.ExecuteReader(); 

     if (dr.HasRows) 
     { 
      while (dr.Read()) 
      { 
       string[] dados_historico = new string[6]; 
       dados_historico[0] = dr["nm_email"].ToString(); 
       dados_historico[1] = dr["dt_verificacao_email"].ToString(); 
       dados_historico[1] = dados_historico[1].Substring(0, 10); 
       //System.Windows.Forms.MessageBox.Show(dados_historico[1]); 
       dados_historico[2] = dr["hr_verificacao_email"].ToString(); 
       dados_historico[3] = dr["ds_tipo_verificacao"].ToString(); 
       dados_historico[5] = dr["cd_login_usuario"].ToString(); 
       historicos.Add(dados_historico); 
      } 

      dr.Close(); 

      sql = "SELECT COUNT(e.cd_historico_verificacao_email) QT FROM emails_lidos e WHERE e.cd_historico_verificacao_email = '" + dr["cd_historico_verificacao_email"].ToString() + "'"; 
      tipo_sql = "seleção"; 
      com2 = new SqlCommand(sql, conexao); 

      for(int i = 0 ; i < historicos.Count() ; i++) 
      { 
       SqlDataReader dr3 = com2.ExecuteReader(); 
       while (dr3.Read()) 
       { 
        historicos[i][4] = dr3["QT"].ToString(); //quantidade de emails lidos naquela verificação 
       } 
       dr3.Close(); 
      } 

     } 

     return historicos; 

Answer 3

1. Оптимальное решение может быть, чтобы попытаться превратить ваше решение в форму, где вам не нужно иметь два считывателя открытых одновременно. В идеале это может быть один запрос. У меня нет времени на это.

2. Если ваша проблема настолько особенная, что вам действительно нужно открывать больше читателей одновременно, а ваши требования позволяют не раньше, чем бэкенд базы данных SQL Server 2005, тогда магическое слово: MARS (несколько активных наборов результатов). http://msdn.microsoft.com/en-us/library/ms345109%28v=SQL.90%29.aspx. Решение связанного раздела Bob Vale показывает, как включить его: укажите в строке подключения MultipleActiveResultSets=true. Я просто говорю это как интересную возможность, но вы должны скорее преобразовать свое решение.

   • , чтобы избежать упомянутой возможности SQL-инъекции, задайте параметры самому SQLCommand, а не вставляйте их в строку запроса. Строка запроса должна содержать только ссылки на параметры, которые вы передаете в SqlCommand.

Answer 4

Попытка объединить запрос, он будет работать намного быстрее, чем выполнение дополнительного запроса в строке. Ik не нравится строка [], которую вы используете, я бы создал класс для хранения информации.

public List<string[]> get_dados_historico_verificacao_email_WEB(string email) 
    { 
     List<string[]> historicos = new List<string[]>(); 

     using (SqlConnection conexao = new SqlConnection("ConnectionString")) 
     { 
      string sql = 
       @"SELECT *, 
          ( SELECT  COUNT(e.cd_historico_verificacao_email) 
           FROM  emails_lidos e 
           WHERE  e.cd_historico_verificacao_email = a.nm_email) QT 
        FROM  historico_verificacao_email a 
        WHERE  nm_email = @email 
        ORDER BY dt_verificacao_email DESC, 
          hr_verificacao_email DESC"; 

      using (SqlCommand com = new SqlCommand(sql, conexao)) 
      { 
       com.Parameters.Add("email", SqlDbType.VarChar).Value = email; 

       SqlDataReader dr = com.ExecuteReader(); 

       while (dr.Read()) 
       { 
        string[] dados_historico = new string[6]; 
        dados_historico[0] = dr["nm_email"].ToString(); 
        dados_historico[1] = dr["dt_verificacao_email"].ToString(); 
        dados_historico[1] = dados_historico[1].Substring(0, 10); 
        //System.Windows.Forms.MessageBox.Show(dados_historico[1]); 
        dados_historico[2] = dr["hr_verificacao_email"].ToString(); 
        dados_historico[3] = dr["ds_tipo_verificacao"].ToString(); 
        dados_historico[4] = dr["QT"].ToString(); 
        dados_historico[5] = dr["cd_login_usuario"].ToString(); 

        historicos.Add(dados_historico); 
       } 
      } 
     } 
     return historicos; 
    } 

Непрошеный, но maybee дает некоторую идею.

Answer 5

У вас может возникнуть такая проблема, когда вы используете two different commands при одном и том же соединении - особенно при вызове второй команды в loop. Это вызывает вторую команду для каждой записи, возвращенной из первой команды.Если есть 10 000 записей, возвращенных первой командой, эта проблема будет более вероятной.

Я использовал, чтобы избежать такого сценария, сделав его как единую команду. Первая команда возвращает все необходимые данные и загружает их в DataTable.

Примечание: MARS может быть решением, но это может быть опасно, и многим это не нравится.

Ссылка

1. What does "A severe error occurred on the current command. The results, if any, should be discarded." SQL Azure error mean?
2. Linq-To-Sql and MARS woes - A severe error occurred on the current command. The results, if any, should be discarded
3. Complex GROUP BY on DataTable

Answer 6

Просто добавьте следующую строку в вашей строке соединения:

MultipleActiveResultSets=True; 

Answer 7

Добавить MultipleActiveResultSets=true в часть поставщика вашей строки подключения. См. Пример ниже:

<add name="DbContext" connectionString="Data Source=(LocalDb)\v11.0;Initial Catalog=dbName;Persist Security Info=True;User ID=userName;Password=password;MultipleActiveResultSets=True" providerName="System.Data.SqlClient" />