Как, когда и нужно ли вообще закрывать connection к базе в singleton классе?

Question

[isxaker]

На все приложение должно существовать единственное подключение к базе.
Реализовано с помощью singletion паттерна.

public sealed class Connection
{
      private static Connection _instance = null;
      private Connection() { }

      public static Connection Instance
      {
          get
          {
              if (_instance == null) _instance = new Connection();
              return _instance;
          }
      }

      public SqlConnection GetSqlConnection()
      {
          return new SqlConnection(ConfigurationManager.ConnectionStrings["CancellationCS"].ToString());
      }
}

Как лучше делать:
-один раз открыть connection и не закрывать до завершения приложения? (как лучше закрывать connection при завершении приложения)
-каждый раз(т.е. при любом запросе к БД) открывать и закрывать connection?

Answer 1

[gleb_kudr]

В общем случае 1. Открытие коннекта - достаточно затратная процедура чтобы делать ее на каждый чих. Можете сами измерить скорость/нагрузку в тесте с открытием одного коннекта или множества коннектов.

Answer 2

[Дмитрий Андриянов]

Зависит от того, насколько часто приложение обращается к БД.
Если часто, то лучше использовать одно общее соединение. Если редко, то лучше закрывать, чтобы освободить ресурсы.

Например, в веб-приложениях, как правило, открывается отдельное соединение на каждый HTTP-запрос и в нем выполняются несколько SQL запросов.

Также почитайте про connection pool.
Грубо говоря, это способ оптимизации приложения, при котором есть некоторый пулл соединений. При открытии соединения, если такое уже есть в пулле, оно берется оттуда (соответственно, не тратятся ресурсы на его инициализацию). При закрытии соединения оно возвращается в пулл и может быть использовано повторно. Инфраструктура .NET Framework управляет пуллом соединений в соответствии с указанными Вами настройками.

Answer 3

[Mephistophele]

Стоит начать с вопроса: это десктоп приложение или веб?
Если первое, то лучше что-то типа:

class StaticClass 
{
   static StaticClass() {
       AppDomain.CurrentDomain.ProcessExit +=
           StaticClass_Dtor;
   }

   static void StaticClass_Dtor(object sender, EventArgs e) {
        // clean it up
   }
}

Если веб, то выпиливайте ваш класс к чертям. :)

ЗЫ: а вообще использование синглтонов для работы с БД - зло. Лучше юзать какой-нибудь ORM, тот же Entity Framework альбо NHibernate.

Comments

[isxaker]

@Mephistophele у меня desktop app.

[isxaker]

@Mephistophele например
static void StaticClass_Dtor(object sender, EventArgs e) {
myConnection.Close();
myConnection.Dispose()
}
???
Я думаю что конекшен и так закроется, убьется и задиспоузится автоматически сборщиком мусора. Как вы думаете? Как ведет себя .net, если конекшен не закрыть а приложение закрыть?

[isxaker]

или физическое соединение с базой останется существовать?

[Mephistophele]

Зависит от самой БД. Например у Оракла останется висеть. У MS SQL Server, будет закрыто по истечении некоторого времени (таймаут выставлется при настройке). Да и как бы не очень хорошо оставлять за собой не подчищенные ресурсы, лучше всё-таки подчищать за собой. Правило хорошего тона, так сказать.