【发布时间】:2011-12-01 17:28:04
【问题描述】:
我有一个简单的数据库管理器类(一个比它的能力更宏大的名字):
class DbManager
{
private MySqlConnectionStringBuilder _connectionString;
public DbManager()
{
_connectionString = new MySqlConnectionStringBuilder();
_connectionString.UserID = Properties.Database.Default.Username;
_connectionString.Password = Properties.Database.Default.Password;
_connectionString.Server = Properties.Database.Default.Server;
_connectionString.Database = Properties.Database.Default.Schema;
_connectionString.MaximumPoolSize = 5;
}
public MySqlConnection GetConnection()
{
MySqlConnection con = new MySqlConnection(_connectionString.GetConnectionString(true));
con.Open();
return con;
}
}
然后我在其他地方有另一个类,它表示其中一个表中的记录,我像这样填充它:
class Contact
{
private void Populate(object contactID)
{
using (OleDbConnection con = DbManager.GetConnection())
{
string q = "SELECT FirstName, LastName FROM Contacts WHERE ContactID = ?";
using (OleDbCommand cmd = new OleDbCommand(q, con))
{
cmd.Parameters.AddWithValue("?", contactID);
using (OleDbDataReader reader = cmd.ExecuteReader())
{
if (reader.HasRows)
{
reader.Read();
this.FirstName = reader.GetString(0);
this.LastName = reader.GetString(1);
this.Address = new Address();
this.Address.Populate(ContactID)
}
}
}
}
}
}
class Address
{
private void Populate(object contactID)
{
using (OleDbConnection con = DbManager.GetConnection())
{
string q = "SELECT Address1 FROM Addresses WHERE ContactID = ?";
using (OleDbCommand cmd = new OleDbCommand(q, con))
{
cmd.Parameters.AddWithValue("?", contactID);
using (OleDbDataReader reader = cmd.ExecuteReader())
{
if (reader.HasRows)
{
reader.Read();
this.Address1 = reader.GetString(0);
}
}
}
}
}
}
现在我认为所有using 语句都会确保连接完成后返回到池中,以供下次使用,但我有一个循环可以创建数百个这样的联系人并填充它们,似乎连接没有被释放。
连接、命令和阅读器都在自己的using语句中。
【问题讨论】:
-
我认为这不在多线程应用程序中?这显然有可能一次尝试使用多个这些东西......这是您创建数据库连接的唯一地方吗?它们不可能泄漏到其他地方吗?
-
我想我一定对连接重用的建议感到困惑。它是多线程的:将同时调用多个
Populate,但我的印象是我不应该有一个共享的OleDbConnection,而是应该根据需要创建新的。 -
只是为了检查发生了什么,如果您在 using 块的末尾显式调用 con.Dispose() 会发生什么?
-
嗯,我认为这让我遇到了一个问题。
populate调用另一个子对象的填充。如果我把它注释掉,那很好。子对象的填充几乎与那个完全一样。 -
@Cylindric:我添加了一个答案来解释我认为正在发生的事情。使用游泳池是个好主意,但本质上你需要确保游泳池足够大以应付。 :)