Подключение к базе данных из другого класса

Question

Я создал класс в своей программе для обработки соединения с базой данных. Этот класс включает метод с именем OpenConnection(), чтобы открыть соединение с базой данных. Я не уверен, что моя программа соответствует стандарту чистого кода. Вот этот метод.

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

Этот метод работает хорошо, но я просто хочу, чтобы убедиться, если это безопасный способ, и я не эксплуатируя свою программу в любом случае. Спасибо заранее

Update

Я также добавил следующие методы в классе, чтобы закрыть соединение и утилизировать его.

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

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

Answer 1

У вас есть возможная утечка ресурсов из-за того, что OracleConnection реализует IDisposable. Кроме того, вызов close в соединении в ConnectionState.Executing или Fetching может быть плохим, поскольку он откатит все незафиксированные транзакции.

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; 
    } 
} 

Answer 2

Вы можете использовать предложение use, и оно будет обрабатывать 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 3

Ваше решение, как кодируются кажется, нормально, если добавить случаи предложенные в this answer и , если он предназначен для использования в одном потоке и в очень ограниченном объеме. Это говорит, что кажется, как вы готовитесь использовать этот класс в качестве параметра во многих вызовы методы, потому что вы хотите бизнес-логику

• смеси и упорство относится
• или разделить экземпляр таким образом, что другие методы не нужно беспокоиться о том, следует ли открывать соединение (например, более высокий вызов в стеке вызовов еще не вызвал Open) или нет (например, предыдущий вызов в стеке вызовов сделал, чтобы открыть соединение (что бы мы вызовите «окружающее» соединение)).

Любая из этих стратегий обычно приводит к проблемам. Это лучше, чтобы область применения небольшой, где вы знаете, что соединение открыто, и когда, чтобы закрыть его:

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

Если у вас есть этот небольшой объем, ваша абстракция в настоящее время не дает никакой ценности.