Класс для работы с базой SQL Server

@NatalieZ · Регистрация: 13.10.2014

Студворк — интернет-сервис помощи студентам

Добрый день. У меня вопрос к знающим людям. Пишу свой класс для работы с базой данных SQL Server в котором есть метод, который возвращает DataSet

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
        private static DataSet DataSetFill(string commandTextOrProcedureName, CommandType cmdType, params SqlParameter[] parameters)
        {
            int rowCount = 0;
            SqlConnection connection = null;
            SqlCommand command = null;
            DataSet dataSet = null;
            SqlDataAdapter dataAdapter = null;
            try
            {
                connection = new SqlConnection(conString);
                command = new SqlCommand(commandTextOrProcedureName, connection);
                command.CommandType = cmdType;
                if (parameters != null)
                {
                    command.Parameters.AddRange(parameters);
                }
                connection.Open();
                dataSet = new DataSet();
                dataAdapter = new SqlDataAdapter();
                dataAdapter.SelectCommand = command;
                rowCount = dataAdapter.Fill(dataSet);
                IsExecuted = true;
            }
            catch (Exception e)
            {
                ShowError(e.ToString());
            }
            finally
            {
                if (connection != null)
                {
                    connection.Close(); connection.Dispose();
                }
                if (command != null)
                {
                    command.Dispose();
                }
                if (dataAdapter != null)
                {
                    dataAdapter.Dispose();
                }
            }
            return dataSet;
        }

Хочу добавить метод который будет возвращать DataTable. Вопрос в том как его лучше реализовать.
1 способ (через вызов предыдущего метода возвращающего DataSet):

C#
1
2
3
4
5
6
7
8
9
10
        public static DataTable CommandDataTableFill(string commandText, params SqlParameter[] parameters)
        {
            DataSet ds = new DataSet();
            DataTable dt = new DataTable();
            ds = DataSetFill(commandText, CommandType.Text, parameters);
            if (ds.Tables.Count > 1)
                return dt = ds.Tables[0];
            else
                return dt;
        }

2 способ (аналог метода с DataSet, только возвращает DataTable) :

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
        private static DataTable DataTableFill(string commandTextOrProcedureName, CommandType cmdType, params SqlParameter[] parameters)
        {
            int rowCount = 0;
            SqlConnection connection = null;
            SqlCommand command = null;
            DataTable dataTable = null;
            SqlDataAdapter dataAdapter = null;
            try
            {
                connection = new SqlConnection(conString);
                command = new SqlCommand(commandTextOrProcedureName, connection);
                command.CommandType = cmdType;
                if (parameters != null)
                {
                    command.Parameters.AddRange(parameters);
                }
                connection.Open();
                dataTable = new DataTable();
                dataAdapter = new SqlDataAdapter();
                dataAdapter.SelectCommand = command;
                rowCount = dataAdapter.Fill(dataTable);
                IsExecuted = true;
            }
            catch (Exception e)
            {
                ShowError(e.ToString());
            }
            finally
            {
                if (connection != null)
                {
                    connection.Close(); connection.Dispose();
                }
                if (command != null)
                {
                    command.Dispose();
                }
                if (dataAdapter != null)
                {
                    dataAdapter.Dispose();
                }
            }
            return dataTable;
        }

Не будет ли второй способ так сказать копипастом метода DataSetFill (отличия только в типе возвращаемого объекта и одной строке кода).

@kodv · 08.04.2015, 04:41

Сообщение от NatalieZ

Хочу добавить метод который будет возвращать DataTable. Вопрос в том как его лучше реализовать.

Лично я бы проголосовал за второй способ, только с одной маленькой поправкой: после создания метода DataTableFill вторым способом вам нужно выделить мышкой полностью метод DataSetFill и удалить его ко всем чертям. Ибо при наличии метода, который будет возвращать DataTable, нафиг не нужен метод, который будет возвращать ту же DataTable, только обернутую в индивидуальный DataSet. А если и понадобиться, то его лучше сделать так:

C#
1
2
3
4
5
6
private static DataSet DataSetFill(string commandTextOrProcedureName, CommandType cmdType, params SqlParameter[] parameters)
{
    DataSet dataSet = new DataSet();
    dataSet.Tables.Add(DataTableFill(commandText, CommandType.Text, parameters));
    return dataSet;
}

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

@NatalieZ · 08.04.2015, 07:52 **[ТС]**

Большое спасибо за помощь, я так и сделаю!

Новые блоги и статьи Все статьи Все блоги /
Модель микоризы: классовый агентный подход anaschu 02.01.2026 Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .	Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Programma_Boinc 01.01.2026 Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Сочетание глобально распределённой вычислительной мощности и инновационных. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .

@NatalieZ 0 / 0 / 0 Регистрация: 13.10.2014 Сообщений: 8
	08.04.2015, 07:52 [ТС]
	Большое спасибо за помощь, я так и сделаю! 0