Как правильно использовать SqlDataReader?

Question

У меня есть 2 метода, как показано ниже:

internal static SqlDataReader SelectData(string sql) 
{ 
    using (var sqlConnection = new SqlConnection(Constant.ConnectionString)) 
    { 
     sqlConnection.Open(); 
     var sqlCommand = new SqlCommand(sql, sqlConnection); 
     var dataReader = sqlCommand.ExecuteReader(); 
     return dataReader; 
    } 
} 

============

И используя этот метод, как:

var dataReader = SelectData(---some sql ---); 

private void AddData(dataReader) 
{ 
    while (dataReader.Read()) 
    { 
     Employee e = new Employee(); 
     e.FirstNamei = dataReader["Name"].ToString(); 
    } 

    dataReader.Close(); 
} 

Я знаю, что мы можем объединить этот два метода, но я смотрю на лучший способ написать это, ИЛИ это может вызвать некоторые проблемы?

Answer 1

Фактически вы фактически оставляете себя немного открытым. Вы действительно хотите, чтобы написать это:

using (SqlConnection cnn = new SqlConnection(cnnString)) 
using (SqlCommand cmd = new SqlCommand(sql, cnn)) 
{ 
    // use parameters in your SQL statement too, so you can do this 
    // and protect yourself from SQL injection, so for example 
    // SELECT * FROM table WHERE field1 = @parm1 
    cmd.Parameters.AddWithValue("@parm1", val1); 

    cnn.Open(); 
    using (SqlDataReader r = cmd.ExecuteReader()) 
    { 

    } 
} 

потому, что вам нужно сделать уверены эти объекты получить расположены. Далее, перейдя в этом направлении, вам не нужно dataReader.Close(). Он будет вызван, когда он автоматически будет удален оператором using.

Теперь оберните эту сборку инструкций внутри try...catch, и вы находитесь в бизнесе.

Answer 2

Пара вещей

1) Так как вы закрываете соединение на SelectData, dataReader должен взорвать на AddData, поскольку она требует открытого соединения

2) AddData не должны закрывать DataReader как он не открывал.

3) Может быть, вы прячетесь код, но я не вижу, что вы используете Employee экземпляр, созданный на AddData

Answer 3

Технически, первый метод будет правильным, если бы вы сделали

sqlCommand.ExecuteReader(CommandBehavior.CloseConnection); 

Тогда , ваш клиент закроет читателя, и ваше соединение также будет закрыто.

Второй пример также был бы правильным, если бы вы не закрыли его внутри. Нет никакого преступника, передающего читателю метод, чтобы его повторить. но его нужно контролировать, откуда он был создан. Как вы его открываете и распоряжаетесь - это другой вопрос.