Блог

проще говоря, мой sqlservr.exe происходит ли утечка памяти, когда когда-либо моя служба использует ее. Итак, вопрос в том, где и почему! : (

Я инициализирую свое соединение как нули. и есть предложение finally, чтобы убедиться, что они закрыты… (я также пробовал .dispose в datareader, но не помогает).

Я пытался определить проблему больше дня. Все, что я могу сказать, это то, что это где-то здесь.

 private Int32 GetCount(String From, String Where)
{

    //Build SQL string from given parameters.
    String sql = "SET dateformat DMY SELECT COUNT(*) as Count FROM "   From   " WHERE "   Where;
    SqlDataReader dataCount = null;
    SqlCommand sqlCommCount = null;
    SqlConnection sqlConCount = null;
    try
    {
        sqlCommCount = new SqlCommand();
        sqlConCount = new SqlConnection();
        sqlCommCount.Connection = sqlConCount;
        sqlConCount.ConnectionString = "connectionstring";
        sqlCommCount.CommandText = sql;
        sqlConCount.Open();

        dataCount = sqlCommCount.ExecuteReader();
        while (dataCount.Read())
        {
            return Convert.ToInt32(dataCount["Count"]);
        }
        return 0;
    }
    finally
    {
        sqlConCount.Close();
        sqlCommCount.Dispose();
        if (dataCount != null)
            dataCount.Close();

    }
}
 

РЕШАЕМАЯ:

1. Утечки нет.
2. Я ненавижу Sqlserver за то, что он не сказал мне, что он кэширует память, которая не используется при подключении, поэтому просто увеличивается и увеличивается (пока это не понадобится).

Реорганизуйте свой код с помощью этого ключевого слова: используя

Занятие памяти не является детерминированным, в вашем примере эффективная память, используемая исполняемым файлом, будет освобождена только при сборке мусора, поэтому вы можете увидеть увеличение памяти, но это может быть не проблемой, рано или поздно она будет собрана, и память вернется к правильному значению.

После просмотра комментария проблема, похоже, не в вашем коде C # (также, если используется ключевое слово using, это хорошая идея!), Если есть проблема…

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

В этом случае я предлагаю вам провести рефакторинг вашего кода, чтобы использовать хотя бы параметры SQL для создания предложения where, чтобы SQL всегда оставался неизменным, и изменялись только параметры, таким образом, SQL server будет кэшировать только один оператор SQL, а не тысячи. Еще лучше создать хранимую процедуру, но параметров обычно достаточно.

Имейте также в виду, что способ, которым вы создаете свой оператор SQL, действительно небезопасен и может привести к атакам с использованием SQL-инъекций, использование параметров также исправит это.

С уважением

Massimo

ваш Conenction sqlConCount никогда не утилизируется.

реализация Dispose SqlConnection закрывает соединение перед удалением. поэтому лучше всего было бы использовать команду using для подключения, и вы можете быть уверены, что проблема не возникнет.

но поскольку вы не сохраняете ссылку, она в любом случае должна быть собрана из мусора, так что на самом деле это не проблема. вы на 100% уверены, что утечка памяти здесь?

Похоже, вы возвращаете свое целое число, прежде чем, наконец, у блока появится шанс сработать. Попробуйте назначить его в блоке try и вернуть его после блока finally

 int value = o;
try
{
  ... 
  value = Convert.ToInt32(dataCount["Count"]);
}
finally
{
//close connection
}
return value;
 

я думаю string string , что вызывает утечку, swtich в StringBuilder. кроме того, вы не SqlConCount удаляете и dataCount (вы могли бы попытаться реорганизовать свой код с помощью using