我需要关闭数据读取器吗

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();吗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);
    }
}

Answer 2

如果没有更多的结果网格， DataTable.Load将自动关闭阅读器。 我的喜好有点错过。 我会使用using ：

using(var myReader = myCommand.ExecuteReader(CommandBehavior.CloseConnection)) {
    dt.Load(myReader);
}