c# - 这个类中的数据库连接是 "reusable"吗？

Question

我是 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 中完成了数百次，但现在，使用类和东西。我完全迷路了。

Answer 1

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

在这种情况下，我建议在需要时打开连接，并在处理时关闭。我们要做的是将 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。

---