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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2339)
4. JavaScript (2030)
5. C# (1930)
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# / Говнокод #10345

   +93

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

   for (int i = 0; i < wastePeriods.Length; i++)
       {
             wastePeriods = 0;
             docPeriods = 0;
             wastePeriodsYear = 0;
       }

   Диггеры-рефакторщики раскопали очередной оригинальный цикл.
   wastePeriods - массив с over300+ записями.
   docPeriods, wastePeriodsYear - тоже массивы

   Запостил: UfoDriver, 22 Мая 2012

   Tweet

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

   • DukeGonzo 22.05.2012 16:38 # 0

     Массивы типа byte =)

     Ответить

   • Steve_Brown 22.05.2012 16:40 # +4

     Не понял? Как массиву можно присвоить целое? (разве что переменная - dynamic, тогда вылетит на второй проверке условия цикла)

     Ответить

     • DukeGonzo 22.05.2012 17:10 # 0

       Работать естественно не будет, в исходном коде пропущены скобки и индекс. К сожалению, редактирование на этом сайте не предусмотрено...

       Ответить

       • roman-kashitsyn 22.05.2012 17:19 # 0

         Бида в том, что их размеры могут не совпадать? Или в том, что не использовали какой-нибудь Fill(array, elem)?

         Ответить

         • DukeGonzo 22.05.2012 17:44 # 0

           И в том, и в другом.

           Ответить

           • vistefan 22.05.2012 18:32 # +1

             И в третьем: надо было пихнуть все эти массивы в такой контекст, где они бы инициализировались нулями автоматически и не требовали рукопашного обнуления. Данная архитектура - говно.

             Ответить

             • DukeGonzo 24.05.2012 15:41 # +1

               В том проекте - всё говно. Сейчас с нуля переписываем

               Ответить

   • absolut 22.05.2012 17:19 # +6

     фейк

     Ответить

     • DukeGonzo 22.05.2012 17:43 # +1

       Не злонамеренный, мессенджер обрезал скобки =(

       Ответить

       • bormand 22.05.2012 18:38 # −1

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

         a [ i ] = 2;

         Ответить

         • vistefan 22.05.2012 18:57 # +2

           Что за извращенство, в мессенждерах код транспортировать? http://pastebin.com
           

           Ответить

           • bormand 22.05.2012 19:08 # +2

             Я не знаю, зачем я это делал :)

             Ответить

   • exile 31.05.2012 08:56 # +1

     for(int i =0; i <= 1kk; ++i)
     {
         //bla bla bla
         --i;
     }

     Ответить

     • vistefan 31.05.2012 08:59 # +1

       даже пусть в конце

       i -= 2;

       что бы убегать ещё дальше от конца цикла...

       Ответить

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

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

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

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

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

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

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