Метод, включающий SQLConnection - хороший подход? - PullRequest
Новая
       3

Метод, включающий SQLConnection - хороший подход?

0 голосов
/ 21 июля 2010

Эй, у меня есть этот метод в моем коде: 

public static DataSet PrepareDataSet(some params)
{
    SqlConnection sqlConnection = new SqlConnection(ConfigurationSettings.AppSettings["ConnectionString"]);
    SqlDataAdapter adapter = new SqlDataAdapter(Utils.EscapeProcedureName(...), sqlConnection);
    adapter.SelectCommand.CommandType = CommandType.StoredProcedure;

    //do some stuff with the adapter using the params

    sqlConnection.Open();
    DataSet dataSet= new DataSet();
    adapter.Fill(dataSet);
    sqlConnection.Close();
    return dataSet;
}

Этот код вызывается со страницы aspx.cs. Является ли хорошим подходом иметь метод соединения SQL и адаптер внутри метода? Если нет, то как это можно изменить? Почему-то я думаю, что это плохо для тестирования, например ...

Спасибо за ваши идеи :)

1 Ответ

0 голосов
/ 21 июля 2010

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

Попробуйте сделать ваше приложение многоуровневым N-Tier или используйте MVC. Шаблон проектирования.

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

...