Блог

У меня есть давно работающая служба с несколькими потоками, вызывающими следующий метод сотни раз в секунду:

 void TheMethod()
{
    using (var c = new SqlConnection("..."))
    {
        c.Open();

        var ret1 = PrepareAndExecuteStatement1(c, args1);
        // some code
        var ret2 = PrepareAndExecuteStatement2(c, args2);
        // more code
    }
}
  

PrepareAndExecuteStatement — это что-то вроде этого:

 void PrepareAndExecuteStatement*(SqlConnection c, args)
{
    var cmd = new SqlCommand("query", c);
    cmd.Parameters.Add("@param", type);
    cmd.Prepare();
    cmd.Parameters["@param"] = args;

    return cmd.execute().read().etc();
}
  

Я хочу повторно использовать подготовленные инструкции, подготавливая один раз для каждого соединения и выполняя их до разрыва соединения. Я надеюсь, что это улучшит производительность.

Могу ли я использовать встроенный пул подключений для достижения этой цели? В идеале каждый раз, когда устанавливается новое соединение, все инструкции должны подготавливаться автоматически, и мне нужно иметь доступ к объектам SqlCommand этих инструкций.

Предлагаю использовать слегка измененный подход. Немедленно закройте ваше соединение после использования. Вы, безусловно, можете повторно использовать свой SqlConnection.

Работа, выполняемая на //some code , может занять много времени. Взаимодействуете ли вы с другими сетевыми ресурсами, дисковыми ресурсами или тратите какое-либо количество времени на вычисления? Может ли вам когда-нибудь в будущем понадобиться это сделать? Возможно, интервалы между выполнением инструкции настолько велики, что вы захотите повторно открыть это соединение. В любом случае, соединение должно быть открыто позже и закрыто раньше.

 using (var c = new SqlConnection("..."))
{
    c.Open();
    PrepareAndExecuteStatement1(c, args);
    c.Close();
    // some code
    c.Open();
    PrepareAndExecuteStatement2(c, args);
    c.Close();
    // more code
}
  

Открывайте поздно, закрывайте рано как журнал MSDN Джона Папы.

Очевидно, что теперь мы имеем здесь кучу дублирующегося кода. Рассмотрите возможность рефакторинга вашего Prepare...() метода для выполнения операций открытия и закрытия.

Возможно, вы бы рассмотрели что-то вроде этого:

 using (var c = new SqlConnection("..."))
{
    var cmd1 = PrepareAndCreateCommand(c, args);

    // some code
    var cmd2 = PrepareAndCreateCommand(c, args);

    c.Open();
    cmd1.ExecuteNonQuery();
    cmd2.ExecuteNonQuery();
    c.Close();
    // more code
}