已有打开的与此命令相关联的 DataReader，必须首先将它关闭。

[che2piaopiao]

1 ExecuteReader 要求已打开且可用的连接。连接的当前状态为正在连接。
2 已有打开的与此命令相关联的 DataReader，必须首先将它关闭。

经常报这2个错误。大家帮我看看是不是代码哪里有问题。。

 private static SqlConnection connection;
        public static SqlConnection Connection
        {
            get
            {
                string connectionString = ConfigurationManager.ConnectionStrings["ConnectionString"].ConnectionString;
                if (connection == null)
                {
                    connection = new SqlConnection(connectionString);
                    connection.Open();
                }
                else if (connection.State == System.Data.ConnectionState.Closed)
                {
                    connection.Open();
                }
                else if (connection.State == System.Data.ConnectionState.Broken)
                {
                    connection.Close();
                    connection.Open();
                }
                return connection;
            }
        }

//其它的就是调用CONNECTION的了。下面的应该没问题
    public static int ExecuteCommand(string safeSql)
        {
            SqlCommand cmd = new SqlCommand(safeSql, Connection);
            int result = cmd.ExecuteNonQuery();
            return result;
        }




        public static int ExecuteCommand(string sql, params SqlParameter[] values)
        {
            SqlCommand cmd = new SqlCommand(sql, Connection);
            cmd.Parameters.AddRange(values);
            return cmd.ExecuteNonQuery();
        }

        public static int GetScalar(string safeSql)
        {
            SqlCommand cmd = new SqlCommand(safeSql, Connection);
            int result = Convert.ToInt32(cmd.ExecuteScalar());
            return result;
        }

[QQlvbo]

建议楼主使用using(SqlConnection conn = new SqlConnection)这样在using结束时就会自动关闭当前的数据库连接了。

[tcmakebest]

不要共用 connection，需要用的时候就 new 一个。 不要一早就打开 connection，需要打开的时候才打开，及时并且要确保 finally {关闭}。

[金色海洋]

习惯不对，连接那能一直open呀。用的时候在open，用完了立刻close

[bdmh]

搞那么多 connection.Open 干嘛，保证只有一个地方open

[che2piaopiao]

引用 2 楼 tcmakebest 的回复:

不要共用 connection，需要用的时候就 new 一个。 不要一早就打开 connection，需要打开的时候才打开，及时并且要确保 finally {关闭}。

能否写个例子我看看

[宝_爸]

类似下面这样，不需要自己写个static Connection. ADO.net的Connection有连接池的，connection创建时先从链接池中找connection string一样的Connection对象。 public static int ExecuteCommand(string safeSql) { using (Connection con = new Connection(ConfigurationManager.ConnectionStrings["ConnectionString"].ConnectionString)) { SqlCommand cmd = new SqlCommand(safeSql, con); int result = cmd.ExecuteNonQuery(); return result; } }