我有一个数据访问类的基类。这个类实现了IDisposable。此基类包含IDbConnection并在构造函数中实例化它。
public class DALBase : IDisposable
{
protected IDbConnection cn;
public DALBase()
{
cn = new MySqlConnection(connString);
}
public void Dispose()
{
if (cn != null)
{
if (cn.State != ConnectionState.Closed)
{
try
{
cn.Close();
}
catch
{
}
}
cn.Dispose();
}
}
}
从此类继承的类实际访问数据库:
public class FooDAL : DALBase
{
public int CreateFoo()
{
// Notice that the cmd here is not wrapped in a using or try-finally.
IDbCommand cmd = CreateCommand("create foo with sql", cn);
Open();
int ident = int.Parse(cmd.ExecuteScalar().ToString());
Close();
cmd.Dispose();
return ident;
}
}
使用FooDAL的类使用using模式来确保在FooDAL上使用以下代码调用Dispose:
using(FooDAL dal = new FooDAL())
{
return dal.CreateFoo();
}
我的问题是,这是否也确保IDbCommand被正确处理掉,即使它没有包含在使用模式或try-finally中?如果在执行命令期间发生异常会发生什么?
另外,出于性能原因,在CreateFoo中而不是在基类的构造函数中实例化连接会更好吗?
感谢任何帮助。
答案 0 :(得分:2)
假设连接是池,只需在CreateFOO方法中创建MySqlConnection(使用using块)。
不要打扰关闭它,因为它将在使用区块的末尾自动处理/关闭。
public int CreateFoo()
{
using (var cn = new MySqlConnection(connString))
{
// Notice that the cmd here is not wrapped in a using or try-finally.
using (IDbCommand cmd = CreateCommand("create foo with sql", cn))
{
cn.Open();
return int.Parse(cmd.ExecuteScalar().ToString());
}
}
}
答案 1 :(得分:0)
如果这一切都符合效率的要求,那么加速代码的最大变化就是避免在每个DbCommand上打开和关闭数据库连接对象。