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

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

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

1. PHP (5705)
2. C++ (2739)
3. Куча (2357)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1115)
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# / Говнокод #11760

   +140

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11
   12. 12

   private void SendFile(string root, string url, DateTime dateTimeBegin)
   {
   	if ( ... isSend ... )
   	{
   		...MoveToArchive();
   		...Dispose();
   	}
   	else
   	{
   		SendFile(root, url, dateTimeBegin);
   	}
   }

   Ну тут всё ясно...
   Если файлики отсылаются нормально, то всё работает хорошо.
   Зато если файлики не отсылаются, то начинается злая рекурсия, которая постепенно сжирает всю память :).

   Запостил: ddv_demon, 13 Сентября 2012

   Tweet

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

   • Steve_Brown 13.09.2012 10:49 # 0

     void SomeMethod(...)
     {
       try
       {
         ...
       }
       catch
       {
         SomeMethod(...);
       }
     }

     Ответить

     • abatishchev 13.09.2012 11:34 # +1

       bool SomeMethod(...)
       {
         ...
       }
       
       while (!SomeMethod(...))
       {
         Thread.Sleep(1);
       }

       Ответить

       • govnomonad 13.09.2012 15:19 # +3

         while (!SomeMethod(...).AssParallel())
         {
           Thread.Sleep(1);
         }

         Ответить

         • wvxvw 13.09.2012 15:48 # +1

           Thread.Sleep(ThreadSleepOptions.Alone);

           Ответить

   • xredor 16.04.2013 11:06 # +1

     Многие компиляторы оптимизируют хвостовую рекурсию. Это развернется в бесконечный цикл.

     Ответить

     • guest 16.04.2013 11:12 # 0

       Если SendFile виртуальная функция, то не развернется

       Ответить

       • roman-kashitsyn 16.04.2013 11:16 # 0

         В сишарпике приватные функции виртуальны? Не верю.
         Резон так делать есть только в C++.

         Ответить

         • guest 16.04.2013 12:43 # 0

           Виртуальны все функции в сишарпике.

           Ответить

           • roman-kashitsyn 16.04.2013 12:51 # +1

             И как её оверрайдить?

             Ответить

           • 3.14159265 16.04.2013 14:25 # 0

             1) не путаем с жабой.
             > все функции
             2) лямбды виртуальны, орлы?

             Ответить

             • roman-kashitsyn 16.04.2013 14:37 # +1

               В жабе как раз невиртуальны приватные и статические функции, потому и удивился.
               Лямбды небось экстендят что-то вроде IFunc и потому виртуальны.

               Ответить

               • guest 16.04.2013 14:51 # 0

                 > IFunc
                 Action
                 Func

                 Ответить

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

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

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

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

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

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

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