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

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

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

1. PHP (5704)
2. C++ (2739)
3. Куча (2355)
4. JavaScript (2029)
5. C# (1928)
6. Java (1502)
7. Си (1113)
8. Pascal (647)
9. Python (583)
10. 1C (538)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

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

   +8

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7

   public TD GetColumn(int i) {
               try {
                   return this.Columns[i];
               } catch {
                   return this.Columns[i - 1];
               }
           }

   в продолжение парсера

   Запостил: Lokich, 29 Октября 2015

   Tweet

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

   • bormand 29.10.2015 17:22 # +4

     Нету пятой колонки? Окей, вернём четвертую, никто же не заметит...

     Ответить

     • inkanus-gray 29.10.2015 17:25 # +1

       Нету первой колонки? Вернём нулевую, никто же не заметит...

       Ответить

       • bormand 29.10.2015 17:34 # +4

         Нету нулевой - вернём минус первую таки наебнёмся с исключением.

         Ответить

         • kegdan 29.10.2015 19:22 # +3

           Нету 6? вернем 5! нет 5? ну тогда наебнемся с исключением

           Ответить

     • 3_14dar 29.10.2015 19:41 # +2

       Нету пятой колонны? Окей, вернём четвертую, никто же не заметит...

       Ответить

       • kegdan 29.10.2015 21:31 # +2

         Нету пятой колонны? Окей, вернём четвертую, никто же не заметит...

         Ответить

         • gost 30.10.2015 21:08 # +2

           Нету пятой колонны? Окей, вернём четвертую, никто же не заметит...

           Ответить

           • 1024-- 30.10.2015 21:17 # +2

             Нету пятой колонны? Окей, вернём четвертую, никто же не заметит...

             Ответить

             • guest 30.10.2015 21:33 # +2

               Нету пятой колонны? Окей, вернём четвертую, никто же не заметит...

               Ответить

               • HiNeX 01.11.2015 17:48 # +2

                 Нету пятой колонны? Окей, вернём четвертую, никто же не заметит...

                 Ответить

                 • Stallman 01.11.2015 18:13 # +2

                   Нету пятой колонны? Окей, вернём четвертую, никто же не заметит...

                   Ответить

                   • bormand 01.11.2015 18:15 # +2

                     Нету пятой колонны? Окей, вернём четвертую, никто же не заметит...

                     Ответить

                     • CHayT 01.11.2015 20:30 # +2

                       C-c-combo breaker!

                       Ответить

                       • bormand 01.11.2015 20:35 # +5

                         CCComboBreakerException in thread 18936
                           at GetColumn
                           at GetColumn
                           at GetColumn
                           at GetColumn
                           ...
                           at GetColumn
                           at main

                         Ответить

   • bormand 29.10.2015 17:23 # +9

     Обобщим идею:

     public TD GetColumn(int i) {
         try {
             return this.Columns[i];
         } catch {
             return GetColumn(i - 1);
         }
     }

     Ответить

     • imihajlov 29.10.2015 17:42 # 0

       Теперь точно не упадет!

       Ответить

       • bormand 29.10.2015 17:48 # +2

         Не, это ж c#. Упадёт или по stack overflow (если компилятор не умеет в хвостовую рекурсию) или по overflow при вычитании (если умеет).

         Ответить

         • inkanus-gray 29.10.2015 18:06 # 0

           Тут в любом случае блоки catch вложенные, значит, стек понадобится хотя бы для исключений.

           Ответить

           • bormand 29.10.2015 18:10 # +1

             > стек понадобится хотя бы для исключений
             Нафига? Мы же рекурсию из catch делаем, а не из try.

             Ответить

         • kegdan 29.10.2015 21:34 # 0

           бесконечность и дааааааааалие....overflow

           Ответить

         • nihau 30.10.2015 12:50 # 0

           Нубский вопрос: разве в шарпах есть оптимизация хвостовой рекурсии?

           Ответить

           • bormand 30.10.2015 18:53 # 0

             Хер знает, если честно, я последний раз шарп палочкой тыкал лет 7 назад...

             Ответить

             • 3.14159265 30.10.2015 18:57 # 0

               Интересно tail-opt как-то поможет в случае try~finally выше?

               Ответить

               • bormand 30.10.2015 20:22 # 0

                 В try - всяко нет. А вот в finally/catch - должен. Там же exception frame уже закрылся.

                 Ответить

                 • 3.14159265 30.10.2015 21:01 # 0

                   Дык, а толку, там же в дерево распидорашивает.
                   Это как просить tail-optim   помочь с квиксортом.

                   Ответить

             • roman-kashitsyn 30.10.2015 18:59 # 0

               Недавно же обсуждали - нету её там.

               Ответить

           • Her 30.10.2015 20:35 # 0

             нет, апликуха падает с unhandled exception of type System.StackOverflowException

             Ответить

             • nihau 31.10.2015 00:46 # 0

               все-таки вопрос был про хвостовую рекурсию, про стэковерфлоу то понятно

               Ответить

               • bormand 31.10.2015 09:21 # 0

                 Ну дык была бы оптимизация - не было бы переполнения стека.

                 Ответить

                 • guest 31.10.2015 14:31 # −4

                   вот борманд всегда дело говорит
                   
                   его даже нахуй посылать неохото, не то что остальных прыщавых тупых дрочунов из этого говносайтинка

                   Ответить

                   • Lokich 02.11.2015 15:27 # +3

                     bormand, перелогинься!

                     Ответить

               • guest 31.10.2015 14:32 # +3

                 >>хвостовую рекурсию
                 ой-ой-ой, какие слова-то мы знаем! Давай еще, про монады скажи.
                 
                 А сам-то небось на PHP и вордпрессе за хлеб с водой наяриваешь

                 Ответить

                 • Stallman 31.10.2015 16:46 # +1

                   В PHP нет оптимизации хвостовой рекурсии, зато есть ini-конфиг и интерфейсы, которые необязательно соблюдать. А еще есть ТЕХПОДДЕРЖКА хостера, которая может поправить PHP.IИI если хорошо попросить, но не будет, потому что ей похуй на пыхоарангутангов. @mysql_fetch_array(@mysql_fetch_array(@m ysql_fetch_array(@mysql_fetch_array(@mys ql_fetch_array(@mysql_fetch_array(@mysql _fetch_array(@mysql_fetch_array(@mysql_f etch_array(@$GLOBALS['BAZA_DANNYH']))))))))) or die("Заплатите мне кто-нибудь!!!11111");

                   Ответить

                   • Vasiliy 31.10.2015 18:10 # +2

                     Хорошо сказал я аж прослезился. А еще в php есть Битрикс

                     Ответить

           • d_fomenok 15.05.2017 12:20 # −5

             показать все, что скрытоЕсть, авторитетно заявляю.

             Ответить

             • TeaBag 15.05.2017 19:48 # −5

               показать все, что скрытоАвторитетно помочился тебе на ебало.

               Ответить

       • Lokich 29.10.2015 17:49 # −1

         >Теперь точно не упадет!
         вы уверены?
         

         GetColumn(-1);

         Ответить

         • imihajlov 29.10.2015 17:54 # +1

           Пока стек не переполнится, точно не упадет.

           Ответить

     • bormand 29.10.2015 18:00 # +5

       public TD GetColumn(int i) {
           try {
               return this.Columns[i];
           } catch {
               return this.Columns[random(0, ColumnCount())];
           }
       }

       Ответить

       • 1024-- 29.10.2015 19:57 # 0

         Надо слить эти два способа:
         > this.Columns[random
         GetColumn(random

         Ответить

     • gost 29.10.2015 19:34 # 0

       Отказоустойчивая система.

       Ответить

   • nihau 30.10.2015 12:50 # 0

     >2015
     >c#
     >this.

     Ответить

     • Lokich 30.10.2015 15:40 # 0

       а this это плохо?

       Ответить

       • kegdan 30.10.2015 17:22 # 0

         this is хорошо

         Ответить

       • nihau 30.10.2015 17:35 # 0

         а нахуя?

         Ответить

         • 1024-- 30.10.2015 18:13 # 0

           Привычней для тех, кто привык к JS.
           А вообще, с "this." можно в теории в лабораторных условиях некоторых ошибок избежать.

           Ответить

           • nihau 30.10.2015 18:15 # 0

             Привычней-хуичней.
             Ты же придерживаешься кодконвеншнов в других языках? Будь добр или иди дальше на джиэсе лупи.

             Ответить

   • 3.14159265 30.10.2015 17:58 # +3

     public TD GetColumn(int i) {
         try {
             return GetColumn(i);
         } finally {
             return GetColumn(i - 1);
         }
     }

     
     
     Теперь точно не упадёт. Не благодарите.

     Ответить

     • nihau 30.10.2015 18:00 # 0

       а где выход из рекурсии? упадет stackoverflowexception

       Ответить

       • 3.14159265 30.10.2015 18:01 # +1

         Не упадёт. Я гарантирую это.Если скомпилится конечно.

         Ответить

         • nihau 30.10.2015 18:03 # 0

           ну тогда лан, сделай пулреквест и в продакшн
           скомпилится офк

           Ответить

           • nihau 30.10.2015 18:05 # +1

             а хуй там: Control cannot leave the body of a finally clause

             Ответить

             • 3.14159265 30.10.2015 18:06 # 0

               Use Java, nihau.
               http://ideone.com/IzzLsd
               stackoverflowexception не будет.

               Ответить

               • nihau 30.10.2015 18:09 # 0

                 в джаве в отличии от шарпов можно так и это компилится и это работает
                 

                 try {
                 return 1;
                     } finally {
                 return 2;
                     }

                 
                 
                 Не знаю как в джаве, но в .нете после 1.0 или 2.0 stackoverflowexception & outofmemoryexception теперь не обрабатываются в try catch блоках (прув ми вронграйт)

                 Ответить

     • 3_14dar 30.10.2015 21:52 # +1

       Пошел в армию, пидар!

       Ответить

       • NrOPb_rAHDOHOB 18.05.2017 17:27 # 0

         Зайди-ка ко мне в медшарагу, а лучше - прямо в лаборантскую.
         Я уже резиновый член припас.

         Ответить

     • TeaBag 19.05.2017 11:45 # −6

       показать все, что скрытоА как это работает?

       Ответить

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

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

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

   Где здесь C++, guest?!

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

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

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