насчет паранойи согласен, вызов Close при условии использования using-а излишен. Но код Артема тоже грешит излишними исходными предпосылками, в конкретно этом простейшем случае этот код сработает:SqlConnection connection = new SqlConnection(connectionString);
try
{
 connection.Open();
 command.ExecuteScalar();
}
catch (Exception e) {}
finally
{
 connection.Close();
}

Однако в реальном приложении в блоке try излишне ретивый программист вполне может написать так:
connection.Close();
connection = null;


А что может твориться в блоке catch вообще представить страшно.
В результате мы получим исключение в блоке finally на ровном месте.
Идеальный вариант - оборачивать SqlConnection в using, это будет защищенный от ошибок код by design. Однако этот оператор есть не во всех языках, соответственно можно использовать и вариант с finally но перед вызовом методов обязательно проверять существование объекта, а для класса SqlConnection неплохо бы еще и проверить его текущий статус. В итоге код выглядеть будет как-то так:SqlConnection connection = new SqlConnection(connectionString);
try
{
    //...
    connection.Open();
    command.ExecuteScalar();
    //...
}
catch (Exception e) {}
finally
{
    if (connection != null
        && connection.State == ConnectionState.Open)
    {
        connection.Close();
    }
}

я поэтому и ответил не на вопрос автора, а на подветку

Ну во-первых, SqlConnection как раз наследует и замечательно реализует метод Dispose() интерфейса IDisposable, так что обернуть его в using(...) замечательно получится и трескать никого не придется. Для справки есть такой сайт MSDN, много интересного пишут http://msdn.microsoft.com/library/default.asp?url=/library/en-us/cpref/html/frlrfsystemdatasqlclientsqlconnectionmemberstopic.asp

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

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




Автор:
Spaider, 31.07.06 11:50

Тема:
Re[2]: Критика кода

Сообщение:
Здравствуйте, Nisus, Вы писали:

N>Идеальный вариант — оборачивать SqlConnection в using, это будет защищенный от ошибок код by design. Однако этот оператор есть не во всех языках, соответственно можно использовать и вариант с finally

Ну, во-первых, как уже было замечено, SqlConnection не реализует IDisposable, так что обернуть его в using (...) не получиться хоть ты тресни.
Во-вторых, using(...) -- это syntactical sugar для C#, в IL всё равно конструкция разворачивается в try{...}finally{...}