对象未沿所有执行路径放置

Question

我有以下代码。

private DataTable LoadSMSCellProviders()
{
    string sqlQuery = "Select * from SMSAddress";
    DataTable dt = new DataTable();
    using (SqlConnection conn = new SqlConnection(Utility.ConnString))
    {
        using (SqlCommand command = new SqlCommand(sqlQuery, conn))
        {
            SqlDataAdapter adapter = new SqlDataAdapter(command);
            adapter.Fill(dt);
            return dt;
        }
    }
}

Microsoft 代码分析告诉我，dt 并未沿所有执行路径进行处理，但我不确定如何更正此问题。如果我尝试在return 之前调用dispose，它将返回一个空值，如果我尝试在方法结束时执行此操作，则代码永远不会到达...

我在这里错过了什么？

这是来自分析工具的消息：

警告：CA2000：Microsoft.Reliability：在方法“x.x()”中，对象 'dt' 不会沿所有异常路径进行处理。称呼 System.IDisposable.Dispose 在对象“dt”上的所有引用之前 超出范围。

Answer 1

您需要在发生异常时处理它。像这样。

private DataTable LoadSMSCellProviders()
{
    string sqlQuery = "Select * from SMSAddress";
    DataTable dt = null;
    try
    {
        dt = new DataTable();
        using (SqlConnection conn = new SqlConnection(Utility.ConnString))
        {
            using (SqlCommand command = new SqlCommand(sqlQuery, conn))
            {
                SqlDataAdapter adapter = new SqlDataAdapter(command);
                adapter.Fill(dt);
                return dt;
            }
        }
    }
    catch
    {
        if(dt != null)
            dt.Dispose();
        throw;
    }
}

这个想法是，如果发生异常，则无法处理 DataTable，因为它不会被传递回调用者。所以，这就是让代码分析愉快的模式。

Answer 2

启动工具。 DataTables 不需要被释放。它们是 IDisposable，因为它们继承了 IComponent，但它们的 Dispose() 方法什么也不做。我觉得很恶心，MS自己的工具不知道这一点。

Answer 3

代码分析警告您该对象可能不会被释放。正如@juharr 正确建议的那样，您应该中和发生异常的代码路径，否则该对象将不会被返回并且不会被显式处理。这将消除警告。

Answer 4

检查@juharr 的答案，并且在您使用数据表之后，您不再需要它了

using(dt){}

它会为你处理它