Говнокод #17960 — C# — Говнокод.ру

C# / Говнокод #17960

+133
1. 01
2. 02
3. 03
4. 04
5. 05
6. 06
7. 07
8. 08
9. 09
10. 10
11. 11
12. 12
13. 13
14. 14
15. 15
16. 16
17. 17
18. 18
19. 19
20. 20
21. 21
22. 22
23. 23
24. 24
25. 25
26. 26
27. 27
28. 28
29. 29
30. 30
31. 31
32. 32
```
if (textBox[0].Text == "" && textBox[1].Text == "" && textBox[2].Text == "" && textBox[3].Text == "" && textBox[4].Text == "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("");
else if (textBox[0].Text != "" && textBox[1].Text == "" && textBox[2].Text == "" && textBox[3].Text == "" && textBox[4].Text == "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("[surname] LIKE '%{0}%'", textBox[0].Text);
else if (textBox[0].Text == "" && textBox[1].Text != "" && textBox[2].Text == "" && textBox[3].Text == "" && textBox[4].Text == "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("[name] LIKE '%{0}%'", textBox[1].Text);
else if (textBox[0].Text == "" && textBox[1].Text == "" && textBox[2].Text != "" && textBox[3].Text == "" && textBox[4].Text == "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("[midname] LIKE '%{0}%'", textBox[2].Text);
else if (textBox[0].Text == "" && textBox[1].Text == "" && textBox[2].Text == "" && textBox[3].Text != "" && textBox[4].Text == "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("course={0}", textBox[3].Text);
else if (textBox[0].Text == "" && textBox[1].Text == "" && textBox[2].Text == "" && textBox[3].Text == "" && textBox[4].Text != "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("group={0}", textBox[4].Text);
else if (textBox[0].Text != "" && textBox[1].Text != "" && textBox[2].Text == "" && textBox[3].Text == "" && textBox[4].Text == "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("[surname] LIKE '%{0}%' and [name] LIKE '%{1}%'", textBox[0].Text, textBox[1].Text);
else if (textBox[0].Text == "" && textBox[1].Text != "" && textBox[2].Text != "" && textBox[3].Text == "" && textBox[4].Text == "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("[name] LIKE '%{0}%' and [midname] LIKE '%{1}%'", textBox[1].Text, textBox[2].Text);
else if (textBox[0].Text == "" && textBox[1].Text == "" && textBox[2].Text != "" && textBox[3].Text != "" && textBox[4].Text == "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("[midname] LIKE '%{0}%' and course={1}", textBox[2].Text, textBox[3].Text);
else if (textBox[0].Text == "" && textBox[1].Text == "" && textBox[2].Text == "" && textBox[3].Text != "" && textBox[4].Text != "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("course={0} and group={1}", textBox[3].Text, textBox[4].Text);
else if (textBox[0].Text != "" && textBox[1].Text != "" && textBox[2].Text != "" && textBox[3].Text == "" && textBox[4].Text == "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("[surname] LIKE '%{0}%' and [name] LIKE '%{1}%' and [midname] LIKE '%{2}%'", textBox[0].Text, textBox[1].Text, textBox[2].Text);
else if (textBox[0].Text == "" && textBox[1].Text != "" && textBox[2].Text != "" && textBox[3].Text != "" && textBox[4].Text == "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("[name] LIKE '%{0}%' and [midname] LIKE '%{1}%' and course={2}", textBox[1].Text, textBox[2].Text, textBox[3].Text);
else if (textBox[0].Text == "" && textBox[1].Text == "" && textBox[2].Text != "" && textBox[3].Text != "" && textBox[4].Text != "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("[midname] LIKE '%{0}%' and course={1} and group={2}", textBox[2].Text, textBox[3].Text, textBox[4].Text);
else if (textBox[0].Text != "" && textBox[1].Text != "" && textBox[2].Text != "" && textBox[3].Text != "" && textBox[4].Text == "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("[surname] LIKE '%{0}%' and [name] LIKE '%{1}%' and [midname] LIKE '%{2}%' and course={3}", textBox[0].Text, textBox[1].Text, textBox[2].Text, textBox[3].Text);
else if (textBox[0].Text == "" && textBox[1].Text != "" && textBox[2].Text != "" && textBox[3].Text != "" && textBox[4].Text != "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("[name] LIKE '%{0}%' and [midname] LIKE '%{1}%' and course={2} and group={3}", textBox[1].Text, textBox[2].Text, textBox[3].Text, textBox[4].Text);
else if (textBox[0].Text != "" && textBox[1].Text != "" && textBox[2].Text != "" && textBox[3].Text != "" && textBox[4].Text != "")
    myViewStudentsTable.DefaultView.RowFilter = string.Format("[surname] LIKE '%{0}%' and [name] LIKE '%{1}%' and [midname] LIKE '%{2}%' and course={3} and group={4}", textBox[0].Text, textBox[1].Text, textBox[2].Text, textBox[3].Text, textBox[4].Text);
```
"Есть 5 текстовых полей и желание понять, как можно в зависимости от пустоты или заполненности этих полей, одного или нескольких создать малое количество операторов if else"

Порадовало "малое количество"

Запостил: Psilon, 08 Апреля 2015

Tweet
Комментарии (50) RSS
- cyperh 08.04.2015 20:43 # −13
  
  показать все, что скрытоUse Maybe monade, Luke!
  Ответить
- cyperh 08.04.2015 20:58 # −14
  
  показать все, что скрытоА вообще есть вариант описать все условия не if else, а состояниями объектов, которые будут храниться в словаре, каждый набор состояний это ключ, значение это то, что должно вернуться. И никаких if else, только правда расширяемость страдает, потому что все это жестко забивается. Но лучше чем забор из if else. В качестве объекта состояния можно даже тот же Tuple использовать.
  Ответить
- kegdan 08.04.2015 21:08 # −13
  показать все, что скрытоЧеловек - рефакторинг спешит на помощь!
  
  . string[] someStrings ={"[surname] LIKE '%{0}%'", "[name] LIKE '%{0}%'", "[midname] LIKE '%{0}%'", "course={0}", "group={0}"}; string middlePart = " and "; bool isNotFirst = false; StringBuilder sb = new StringBuilder(); for (int i = 0; i < 5; i++) { if (textBox[i].Text != string.Empty) { if (isNotFirst) sb.Append(middlePart); sb.Append(string.Format(someStrings[i], textBox[i].Text)); isNotFirst = true; } } myViewStudentsTable.DefaultView.RowFilter = sb.ToString();
  Ответить
  - wvxvw 09.04.2015 00:05 # −14
    
    показать все, что скрыто
    query_parts(['[surname] LIKE "%~p%"', '[name] LIKE "%~p%"', '[midname] LIKE "%~p%"', 'course="~p"', 'group="~p"']). tuple([], [], []) :- !. tuple([X | Xs], [Y | Ys], [(X, Y) | Zs]) :- tuple(Xs, Ys, Zs). exclude_query((false, _)). interleave([X], _, X) :- !. interleave([X | Xs], Glue, Result) :- interleave(Xs, Glue, Previous), format(atom(Result), "~p~p~p", [X, Glue, Previous]). build_query(fields(Surname, Name, Midname, Course, Group), Query) :- Fields = [Surname, Name, Midname, Course, Group], query_parts(Parts), tuple(Fields, Parts, Tuples), exclude(exclude_query, Tuples, Filtered), findall(Z, (member((X, Y), Filtered), format(atom(Z), Y, [X])), Disjoint), interleave(Disjoint, ' and ', Query). %% ?- build_query(fields('Kaczynski', 'Ted', false, 'Number Theory', false), Q). %% Q = '[surname] LIKE "%Kaczynski%" and [name] LIKE "%Ted%" and course="Number Theory"'.
    
    Но что-то мне подсказывает, что наверное можно было лучше.
    Ответить
    - myaut 09.04.2015 00:14 # −13
      
      показать все, что скрытоНаш сайт написан на PHP со вставками Хаскеля?
      Ответить
      
      wvxvw 09.04.2015 00:47 # −11
      
      показать все, что скрытоФейсбук чтоли?
      Ответить
      
      roman-kashitsyn 09.04.2015 10:29 # −13
      
      показать все, что скрытоХаскелем у них спам фильтруется
      Ответить
  - Lokich 09.04.2015 11:37 # −13
    
    показать все, что скрытозачем тебе булевая isNotFirst? ты по счетчику цикла знаешь какая это итерация
    Ответить
    - bormand 09.04.2015 11:54 # −13
      
      показать все, что скрытоБолее того - нахуя вообще делать переменные с негативным смыслом? Нечитабельно же.
      Ответить
      
      kegdan 09.04.2015 18:11 # −14
      
      показать все, что скрытоМеня больше !isFirst запутает
      Ответить
    - kegdan 09.04.2015 18:10 # −14
      
      показать все, что скрытоЧто бы знать когда серединку вставлять вестимо
      Ответить
      
      bormand 09.04.2015 21:59 # −13
      
      показать все, что скрытоif (i != 0) или даже if (i), если шарпик позволяет.
      Ответить
      
      kegdan 09.04.2015 22:06 # −14
      
      показать все, что скрытоА если там нет параметров и ниче вообще в серединку вставлять не нужно?
      Ответить
      
      bormand 09.04.2015 22:09 # −14
      
      показать все, что скрытоАаааа, вон че, строчкой выше textBox[i].Text проверяется на пустоту. Тогда без этой переменной не обойдёшься... Но not я бы выкинул нахуй.
      
      P.S. А Join тут поюзать не получится вместо поклейки вручную?
      Ответить
      
      kegdan 09.04.2015 22:12 # −14
      
      показать все, что скрытоЕсли будет не пустым только 4 текстбокс то мы не должны перед ним добавлять and ибо до него все пусто.
      
      можно всегда писать после and а потом стереть последние n символов из стрингбилдера, но этот как то не тру что-ли...
      Ответить
      
      bormand 09.04.2015 22:17 # −14
      
      показать все, что скрытоАга. А набивание в какой-нибудь лист и Join (или как там у вас в шарпе клеят) тут не прокатят?
      Ответить
      
      kegdan 09.04.2015 22:19 # −14
      
      показать все, что скрытоможно, но это не лучше моего решения.
      Ответить
      
      bormand 09.04.2015 22:23 # −13
      
      показать все, что скрытоЛучше, т.к. меньше частных случаев: тупо набиваем в массив, тупо джойним, не надо думать о том, первая ли запись.
      Ответить
      
      kegdan 09.04.2015 22:27 # −12
      
      показать все, что скрыторефакторить проще.
      
      Хотя может это говорит маленький псевдооптимизптор внутри меня
      Ответить
      
      bormand 09.04.2015 22:29 # −14
      
      показать все, что скрыто> рефакторить проще
      Вот с массивом то как раз проще...
      List<String> filters = new List<String>(); for (int i = 0; i < textBox.Length; i++) if (textBox[i].Text != "") filters.Append(string.Format(someStrings[i], textBox[i].Text)); myViewStudentsTable.DefaultView.RowFilter = string.join(" and ", filters);
      И с массивом можно убрать цикл, при необходимости. С ручной поклейкой and'ов ты заебёшься это делать.
      Ответить
      
      kegdan 09.04.2015 22:31 # −13
      
      показать все, что скрыто
      string[] someStrings ={"[surname] LIKE '%{0}%'", "[name] LIKE '%{0}%'", "[midname] LIKE '%{0}%'", "course={0}", "group={0}"}; string middlePart = " and "; List<string> sl = new List<string>(someStrings.Length); for (int i = 0; i < someStrings.Length; i++) if (textBox[i].Text != string.Empty) sl.Add(string.Format(someStrings[i], textBox[i].Text)); myViewStudentsTable.DefaultView.RowFilter = String.Join(middlePart,sl);
      Ответить
      
      bormand 09.04.2015 22:33 # −14
      
      показать все, что скрытоКстати, не уподобляйся PHP'шникам... В твоём коде есть SQL иньекция (впрочем, она была изначально в коде автора).
      Ответить
      
      kegdan 09.04.2015 22:34 # −14
      
      показать все, что скрытоГде?
      Ответить
      
      bormand 09.04.2015 22:39 # −14
      
      показать все, что скрытоА в шарпе есть нормальный QueryBuilder? Или только ручками клеить строки, только хардкор?
      Ответить
      
      kegdan 09.04.2015 22:46 # −13
      
      показать все, что скрытосвертка листа? только через линк
      Ответить
      
      kegdan 09.04.2015 22:50 # −14
      
      показать все, что скрытоа, ты про sql
      
      там же Entity Framework и linq to sql
      
      пишешь на линке, он сам в sql запрос конвертит
      Ответить
      
      bormand 09.04.2015 22:52 # −14
      
      показать все, что скрытоА неполный запрос для RowFilter сможет собрать?
      Ответить
      
      kegdan 09.04.2015 22:59 # −14
      
      показать все, что скрытоЧто есть неполный запрос? Первый раз слышу, даже интересно
      Ответить
      
      bormand 09.04.2015 23:11 # −13
      
      показать все, что скрытоНу вот как здесь - одно where без from и select.
      Ответить
      
      kegdan 09.04.2015 23:27 # −10
      
      показать все, что скрытоА в чем проблема? можно же linq to sql юзать и в обычной точечной нотации
      
      someDataSet.Where(cond);
      
      Только я не уверен будет ли это неполным запросом, или эфик его добьет до полного. По сути там режим - пиши и молись что он преобразует так как надо.
      
      Ну и в любом случае можно инъекции писать самому, если не нравиться ORM
      
      В общем нужно курить маны по эфику что бы точно ответить.
      
      А где юзаются такие неполные запросы? Просто я сертификацию собираюсь проходить по sql, хочется быть в курсе
      Ответить
      
      bormand 10.04.2015 00:00 # −13
      
      показать все, что скрыто> А где юзаются такие неполные запросы?
      Ну ты же сам написал - myViewStudentsTable.DefaultView.RowFilter.
      Ответить
      
      kegdan 10.04.2015 00:05 # −11
      
      показать все, что скрытовсе, уже туплю, пора спать)
      Ответить
      
      bormand 09.04.2015 22:38 # −14
      
      показать все, что скрыто> string middlePart = " and ";
      Константы головного мозга? Разве что захочется замутить переключалку or/and...
      Ответить
      
      kegdan 09.04.2015 22:44 # −13
      
      показать все, что скрытоя не думаю что это плохо. Лучше написать переменную, чем потом менять значения в нескольких местах. Мне вообще казалось, что константы надо выносить от греха.
      Ответить
      
      kegdan 09.04.2015 22:36 # −14
      
      показать все, что скрытоладно, я проиграл)
      Ответить
      
      bormand 09.04.2015 22:22 # −12
      
      показать все, что скрыто
      string.join(" and ", someStrings.zip(textBox, (s, tb) => tb.Text != "" ? s.format(tb.Text) : "").where(s => s != ""))
      Я наркоман?
      Ответить
      
      kegdan 09.04.2015 22:24 # −14
      
      показать все, что скрытотакое ощущение, что ты робот, созданный мелкософтом)
      
      Отсыпь)
      Ответить
    - bormand 09.04.2015 22:11 # −14
      
      показать все, что скрытоНе знает. Он же пустые элементы пропускает.
      Ответить
- xyilka 09.04.2015 01:57 # +12
  
  cleaned
  Ответить
  - botbatbotbatbot 09.04.2015 03:27 # −15
    
    показать все, что скрытоcleaned
    Ответить
  - botbatbotbatbot 09.04.2015 03:29 # −15
    
    показать все, что скрытоcleaned
    Ответить
  - gazenva85 09.04.2015 04:14 # −15
    
    показать все, что скрытоcleaned
    Ответить
  - StackBlower 09.04.2015 04:15 # −14
    
    показать все, что скрытоcleaned
    Ответить
  - treoie 09.04.2015 04:17 # −15
    
    показать все, что скрытоcleaned
    Ответить
  - gost 09.04.2015 10:21 # −14
    
    показать все, что скрытоКто здесь?!
    Ответить
- dmli 10.04.2015 20:37 # −14
  
  показать все, что скрытокто здесь минусует???
  Ответить
  - 3.14159265 10.04.2015 20:40 # −13
    
    показать все, что скрытоКакой-то поехавший хохол-нищеброд.
    Ответить
    - guest 10.04.2015 20:42 # −12
      
      показать все, что скрыто
      ...Не ты - поехавший, не он - поехавший,это я, я поехавший!..
      Ответить
      
      bormand 10.04.2015 21:17 # −11
      
      показать все, что скрыто> не ты
      > не он
      Стертор мстит за Сёму?
      Ответить
    - kalximik 10.04.2015 22:51 # −1
      
      cleaned
      Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только во вторник, пятницу или субботу
Помни, guest, за тобой могут следить!

А не использовать ли нам bbcode?

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

Говнокод: по колено в коде.

C# / Говнокод #17960

Комментарии (50) RSS

Добавить комментарий