我是否需要关闭datareader

Question

我有这个简单的函数来处理所有的查询:

DataTable ReadIt(string query, params MySqlParameter[] sqlParams)
{
    DataTable dt = new DataTable();

    using(MySqlConnection myConnection = new MySqlConnection(sConnection))
    {
        myConnection.Open();
        MySqlCommand myCommand = myConnection.CreateCommand();
        myCommand.CommandText = query;
        foreach (var p in sqlParams)
        {
            myCommand.Parameters.Add(p);
        }
        MySqlDataReader myReader = myCommand.ExecuteReader(CommandBehavior.CloseConnection);
        dt.Load(myReader);
        myReader.Close();
    }

    return dt;
}

myReader.Close();将数据加载到数据表后是否需要放置或读取器自动关闭后using？

Answer 1

您不知道(或者至少您不应该知道)DataReader拥有哪些资源.
另一方面,应该处理每个一次性物体,因为在处理代码中,物体有机会尽快释放所使用的资源.
因此,您的MySqlDataReader也应遵循用于MySqlConnection以及MySqlCommand的相同模式.使用声明是你的朋友

using(MySqlConnection myConnection = new MySqlConnection(sConnection))
using(MySqlCommand myCommand = myConnection.CreateCommand())
{
    myConnection.Open();
    myCommand.CommandText = query;
    foreach (var p in sqlParams)
        myCommand.Parameters.Add(p);
    using(MySqlDataReader myReader = myCommand.ExecuteReader(CommandBehavior.CloseConnection))
    {
        dt.Load(myReader);
    }
}