Question

我在程序中创建了一个用于处理数据库连接的类。此类包含一个名为OpenConnection()的方法，用于打开与数据库的连接。我不相信我的程序符合清洁代码的标准。这是方法。

public void OpenConnection()
{
  if(connection==null || connection.State != Connection.Open)
  {
     connection = new OracleConnection(this.connectionString);
     connection.Open();
  }
}

这种方法运行正常，但我只是想确定这是否是一种安全的方式而且我没有以任何方式利用我的程序。提前谢谢

更新

我还在类中添加了以下方法来关闭连接并进行处理。

    public void CloseConnection()
    {
        if (dbconnect != null | dbconnect.State != ConnectionState.Closed)
        {
            dbconnect.Close();
        }
    }

    //Here the IDsiposable method is implemented
    public void Dispose()
    {
        CloseConnection();
    }

Answer 1

您可以使用using子句，它将自动处理Dispose。

using (SqlConnection connection = new SqlConnection(connectionString))
    {

    connection.Open();
    SqlCommand command = new SqlCommand("UpdateEmployeeTable", connection);
    command.CommandType = CommandType.StoredProcedure;
    command.Parameters.Add(new SqlParameter("@EmployeeID", 123));
    command.CommandTimeout = 5;

    command.ExecuteNonQuery();

    connection.close();
}

Answer 2

如果您添加this answer中建议的案例，并且如果要在单个线程上使用并且在非常有限的范围内，那么您编码的解决方案似乎没问题。这就是说你似乎正准备将这个类用作许多方法调用的参数，因为你想要

混合业务逻辑和持久性问题
或以这样的方式共享实例，使得其他方法不必关心是否打开连接（例如，调用堆栈中的较高调用尚未调用Open）（例如调用堆栈中的事先调用确实打开连接（我们称之为“环境”连接）。）

这些策略中的任何一种通常都会导致麻烦。最好保持范围较小，您知道连接是打开的以及何时关闭它：

using (var connection = new OracleConnection(...))
{
    connection.Open();
    ...
}

当你拥有这个小范围时，你的抽象现在没有任何价值。

Answer 3

由于OracleConnection实现了IDisposable，您可能会发生资源泄漏。此外，在ConnectionState.Executing或Fetching中调用close连接可能会很糟糕，因为它将回滚所有未提交的事务。

public void OpenConnection()
{
  if (connection == null)
  {
     connection = new OracleConnection(this.connectionString);
     connection.Open();
     return;
  }

  switch (connection.State)
  {
     case ConnectionState.Closed:
     case ConnectionState.Broken:
       connection.Close();
       connection.Dispose();
       connection = new OracleConnection(this.connectionString);
       connection.Open();
       return;
  }
}

来自其他类的数据库连接

3 个答案: