Amrender Saini Ответов: 3

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

С этой командой уже связан открытый DataReader, который должен быть закрыт в первую очередь.


<%@ WebHandler Language="C#" Class="Handler" %>


using System;
using System.Web;
using System.Web.Services;
using System.Data.SqlClient;
using System.Data;

 using connectionclass;

    public class Handler : IHttpHandler
    {

        public void ProcessRequest(HttpContext context)
        {

            connection con = new connection();
            con.getconnectionstring();       
            string imageid = context.Request.QueryString["ImID"];
            SqlCommand command = new SqlCommand("select Images from Upload_images where Image_ID=" + imageid, connection.con);
            SqlDataReader dr = command.ExecuteReader();
            dr.Read();
            context.Response.BinaryWrite((Byte[])dr[0]);
            context.Response.End();
            dr.Close();
            connection.con.Close();
        }


        public bool IsReusable
        {
            get
            {
                return false;
            }
        }
    }

3 Ответов

Рейтинг:
28

Qadri Jillani

Старайтесь избегать использования таких ридеров:

SqlConnection connection = new SqlConnection("connection string");
SqlCommand cmd = new SqlCommand("SELECT * FROM SomeTable", connection);
SqlDataReader reader = cmd.ExecuteReader();
connection.Open();
if (reader != null)
{
      while (reader.Read())
      {
              //do something
      }
}
reader.Close(); // <- too easy to forget
reader.Dispose(); // <- too easy to forget
connection.Close(); // <- too easy to forget

Вместо этого оберните их в операторы using:
using(SqlConnection connection = new SqlConnection("connection string"))
{

    connection.Open();

    using(SqlCommand cmd = new SqlCommand("SELECT * FROM SomeTable", connection))
    {
        using (SqlDataReader reader = cmd.ExecuteReader())
        {
            if (reader != null)
            {
                while (reader.Read())
                {
                    //do something
                }
            }
        } // reader closed and disposed up here

    } // command disposed here

} //connection closed and disposed here


Оператор using обеспечит правильное распоряжение объектом и высвобождение ресурсов.

Если вы забыли, то вы оставляете уборку на сборщик мусора, что может занять некоторое время

Рейтинг:
1

thanh_bkhn

Может быть, в следующей ссылке, может помочь вам
http://stackoverflow.com/questions/15875837/when-putting-multiple-sql-commands-showing-error-there-is-already-an-open-datar[^]

Рейтинг:
0

Sushil Mate

Закрытие объекта DataReader &амп; затем объект sqlconnection, если существует исключение, когда она закончилась.

рекомендации по написанию этого кода в блоке try catch finally block.

в последнем блоке закройте объекты.

Что-то вроде этого.


SqlConnection conn = null;
SqlCommand cmd = null;

try
{
    conn = new SqlConnection(Settings.Default.qlsdat_extensionsConnectionString)
    cmd = new SqlCommand(reportDataSource, conn);
    cmd.CommandType = CommandType.StoredProcedure;
    cmd.Parameters.Add("@Year", SqlDbType.Char, 4).Value = year;
    cmd.Parameters.Add("@startDate", SqlDbType.DateTime).Value = start;
    cmd.Parameters.Add("@endDate", SqlDbType.DateTime).Value = end;

        conn.Open(); //opens connection

    DataSet dset = new DataSet();
    new SqlDataAdapter(cmd).Fill(dset);
    this.gridDataSource.DataSource = dset.Tables[0];
}
catch(Exception ex)
{
    Logger.Log(ex);
    throw;
}
finally
{
    if(conn != null)
        conn.Dispose();

        if(cmd != null)
        cmd.Dispose();
}