Всегда ли нужно вызывать EndExecuteReader?

Question

[isxaker]

При открытии формы (.net4, WinForm app, один SqlConnection на все приложение) в обработчике события Load происходит длительный запрос к БД, который выполняется асинхронно. Если пользователю надоест ждать, он жмет на крестик, форма закрывается. Как в это случае очистить SqlDaraReader?

public static SqlDataReader SafeExecuteReader(SqlCommand command, AutoResetEvent evtCancel)
{
       IAsyncResult res = command.BeginExecuteReader();
       Int32 index = WaitHandle.WaitAny(new WaitHandle[] { res.AsyncWaitHandle, evtCancel }, command.CommandTimeout * 1000);
       if (index == 1 && !res.IsCompleted)
            {
                try
                {
                    //например так?
                    //SqlDataReader reader = command.EndExecuteReader(res);
                    //if (reader != null) reader.Close();
                }
                catch (SqlException /*sex*/) { }
                return null;
            }
            else if (!res.IsCompleted)
            {
                //command.EndExecuteReader(res);
                throw new Exception("Timeout");
            }
            return command.EndExecuteReader(res);
}

Или EndExecuteReader необходим только для завершения асинхронного выполнения и возвращения запрошенного SqlDataReader?

Answer 1

[aush]

Советую вам изучить паттерны асинхронного программирования, сейчас у вас каша. Можете начать отсюда msdn.microsoft.com/en-us/library/jj152938(v=vs.110...

BeginExecuteReader и EndExecuteReader реализуют устаревшую на данный момент модель Asynchronous Programming Model (APM).

SqlDataReader реализует IDisposable, поэтому, если операция успешно завершилась, вам следует вызвать у него Dispose(), когда вы закончите с ним работать.

Comments

[isxaker]

aush, спасибо за ссылку. А если операция завершится не успешно, что тогда делать?

[isxaker]

@aush т.к. используется .net 4, поэтому и устаревшее

[aush]

@isxaker что значит "не успешно"? Если EndExecuteReader вернет SqlDataReader, то его стоит задиспозить, если не вернет, то диспозить нечего. Вообще, воспользуйтесь перегрузкой BeginExecuteReader(AsyncCallback, Object), с ней, возможно, вам будет проще уложить в голове, что вы пытаетесь сделать.

[isxaker]

@aush не успешно значит, что выполнение прервано(или отменено пользователем) и форма, которая подгружала данные закрывается

[aush]

@isxaker в этом случае у вас нет объекта SqlDataReader, который можно было бы задиспозить.