Условия в строке с командой?

@AxOn · Регистрация: 28.04.2014

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

Здравствуйте, написал как мог код с условиями что бы не путать словами, так работает

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
45
46
47
48
49
50
51
public IEnumerable<Tasks> TaskSelect(int curId, int tagId, string date)
        {
            using (var sqlConn = SqlConnection())
            {
                string sqlSel = "SELECT * FROM Tasks WHERE UsId = @UsId";
 
                if (tagId != 0)
                {
                    sqlSel = "SELECT * FROM Tasks AS t JOIN CrossTasksTags AS c ON (t.TaskId = c.TaskId) WHERE t.UsId = @UsId and c.TagsId = @TagsId ORDER BY c.TaskId";
                }
                if (date != "")
                {
                    sqlSel = "SELECT * FROM Tasks AS t JOIN CrossTasksTags AS c ON (t.TaskId = c.TaskId) WHERE t.UsId = @UsId and t.TaskTerm = @TaskTerm ORDER BY c.TaskId";
                }
                if (tagId != 0 && date != "")
                {
                    sqlSel = "SELECT * FROM Tasks AS t JOIN CrossTasksTags AS c ON (t.TaskId = c.TaskId) WHERE t.UsId = @UsId and c.TagsId = @TagsId and TaskTerm = @TaskTerm ORDER BY c.TaskId";
                }
 
                using (SqlCommand cmdSel = new SqlCommand(sqlSel, sqlConn))
                {
 
                    cmdSel.Parameters.AddWithValue("@UsId", curId);
                    cmdSel.Parameters.AddWithValue("@TagsId", tagId);
                    if (date != "")
                    {
                        cmdSel.Parameters.Add("@TaskTerm", SqlDbType.Date).Value = date;
                    }
 
 
                    List<Tasks> query = new List<Tasks>();
                    using (SqlDataReader reader = cmdSel.ExecuteReader())
                    {
                        while (reader.Read())
                        {
                            query.Add(new Tasks()
                            {
                                StatusString = GetStatusString(int.Parse(reader["StatusId"].ToString())),
                                Description = reader["Description"].ToString(),
                                TaskId = int.Parse(reader["TaskId"].ToString()),
                                Title = reader["Title"].ToString(),
                                Tags = reader["Tags"].ToString(),
                                TaskTerm = DateTime.Parse(reader["TaskTerm"].ToString()),
                                UsId = int.Parse(reader["UsId"].ToString())
 
                            });
                        }
                    }
                    return query;
                }
            }

Это ужасно когда столько условий переопределяют одну переменную с мало отличимым sql запросом, но не знаю как написать иначе, правильно и красиво, отрефакторите плз
Разница лишь фильтре

@OwenGlendower · 14.08.2015, 19:37

AxOn, так например можно переписать:

Кликните здесь для просмотра всего текста

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
45
46
47
48
49
50
51
52
53
54
55
56
public IEnumerable<Tasks> TaskSelect(int curId, int tagId, string date)
{
    using (var sqlConn = SqlConnection())
    {
        using (SqlCommand cmdSel = new SqlCommand())
        {
            cmdSel.Connection = sqlConn;
            cmdSel.Parameters.AddWithValue("@UsId", curId);
        
            StringBuilder sbWhere = new StringBuilder();
            if (tagId != 0)
            {
                sbWhere.Append(" AND c.TagsId = @TagsId");
                cmdSel.Parameters.AddWithValue("@TagsId", tagId);
            }
            if (date != "")
            {
                sbWhere.Append(" AND t.TaskTerm = @TaskTerm");
                cmdSel.Parameters.Add("@TaskTerm", SqlDbType.Date).Value = date;
            }
            
            StringBuilder sbSql = new StringBuilder("SELECT * FROM Tasks AS t");
            if (sbWhere.Length > 0)
            {
                sbSql.Append(" JOIN CrossTasksTags AS c ON (t.TaskId = c.TaskId)");
            }
            sbSql.Append(" WHERE t.UsId = @UsId");
            if (sbWhere.Length > 0)
            {
                sbSql.Append(sbWhere);
                sbSql.Append(" ORDER BY c.TaskId");
            }
            
            cmdSel.CommandText = sbSql.ToString();
    
            List<Tasks> query = new List<Tasks>();
            using (SqlDataReader reader = cmdSel.ExecuteReader())
            {
                while (reader.Read())
                {
                    query.Add(new Tasks()
                    {
                        StatusString = GetStatusString(int.Parse(reader["StatusId"].ToString())),
                        Description = reader["Description"].ToString(),
                        TaskId = int.Parse(reader["TaskId"].ToString()),
                        Title = reader["Title"].ToString(),
                        Tags = reader["Tags"].ToString(),
                        TaskTerm = DateTime.Parse(reader["TaskTerm"].ToString()),
                        UsId = int.Parse(reader["UsId"].ToString())
                    });
                }
            }
            return query;
        }
    }
}

Беда с подобными решениями что их становится трудно поддерживать при большом кол-ве запросов. Советую изучать EF или NHibernate. Там задачи динамической фильтрации (и построения запрсов) решаются проще.

Еще бы заменил тип вовращаемого значения на ReadOnlyCollection<Tasks>. Это при условии если ты выбрал IEnumerable<T> чтобы вызывающий код не мог менять набор данных. В return тогда нужно написать query.AsReadOnly().

Новые блоги и статьи Все статьи Все блоги /
Переходник USB-CAN-GPIO Eddy_Em 20.03.2026 Достаточно давно на работе возникла необходимость в переходнике CAN-USB с гальваноразвязкой, оный и был разработан. Однако, все меня терзала совесть, что аж 48-ногий МК используется так тупо: просто. . .	Оттенки серого Argus19 18.03.2026 Оттенки серого Нашёл в интернете 3 прекрасных модуля: Модуль класса открытия диалога открытия/ сохранения файла на Win32 API; Модуль класса быстрого перекодирования цветного изображения в оттенки. . .	SDL3 для Desktop (MinGW): Рисуем цветные прямоугольники с помощью рисовальщика SDL3 на Си и C++ 8Observer8 17.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-rectangles-sdl3-c. zip finish-rectangles-sdl3-cpp. zip	Символические и жёсткие ссылки в Linux. algri14 15.03.2026 Существует два типа ссылок — символические и жёсткие. Ссылка в Linux — это запись в каталоге, которая может указывать либо на inode «файла-ИСТОЧНИКА», тогда это будет «жёсткая ссылка» (hard link),. . .
[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ФедосеевПавел 14.03.2026 Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .	делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .