Нужно ли мне закрыть 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(); после загрузки данных в DataTable или же читатель закрывается автоматически после того, как Do 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); 
}