c# - 在我的代码中使用 "using"的正确方法

Question

我只想知道在我的 CRUDStudentTable() 中使用“using”是否正确。

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

如果这不对，请给我一个如何正确执行此操作的建议。谢谢。

我在这里包含了另一个方法，只要它不为 null 就返回“消息”。

Answer 1

这是一种“腰带和吊带”方法:您应该使用 using 或 finally，但不能同时使用两者。

using block 保证关闭所有代码路径中的连接，包括有异常的代码路径。因此你可以重写代码如下:

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