c# - ExecuteReader 需要一个打开且可用的连接。连接的当前状态是 Connecting

Question

当尝试通过 ASP.NET 在线连接到 MSSQL 数据库时，当两个或更多人同时连接时，我将得到以下信息:

ExecuteReader requires an open and available Connection. The connection's current state is Connecting.

该站点在我的本地主机服务器上运行良好。

这是粗略的代码。

public Promotion retrievePromotion()
{
    int promotionID = 0;
    string promotionTitle = "";
    string promotionUrl = "";
    Promotion promotion = null;
    SqlOpenConnection();
    SqlCommand sql = SqlCommandConnection();

    sql.CommandText = "SELECT TOP 1 PromotionID, PromotionTitle, PromotionURL FROM Promotion";

    SqlDataReader dr = sql.ExecuteReader();
    while (dr.Read())
    {
        promotionID = DB2int(dr["PromotionID"]);
        promotionTitle = DB2string(dr["PromotionTitle"]);
        promotionUrl = DB2string(dr["PromotionURL"]);
        promotion = new Promotion(promotionID, promotionTitle, promotionUrl);
    }
    dr.Dispose();
    sql.Dispose();
    CloseConnection();
    return promotion;
}

我可以知道可能出了什么问题以及如何解决吗？

编辑:不要忘记，我的连接字符串和连接都是静态的。我相信这就是原因。请指教。

public static string conString = ConfigurationManager.ConnectionStrings["dbConnection"].ConnectionString;
public static SqlConnection conn = null;

Answer 1

很抱歉一开始只发表评论，但我几乎每天都发布类似的评论，因为许多人认为将 ADO.NET 功能封装到 DB 类中是明智的(10 年前我也是) .大多数情况下，他们决定使用静态/共享对象，因为它似乎比为任何操作创建新对象更快。

无论是在性能方面还是在故障安全方面，这都不是一个好主意。

不要偷猎连接池的领地

ADO.NET 在内部管理 ADO-NET Connection-Pool 中与 DBMS 的底层连接是有充分理由的。 :

In practice, most applications use only one or a few differentconfigurations for connections. This means that during applicationexecution, many identical connections will be repeatedly opened andclosed. To minimize the cost of opening connections, ADO.NET uses anoptimization technique called connection pooling.

Connection pooling reduces the number of times that new connectionsmust be opened. The pooler maintains ownership of the physicalconnection. It manages connections by keeping alive a set of activeconnections for each given connection configuration. Whenever a usercalls Open on a connection, the pooler looks for an availableconnection in the pool. If a pooled connection is available, itreturns it to the caller instead of opening a new connection. When theapplication calls Close on the connection, the pooler returns it tothe pooled set of active connections instead of closing it. Once theconnection is returned to the pool, it is ready to be reused on thenext Open call.

所以显然没有理由避免创建、打开或关闭连接，因为实际上根本没有创建、打开和关闭连接。这“仅”是连接池知道何时可以重用连接的标志。但这是一个非常重要的标志，因为如果连接“正在使用”(连接池假设)，则必须为 DBMS 打开一个新的物理连接，这是非常昂贵的。

因此，您没有获得任何性能提升，反而适得其反。如果达到指定的最大池大小(默认值为 100)，您甚至会遇到异常(打开的连接过多...)。因此，这不仅会极大地影响性能，而且会成为严重错误和(不使用事务)数据转储区域的来源。

如果您甚至使用静态连接，您就是在为每个试图访问该对象的线程创建一个锁。 ASP.NET 本质上是一个多线程环境。因此，这些锁很有可能充其量会导致性能问题。实际上迟早你会遇到许多不同的异常(比如你的ExecuteReader 需要一个开放且可用的连接)。

结论:

• 根本不要重复使用连接或任何 ADO.NET 对象。
• 不要将它们设为静态/共享(在 VB.NET 中)
• 总是在需要的地方创建、打开(在连接的情况下)、使用、关闭和处置它们(例如在方法中)
• 使用using-statement隐式处理和关闭(在连接的情况下)

这不仅适用于 Connections(尽管最引人注目)。每个对象实现 IDisposable应在 System.Data.SqlClient 命名空间中进行处理(最简单的 using-statement )。

以上所有内容都反对封装和重用所有对象的自定义 DB 类。这就是为什么我评论要丢弃它的原因。那只是一个问题来源。

---

编辑:这是您的retrievePromotion-方法的可能实现:

public Promotion retrievePromotion(int promotionID)
{
    Promotion promo = null;
    var connectionString = System.Configuration.ConfigurationManager.ConnectionStrings["MainConnStr"].ConnectionString;
    using (SqlConnection connection = new SqlConnection(connectionString))
    {
        var queryString = "SELECT PromotionID, PromotionTitle, PromotionURL FROM Promotion WHERE PromotionID=@PromotionID";
        using (var da = new SqlDataAdapter(queryString, connection))
        {
            // you could also use a SqlDataReader instead
            // note that a DataTable does not need to be disposed since it does not implement IDisposable
            var tblPromotion = new DataTable();
            // avoid SQL-Injection
            da.SelectCommand.Parameters.Add("@PromotionID", SqlDbType.Int);
            da.SelectCommand.Parameters["@PromotionID"].Value = promotionID;
            try
            {
                connection.Open(); // not necessarily needed in this case because DataAdapter.Fill does it otherwise 
                da.Fill(tblPromotion);
                if (tblPromotion.Rows.Count != 0)
                {
                    var promoRow = tblPromotion.Rows[0];
                    promo = new Promotion()
                    {
                        promotionID    = promotionID,
                        promotionTitle = promoRow.Field<String>("PromotionTitle"),
                        promotionUrl   = promoRow.Field<String>("PromotionURL")
                    };
                }
            }
            catch (Exception ex)
            {
                // log this exception or throw it up the StackTrace
                // we do not need a finally-block to close the connection since it will be closed implicitly in an using-statement
                throw;
            }
        }
    }
    return promo;
}