C#. DataAdapter нужно ли вызывать Dispose?

Question

[Йцу Йцуевич]

Нужно ли вызывать метод Dispose или закидывать в using?

adapter = new SqlDataAdapter(cmd);
// ... actions
adapter.Dispose();    // здесь нужно?

Просто проверял через декомпилятор и увидел там, что Sql, OleDb и Odbc не переопределяют метод Dispose.

Answer 1

[zvoice]

Используйте using конструкцию - никогда не ошибетесь - если нужен вызов Dispose - компилятор вызовет его за Вас.

Comments

[Йцу Йцуевич]

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

Answer 2

[Станислав Макаров]

Толк от вызова метода Dispose есть всегда, когда есть сам метод Dispose (а если быть точным - если реализован интерфейс System.IDisposable). Тут просто не ваша ответственность разбираться - нужно ли вызывать Dispose или нет. Если автор класса реализовал этот интерфейс, значит для чего-то это ему нужно. Как правило это нужно, если используются управляемые вручную ресурсы (например, открытые файлы), либо в объекте создаются другие объекты, которые также IDisposable, и они должны подчищаться при disposing-е основного объекта, их создавшего.
Если вы сталкивались с С++, то на мой взгляд Dispose - это и есть настоящий "деструктор" для объекта, который нужно вызывать самому (прямо или косвенно, с помощью using), чтобы корректно завершить жизненный цикл объекта. Так или иначе, вызывайте Dispose всегда, когда завершаете взаимодействие с объектом. Важно, что даже если СЕЙЧАС в реализации этого метода особо ничего не происходит, функционал может быть добавлен ПОЗДНЕЕ, в новой версии библиотеки которую вы используете. И тогда вам придется исправлять ваш код, чтобы избежать утечки ресурсов (незакрытые файлы или коннекции к БД часто гораздо бОльшая проблема, чем подтекающая память).

Comments

[Йцу Йцуевич]

спасибо за наставление. (С++ помню!).
Просто код становится менее читабельным когда в методе 3 юзинга, сами понимаете, и дабы избежать такого, думал не обязательно так делать.

Значит примерно такой код будет:

using (SqlConnection con = ...)
{
using (SqlCommand cmd = ...)
{
using (SqlReader reader = ...)
{
...
}
}

}

[Йцу Йцуевич]

пробелы не списались см. ниже

Answer 3

[Йцу Йцуевич]

Для Nipheris
Там с пробелами проблемы...

Вот так будет выглядеть код:

using (SqlConnection con = ...)
{
     using (SqlCommand cmd = ...)
     {
           using (SqlReader reader = ...)
           {
                     // ...
           }
     }
 }

Comments

[Станислав Макаров]

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

using (SqlConnection con = ...)
{
List students = QueryStudents(con);
}

а в функции QueryStudents уже создавать команду и ридер. Если подобных функций будет несколько, то все эти using-и уже не смотрятся излишними)