[英]Is the database connection in this class “reusable”?
我是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中完成了数百次,但是现在,有了类和东西。 我完全迷失了。
您遇到的问题是您已将自己编码为“每个操作的新连接”角落。 您真正想要的目标是“每批操作的新连接”,并被认为是最佳实践。
在这种情况下,我建议的是在需要时打开连接,在处理时关闭。 我们要做的是将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方法中为您关闭该连接。
我改变的事情:
编辑:
是的,您发布的代码每次调用方法时都会创建一个新的数据库连接,但这不是问题。 问题是它没有正确处理连接。 处理这样的事情的方法如下:
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();
}
这几乎就是这个想法。
您应该让连接池处理您的连接; 这意味着你尽快Close()
每个连接,并且只在最后一刻创建一个新连接。
所以是 - 继续为单独的交易创建新的。
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.