: System.InvalidOperationException: 已有打开的与此连接相关联的 DataReader，必须首先将它关闭。

[qxkls]

sqlConn = new SqlConnection(BP.DataAccess.Constants.ConnectString); string sql = "select count(name)as c,name from get_movie where datediff(dd,datet,getdate()) <7 and serialno = '"+name+"' group by name";
string sql1 = "insert into get_movie values('"+name+"','"+this.TextBox1.Text+"','"+DateTime.Now.ToString()+"','"+url+"','"+0+"','"+des+"')";
string sql2 = "select counts from hardinfo where serianlo = '"+name+"'";
cmd = new SqlCommand(sql,sqlConn);
cmd1 = new SqlCommand(sql1,sqlConn);
cmd2 = new SqlCommand(sql2,sqlConn);
SqlDataReader read = null;
SqlDataReader read2 = null;
sqlConn.Open();
read = cmd.ExecuteReader();
read2 = cmd2.ExecuteReader();
while(read.Read() && read2.Read())
{

int count = Convert.ToInt32(Convert.ToString(read.GetValue(0)));
int hard_info_count = Convert.ToInt32(read2.GetValue(0));
string names = Convert.ToString(read.GetValue(1));
if(count>hard_info_count)
{

Response.Write("<script>alert('请求失败,一周之内只能请求'"+hard_info_count+"'部影片!')</script>");
p =true;
return;
}
if(names == this.TextBox1.Text)
{
Response.Write("<script>alert('您已请求过此影片,请不要重复求片!')</script>");
p =true;
return;
}
}
sqlConn.Close();
if(p == false)
{
sqlConn.Open();
cmd1.ExecuteNonQuery();
sqlConn.Close();
Response.Write("<script>alert('影片信息已发出请等候!')</script>");
}






--------------


行 73： sqlConn.Open();
行 74： read = cmd.ExecuteReader();
行 75： read2 = cmd2.ExecuteReader();
行 76： while(read.Read() && read2.Read())
行 77： {

[Adechen]

应尽量早关闭连接，

[风骑士之怒]

MSDN:
在使用 SqlDataReader 时，关联的 SqlConnection 正忙于为 SqlDataReader 服务，对 SqlConnection 无法执行任何其他操作，只能将其关闭。除非调用 SqlDataReader 的 Close 方法，否则会一直处于此状态。例如，在调用 Close 之前，无法检索输出参数。

[wangzhenyue]

sqlConn = new SqlConnection(BP.DataAccess.Constants.ConnectString); string sql = "select count(name)as c,name from get_movie where datediff(dd,datet,getdate()) <7 and serialno = '"+name+"' group by name";
string sql1 = "insert into get_movie values('"+name+"','"+this.TextBox1.Text+"','"+DateTime.Now.ToString()+"','"+url+"','"+0+"','"+des+"')";
string sql2 = "select counts from hardinfo where serianlo = '"+name+"'";
cmd = new SqlCommand(sql,sqlConn);
cmd1 = new SqlCommand(sql1,sqlConn);
cmd2 = new SqlCommand(sql2,sqlConn);
SqlDataReader read = null;
SqlDataReader read2 = null;
sqlConn.Open();
read = cmd.ExecuteReader();
read2 = cmd2.ExecuteReader();
while(read.Read() && read2.Read())
{

int count = Convert.ToInt32(Convert.ToString(read.GetValue(0)));
int hard_info_count = Convert.ToInt32(read2.GetValue(0));
string names = Convert.ToString(read.GetValue(1));




//这里啊

read.Close();
read2.Close();




if(count>hard_info_count)
{

Response.Write(" <script>alert('请求失败,一周之内只能请求'"+hard_info_count+"'部影片!') </script>");
p =true;
return;
}
if(names == this.TextBox1.Text)
{
Response.Write(" <script>alert('您已请求过此影片,请不要重复求片!') </script>");
p =true;
return;
}
}
sqlConn.Close();
if(p == false)
{
sqlConn.Open();
cmd1.ExecuteNonQuery();
sqlConn.Close();
Response.Write(" <script>alert('影片信息已发出请等候!') </script>");
}

[macker0030]

行 73： sqlConn.Open();
行 74： read = cmd.ExecuteReader();
行 75： read2 = cmd2.ExecuteReader();
行 76： while(read.Read() && read2.Read())
行 77： {

两个DataReader不能共用一个打开的Connection的，你需要把read2使用另一个SqlConnection.
而且楼主请记住，使用SqlDataReader后，必须要及时进行关闭，例如：read.Close()

这段代码的质量的确不敢恭维，我也建议楼主起码学习一下sqlhelper,自己搜一下吧。

[jiang_jiajia10]

太乱了建议整理一下代码
查询的时候用dataset插入的时候用command对象。好好整理一下这样太乱了。
[/code]

[jiang_jiajia10]

[Quote=引用 14 楼 macker0030 的回复:]
行 73： sqlConn.Open();
行 74： read = cmd.ExecuteReader();
行 75： read2 = cmd2.ExecuteReader();
行 76： while(read.Read() && read2.Read())
行 77： {

两个DataReader不能共用一个打开的Connection的，你需要把read2使用另一个SqlConnection.
而且楼主请记住，使用SqlDataReader后，必须要及时进行关闭，例如：read.Close()

这段代码的质量的确不敢恭维，我也建议楼主起码学习一下sqlhelper,自己搜一…
[/Quote]
传说真的AJ么。LZ整理一下代码吧。

[jiang_jiajia10]

下午有活动就不给你整理了。自己好好整理一下我给你几个方法你封装到一个类里面。用的时候调用

//这个方法适合增删改。
 public Boolean Exect(string sQueryString) {

        SqlConnection conn = new SqlConnection(ConfigurationManager.AppSettings["conStr"]);
        conn.Open();
        SqlCommand cmd = new SqlCommand(sQueryString, conn);
        try
        {

            cmd.ExecuteNonQuery();
            conn.Close();
        }
        catch (System.Exception e) {

            conn.Close();
            return false;
        }
        return true;
    }
//这个方法适合查
    public System.Data.DataSet GetDataSet(string sQueryString, string TableName) {

        SqlConnection conn = new SqlConnection(ConfigurationManager.AppSettings["conStr"]);
        conn.Open();
        SqlDataAdapter dbAdapter = new SqlDataAdapter(sQueryString, conn);
        DataSet ds = new DataSet();
        dbAdapter.Fill(ds, TableName);
        conn.Close();
        return ds;
    }

[lovehongyun]

错误报的很明显.

要不你就分别指定连接或是把reader换成datatable

[HDNGO]

用sqlhelper吧。。。。

[HDNGO]

insert为啥用ExecuteReader？

[qinhl99]

只有把read和read2先关闭，才可以再使用与它关联的sqlConn!

[troy-zhou]

因为DataReader是只读的,所以必顶先把它关闭才能执行其他的操作!!!!!!!!

[qinhl99]

行 73： sqlConn.Open();
行 74： read = cmd.ExecuteReader();
行 75： read2 = cmd2.ExecuteReader();
行 76： while(read.Read() && read2.Read())
行 77： {

//哥们,SqlDataReader的实例对数据库连接是独占的,除非read关闭了,才能把sqlConn给read2用!

[qxkls]

我While(read.Read())
可以用
我While(read.Read()&&read2.Read())
就提示read2 = cmd2.ExecuteReader();出错了

[anncesky]

你实在是个初学者，还是不要用DataReader吧

用SqlDataAdapter,就这会有这样的问题

DataReader是独占的资源，两个一起打开是一行嘀，用完还要马上关闭

[HDNGO]

如果只是为了得到一个值做判断，用变量吧~

而且，count的话，一定会有一行的，不会为空的~

DataReader用完一定要关闭，要不然过一会儿数据库就挂了~

[webdiyer]

执行insert语句应该用ExecuteNonQuery()方法，而计算总记录数应该用ExecuteScalar()方法。

[criedshy]

没有关闭DataReader