ExecuteReader需要一个开放且可用的连接。连接的当前状态为“正在连接”


114

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

ExecuteReader需要一个开放且可用的连接。连接的当前状态为“正在连接”。

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

这是粗糙的代码。

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;

24
不要在像ASP.NET这样的多线程环境中使用共享/静态连接,因为这会生成锁或异常(打开的连接过多等)。将DB-Class扔到垃圾桶中,并在需要它们的地方创建,打开,使用,关闭,放置ado.net对象。还要看看使用语句。
Tim Schmelter 2012年

2
您能给我有关SqlOpenConnection();和sql.ExecuteReader();的详细信息吗?功能?..
ankit rajput 2012年

私有无效SqlOpenConnection(){试试{conn = new SqlConnection(); conn.ConnectionString = conString; conn.Open(); } catch(SqlException ex){抛出ex; }}
郭芳林

@GuoHongLim:我忘了提一句,即使是静态conString也不会增加性能,因为还是默认情况下缓存的(作为当前应用程序的每个配置值)。
蒂姆·施密特

...并且只是为了使它变得未知:确保读者也获得数据库事务处理/正确的工作单元,这是读者的一项练习。
mwardm '16

Answers:


225

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

就性能或故障安全而言,这既不是一个好主意。

不要在Connection-Pool的领土上偷猎

ADO.NET内部管理ADO-NET Connection-Pool中与DBMS的基础连接是有充分的理由的:

实际上,大多数应用程序仅使用一种或几种不同的配置进行连接。这意味着在应用程序执行期间,许多相同的连接将被反复打开和关闭。为了最小化打开连接的成本,ADO.NET使用了一种称为连接池的优化技术。

连接池减少了必须打开新连接的次数。池管理者维护物理连接的所有权。它通过为每个给定的连接配置保留一组活动的连接来管理连接。每当用户在连接上调用“打开”时,池管理器就会在池中寻找可用的连接。如果池化连接可用,它将把它返回给调用者,而不是打开一个新连接。当应用程序在连接上调用“关闭”时,池化程序将其返回到活动连接的池化集中,而不是将其关闭。一旦将连接返回到池,就可以在下一个Open调用中重用该连接。

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

因此,您没有获得任何性能改进,反而相反。如果达到指定的最大池大小(默认为100),您甚至会收到异常(打开的连接过多...)。因此,这不仅会极大地影响性能,而且还会成为令人讨厌的错误和(不使用事务处理)数据转储区域的来源。

如果您甚至使用静态连接,那么您都将为尝试访问该对象的每个线程创建一个锁。ASP.NET本质上是一个多线程环境。因此,这些锁极有可能导致性能问题。实际上,迟早您会收到许多不同的异常(例如ExecuteReader需要一个开放且可用的Connection)。

结论

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

这不仅适用于Connections(尽管最值得注意)。每个实现的对象IDisposable都应using-statementSystem.Data.SqlClient命名空间中处置(由简化)。

以上所有都是针对自定义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 implicitely in an using-statement
                throw;
            }
        }
    }
    return promo;
}

这对于提供连接工作范式非常有用。感谢您的解释。
aminvincent '18

写得很好,是许多人偶然发现的解释,我希望更多的人知道这一点。(+1)
安德鲁·希尔

1
谢谢您,先生,我认为这是我读过的关于这个主题的最好的解释,这是非常重要的,很多新手都会犯错。我必须赞扬你出色的写作能力。
Sasinosoft

@Tim Schmelter如何使用您建议的方法使在不同线程上运行的查询利用单个事务进行提交/回退?
geeko

1

我几天前发现了这个错误。

就我而言,这是因为我在Singleton上使用了Transaction。

如上所述,.Net与Singleton不能很好地配合使用。

我的解决方案是这样的:

public class DbHelper : DbHelperCore
{
    public DbHelper()
    {
        Connection = null;
        Transaction = null;
    }

    public static DbHelper instance
    {
        get
        {
            if (HttpContext.Current is null)
                return new DbHelper();
            else if (HttpContext.Current.Items["dbh"] == null)
                HttpContext.Current.Items["dbh"] = new DbHelper();

            return (DbHelper)HttpContext.Current.Items["dbh"];
        }
    }

    public override void BeginTransaction()
    {
        Connection = new SqlConnection(Entity.Connection.getCon);
        if (Connection.State == System.Data.ConnectionState.Closed)
            Connection.Open();
        Transaction = Connection.BeginTransaction();
    }
}

我将HttpContext.Current.Items用于实例。这个类DbHelper和DbHelperCore是我自己的类

By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.