Правильный способ использования «использования» на моем коде

Question

Я просто хочу знать, правильно ли я использовал «использование» в моем CRUDStudentTable().

private string Query; 
public string message; 

public CRUD(string myQuery) 
{ 
    Query = myQuery; 
} 

public void CRUDStudentTable() 
    { 
      using (OleDbConnection conn = new OleDbConnection(@"Provider=Microsoft.ACE.OLEDB.12.0;Data Source=C:\Documents and Settings\Microsoft User\Desktop\Student.accdb")) 
      { 
       try 
       { 
        conn.Open(); 
        using (OleDbCommand cmd = new OleDbCommand(Query, conn)) 
        { 
         cmd.ExecuteNonQuery(); 
        } 
       } 
       catch (Exception exception) 
       { 
        message = exception.Message; 
       } 
       finally 
       { 
        conn.Close(); 
       } 
      } 
    } 

public string HasError() 
    { 
     if (string.IsNullOrEmpty(message)) 
     { 
      message = "Successful"; 
      return message; 
     } 
     else 
     { 
      return message; 
     } 
    } 

Если это не так, любезно дайте мне совет, как это сделать правильно. Спасибо.

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

Answer 1

Это своего рода подход «пояс и подтяжки»: вы должны использовать либо using, либо finally, но не оба.

Блок using гарантирует закрытие соединения во всех кодах, в том числе с исключениями. Поэтому вы можете переписать код следующим образом:

public void CRUDStudentTable() 
{ 
     using (OleDbConnection conn = new OleDbConnection(@"Provider=Microsoft.ACE.OLEDB.12.0;Data Source=C:\Documents and Settings\Microsoft User\Desktop\Student.accdb")) 
     { 
      try 
      { 
       conn.Open(); 
       using (OleDbCommand cmd = new OleDbCommand(Query, conn)) 
       { 
        cmd.ExecuteNonQuery(); 
       } 
      } 
      catch (Exception exception) 
      { 
       message = exception.Message; 
       // Consider re-throwing the exception here 
       // to let callers know what happened. 
       // Silently harvesting the message and continuing 
       // is not a good practice of handling exceptions. 
      } 
     } 
}