此类中的数据库连接是否“可重用”?

时间:2011-09-21 00:25:38

标签: c# asp.net mysql database database-connection

我是asp.net的新手,所以这可能是一个非常基本的问题,但我无法弄明白。

我在互联网上找到了一些连接到数据库的代码。我创建了一个名称空间和一些类,以便在不同的项目中使用相同的代码。

代码和我的课程如下:

namespace databaseFunctions
{
    public class databaseConnection
    {
private static string databaseConnectionString()
        {
            return "DRIVER={MySQL ODBC 5.1 Driver}; ........";
        }

        public static DataTable getFromDatabase(string SQL)
        {
            DataTable rt = new DataTable();
            DataSet ds = new DataSet();
            OdbcDataAdapter da = new OdbcDataAdapter();
            OdbcConnection con = new OdbcConnection(databaseConnectionString());
            OdbcCommand cmd = new OdbcCommand(SQL, con);
            da.SelectCommand = cmd;
            da.Fill(ds);
            try
            {
                rt = ds.Tables[0];
            }
            catch
            {   
                rt = null;
            }
            return rt;
        }

        public static Boolean insertIntoDatabase(string SQL)
        {

            OdbcDataAdapter da = new OdbcDataAdapter();
            OdbcConnection con = new OdbcConnection(databaseConnectionString());
            OdbcCommand cmd = new OdbcCommand(SQL, con);
            con.Open();
            try
            {
                cmd.ExecuteNonQuery();
                return true;
            }
            catch
            {
                return false;
            }

        }

}

从数据库获取数据或将数据插入某个数据库没有问题。 但。当我尝试从mysql数据库中获取last_insert_id()时。我只得到零。

这就是为什么我认为我创建并从互联网上复制的这段代码,每次调用“getFromDatabase(SQL)”时都会创建一个新连接

是否有人可以帮我修复此类getFromDatabase()以保持数据库连接处于活动状态,直到我告诉程序放弃连接为止?

我想是应该更改的“新OdbcConnection”?是否可以检查是否已存在连接? 我已经在经典的asp中完成了数百次,但是现在,有了类和东西。我完全迷失了。

3 个答案:

答案 0 :(得分:10)

您遇到的问题是您已将自己编码为“每个操作的新连接”角落。您真正想要的目标是“每批操作的新连接”,并被认为是最佳实践。

在这种情况下,我建议在需要时打开连接,在处理时关闭。我们要做的是将odbc适配器移动到更大的范围变量,以便可以在类中访问它。

namespace databaseFunctions
{
    public class databaseConnection:IDisposable
    {
        private OdbcConnection con;
        private string connectionString;

        public databaseConnection(string connectionString){
            this.connectionString = connectionString;
        }


        public void OpenConnection(){
            if (con == null || con.IsClosed ){ // we make sure we're only opening connection once.
                con = new OdbcConnection(this.connectionString);
            }
        }
        public void CloseConnection(){
            if (con != null && con.IsOpen){ // I'm making stuff up here
                con.Close();
            }
        }

        public DataTable getFromDatabase(string SQL)
        {
            OpenConnection();

            DataTable rt = new DataTable();
            DataSet ds = new DataSet();
            OdbcCommand cmd = new OdbcCommand(SQL, con);
            da.SelectCommand = cmd;
            da.Fill(ds);
            try
            {
                rt = ds.Tables[0];
            }
            catch
            {   
                rt = null;
            }
            return rt;
        }

        public Boolean insertIntoDatabase(string SQL)
        {
            OpenConnection();

            OdbcCommand cmd = new OdbcCommand(SQL, con);
            con.Open();
            try
            {
                cmd.ExecuteNonQuery();
                return true;
            }
            catch
            {
                return false;
            }

        }


        // Implementing IDisposable method
        public void Dispose(){
            CloseConenction();
        }
    }
}

现在您下次使用课程时会执行类似

的操作
using (DatabaseConnection db = new DatabaseConnection()){
    db.InsertIntoDatabase(...);
    db.GetLastInsertID();
    db.GetFromDatabase(...);
}

在该代码块的末尾,因为它是IDisposeable,它将在dispose方法中为您关闭该连接。


我改变的事情:

  • 已实施IDisposable界面
  • 将方法从静态更改为类方法。
  • 添加了打开关闭连接的新方法
  • 将连接变量移至类级别范围
  • 在构造函数中添加了一个参数,允许您传入连接字符串(您应该将此连接字符串放在Web.Config中

编辑:

  • 构造函数根据建议接受connectionString。

答案 1 :(得分:4)

是的,您发布的代码每次调用方法时都会创建一个新的数据库连接,但这不是问题。问题是它没有正确处理连接。处理这类事情的方法如下:

using (OdbcConnection con = new OdbcConnection("yourconnectionsstring"))
{
   con.open();
   OdbcCommand command = new OdbcCommand("command_text",con);
   command.ExecuteQuery(); //or what ever you need to do
}

这样就可以正确处理连接,因为使用只是try/finally

的语法糖

您需要做的是在同一事务中执行2个sql语句,方法是在第一个sql语句中插入记录,并在结束事务之前检索下一个插入的最后一个插入的id。例如:

 using (OdbcConnection con = new OdbcConnection("yourconnectionsstring"))
    {
       con.open();
       OdbcTransaction tran = con.BeginTransaction()
       OdbcCommand command = new OdbcCommand("first_sql_statement_here",con);
       command.ExecuteNonQuery(); 
       command.CommandText = "select last_insert_id();";
       int result =command.ExecuteScalar(); 
       tran.commit();
    }

这就是这个想法。

答案 2 :(得分:1)

您应该让连接池处理您的连接;这意味着您尽快Close()每个连接,并且只在最后一刻创建一个新连接。

是的 - 继续为单独的交易创建新的。