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

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5698)
2. C++ (2733)
3. Куча (2324)
4. JavaScript (2027)
5. C# (1925)
6. Java (1501)
7. Си (1095)
8. Pascal (647)
9. Python (582)
10. 1C (531)
11. SQL (431)
12. ActionScript (292)
13. Objective C (202)
14. bash (196)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (28)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

1. C# / Говнокод #15728

   +137

   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
   33. 33
   34. 34
   35. 35
   36. 36
   37. 37
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49
   50. 50

   model.PID = row["PID"].ToString();
   model.Rect = row["RECt"].ToString();
   model.Pubt = row["PUBt"].ToString();
   model.Ex = row["Ex"].ToString();
   model.Prev_Vol = row["Prev_Vol"].ToString() != "" ? Tools.ParseDecimalValue(row["Prev_Vol"].ToString()).FormatAmountWithoutDecimal() : "";
   model.Curr_Vol = row["Curr_Vol"].ToString() != "" ? Tools.ParseDecimalValue(row["Curr_Vol"].ToString()).FormatAmountWithoutDecimal() : "";
   model.Mov_Vol = row["Mov_Vol"].ToString() != "" ? Tools.ParseDecimalValue(row["Mov_Vol"].ToString()).FormatAmountWithoutDecimal() : "";
   decimal tonavgvol = Tools.ParseDecimalValue(row["Mov_Vol"].ToString());
   decimal currentvol = Tools.ParseDecimalValue(row["Curr_Vol"].ToString());
   if (row["Mov_Vol"].ToString() != "" && row["Curr_Vol"].ToString() != "" && tonavgvol != 0)
   	model.VolRat = (currentvol / tonavgvol).FormatAmount();
   model.Open_Vol = row["Open_Vol"].ToString() != "" ? Tools.ParseDecimalValue(row["Open_Vol"].ToString()).FormatAmountWithoutDecimal() : "";
   model.Close_Vol = row["Close_Vol"].ToString() != "" ? Tools.ParseDecimalValue(row["Close_Vol"].ToString()).FormatAmountWithoutDecimal() : "";
   model.AllTicks = row["AllTicks"].ToString() != "" ? Tools.ParseDecimalValue(row["AllTicks"].ToString()).FormatAmountWithoutDecimal() : "";
   model.ATR = row["ATR14"].ToString() != "" ? Tools.ParseDecimalValue(row["ATR14"].ToString()).FormatAmountFourDecimals() : "";
   decimal open = Tools.ParseDecimalValue(row["Open"].ToString());
   decimal close = Tools.ParseDecimalValue(row["Close"].ToString());
   if (row["Open"].ToString() != "" && row["Close"].ToString() != "" && open != 0)
   	model.PrevDay = (((close - open) / open) * 100).FormatPercent();
   model.PrevClose = row["PrevClose"].ToString() != "" ? Tools.ParseDecimalValue(row["PrevClose"].ToString()).FormatAmount() : "";
   model.DayOpen = row["DayOpen"].ToString() != "" ? Tools.ParseDecimalValue(row["DayOpen"].ToString()).FormatAmount() : "";
   decimal prevclose = Tools.ParseDecimalValue(row["PrevClose"].ToString());
   decimal dayopen = Tools.ParseDecimalValue(row["DayOpen"].ToString());
   if (row["DayOpen"].ToString() != "" && row["PrevClose"].ToString() != "" && prevclose != 0)
   	model.OpenGap = (((dayopen - prevclose) / prevclose) * 100).FormatPercent();
   model.SPYLast = row["TONSPLast"].ToString() != "" ? Tools.ParseDecimalValue(row["TONSPLast"].ToString()).FormatAmount() : "";
   model.TONOpen = row["TONOpen"].ToString() != "" ? Tools.ParseDecimalValue(row["TONOpen"].ToString()).FormatAmount() : "";
   model.TONHigh = row["TONHigh"].ToString() != "" ? Tools.ParseDecimalValue(row["TONHigh"].ToString()).FormatAmount() : "";
   model.TONLow = row["TONLow"].ToString() != "" ? Tools.ParseDecimalValue(row["TONLow"].ToString()).FormatAmount() : "";
   model.TONLast = row["TONLast"].ToString() != "" ? Tools.ParseDecimalValue(row["TONLast"].ToString()).FormatAmount() : "";
   model.EODHigh = row["EODHigh"].ToString() != "" ? Tools.ParseDecimalValue(row["EODHigh"].ToString()).FormatAmount() : "";
   model.EODLow = row["EODLow"].ToString() != "" ? Tools.ParseDecimalValue(row["EODLow"].ToString()).FormatAmount() : "";
   model.EODClose = row["EODClose"].ToString() != "" ? Tools.ParseDecimalValue(row["EODClose"].ToString()).FormatAmount() : "";
   model.SPYClose = row["EODSPClose"].ToString() != "" ? Tools.ParseDecimalValue(row["EODSPClose"].ToString()).FormatAmount() : "";
   model.PostVWAP = row["PostVWAP"].ToString() != "" ? Tools.ParseDecimalValue(row["PostVWAP"].ToString()).FormatAmount() : "";
   model.PreVWAP = row["PreVWAP"].ToString() != "" ? Tools.ParseDecimalValue(row["PreVWAP"].ToString()).FormatAmount() : "";
   model.MainVWAP = row["MainVWAP"].ToString() != "" ? Tools.ParseDecimalValue(row["MainVWAP"].ToString()).FormatAmount() : "";
   model.AllVWAP = row["AllVWAP"].ToString() != "" ? Tools.ParseDecimalValue(row["AllVWAP"].ToString()).FormatAmount() : "";
   model.EODVWAP = row["EODVWAP"].ToString() != "" ? Tools.ParseDecimalValue(row["EODVWAP"].ToString()).FormatAmount() : "";
   
   decimal tonlast = Tools.ParseDecimalValue(row["TONLast"].ToString());
   decimal eodhigh = Tools.ParseDecimalValue(row["EODHigh"].ToString());
   decimal eodlow = Tools.ParseDecimalValue(row["EODLow"].ToString());
   decimal tonhigh = Tools.ParseDecimalValue(row["TONHigh"].ToString());
   decimal tonlow = Tools.ParseDecimalValue(row["TONLow"].ToString());
   decimal eodclose = Tools.ParseDecimalValue(row["EODClose"].ToString());
   decimal oedspclose = Tools.ParseDecimalValue(row["EODSPClose"].ToString());
   decimal tonsplast = Tools.ParseDecimalValue(row["TONSPLast"].ToString());
   
   // еще где-то 100-150 строчек в таком же духе

   ADO.NET во все поля.
   Бизнес-логика? Что это такое?
   
   P.S. У класса model все свойства типа string, у всех decimal переменных потом тоже вызывается ToString().

   Запостил: vertu, 11 Апреля 2014

   Tweet

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

   • Smekalisty 11.04.2014 16:19 # +1

     Похоже на генерированный код, не могу себе представить что кто-то может такое писать

     Ответить

     • vertu 11.04.2014 16:23 # 0

       Не знаю, разочарую или обрадую, но это писал реальный человек. Весь проект это набор файлов: *Func.cs (IndexFunc, UserFunc, AuthFunc и т.д.), в которых по статичному классу со статичными методами. Это он называет DAL.

       Ответить

       • TauSigma 11.04.2014 16:29 # 0

         DbNull.Value не часто вылезает в row[].ToString()?
         Вроде так он называется

         Ответить

         • vertu 11.04.2014 16:35 # +1

           DBNull.ToString() - возвращает пустую строку.

           Ответить

           • cyperh 12.04.2014 11:31 # 0

             тогда даже так string.IsNullOrEmpty(DBNull.ToString())

             Ответить

   • brutushafens 11.04.2014 16:24 # −22

     показать все, что скрытоcleaned

     Ответить

     • Мистер Хэнки 11.04.2014 17:08 # +2

       штопай срачло, Маня.

       Ответить

       • brutushafens 11.04.2014 17:27 # −1

         cleaned

         Ответить

     • bormand 11.04.2014 17:35 # +3

       Харош раскручивать левые сайты. Сколько они тебе за пиар заплатили?

       Ответить

       • brutushafens 11.04.2014 17:38 # 0

         cleaned

         Ответить

         • bormand 11.04.2014 17:40 # +2

           Этот = ГК? Ну да. Спамишь на этом сайте рекламой всяких секс-двигателей...

           Ответить

   • TauSigma 14.04.2014 13:25 # 0

     А что автору мешало использовать типизированные датасеты? Там даже через partial классы можно Nullable заюзать.
     Без дурацких методов Is{PropertyName}Null();

     Ответить

   • kegdan 19.04.2014 14:32 # 0

     Адошка, что с тобой этот ирод сделал!? Мое сердце плачет как сакура на ветру

     Ответить

   • planaric 24.04.2014 11:39 # 0

     Индус-велосипедостроитель.
     Strongly Typed Dataset? Entity Framework? Не, не слышали.

     Ответить

     • roman-kashitsyn 24.04.2014 11:41 # +2

       Stringly Typed Dataset

       Ответить

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

   Ошибка компиляции комментария:

   1. Гости могут высказаться только в понедельник, среду, четверг или воскресение

   Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

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

   • [b]жирный[/b] — жирный
   • [i]курсив[/i] — курсив
   • [u]подчеркнутый[/u] — подчеркнутый
   • [s]перечеркнутый[/s] — перечеркнутый
   • [blink]мигающий[/blink] — мигающий
   • [color=red]цвет[/color] — цвет (подробнее)
   • [size=20]размер[/size] — размер (подробнее)
   • [code=<language>]some code[/code] (подробнее)

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