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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2337)
4. JavaScript (2030)
5. C# (1929)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   +132

   1. 1
   2. 2
   3. 3

   long id;
   try { id = long.Parse(e.Item.Cells[0].Text); }
   catch { id = 0; }

   Эта жемчужина прекрасно описывает весь говнокод производимый различными группами людей в моей компании.

   Запостил: piocsic, 12 Июня 2012

   Tweet

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

   • guest 12.06.2012 14:11 # +1

     Что-то подобное написал на 2 курсе. Препод поржал)

     Ответить

   • Lure Of Chaos 12.06.2012 16:00 # +1

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

     Ответить

     • 3.14159265 12.06.2012 16:04 # +2

       >длинные стринги
       Узкие шорты

       Ответить

   • guest 12.06.2012 22:11 # 0

     апофеоз, не спорю (: но все же, как правильно распарсить стринги в число?

     Ответить

     • bormand 12.06.2012 22:18 # 0

       http://ideone.com/3M32I
       Как-то так?

       Ответить

       • abatishchev 12.06.2012 22:32 # +1

         int i;
         Int32.TryParse(input, out i);

         документация по out гарантирует что переменная будет инициализирована.
         и будет это default(Int32).

         Ответить

         • guest 12.06.2012 22:34 # +2

           Да, кстати тогда и инициализировать в ноль не нужно.

           Ответить

         • dirtygopher 13.06.2012 13:19 # +1

           Возможно код из .net framework 1.1, насколько я помню, там не было long.TryParse

           Ответить

           • guest 22.06.2012 17:53 # +1

             Проверил, так похоже и есть. В msdn по long.TryParse нет инфы для 1.1

             Ответить

             • bormand 22.06.2012 18:05 # 0

               Даже в обсираемой всеми Delphi 7 был TryStrToInt ;)
               Как же MS так пролетели.

               Ответить

               • guest 22.06.2012 18:21 # 0

                 Потому что судя по всему 1.0 выпускали впопыхах, а 1.1 был во многом просто багфикс 1.0

                 Ответить

                 • guest 22.06.2012 18:22 # 0

                   там было не до TryParse

                   Ответить

               • guest 22.06.2012 18:26 # 0

                 А ещё возможно потому что в первых версиях многое тупо без задней мысли копировалось 1-к-1 с Java, и в Java'овском прототипном Long нет метода tryParseLong, есть только parseLong.

                 Ответить

             • piocsic 22.06.2012 18:34 # 0

               Судя по перфорсу, код этот был написан во времена дотнет 2.0.
               
               Да если бы он и был написал во времена 1.0, всё равно нельзя проглатывать все исключения. Надо было хотя бы прописать catch (FormatException) {}

               Ответить

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

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

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

   Семь раз отмерь — один отрежь, guest!

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

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

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