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

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

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

1. PHP (5703)
2. C++ (2737)
3. Куча (2343)
4. JavaScript (2028)
5. C# (1928)
6. Java (1502)
7. Си (1108)
8. Pascal (647)
9. Python (583)
10. 1C (535)
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 (30)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

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

   +135.9

   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

   public class PlayerWindow
   {
       private readonly ScheduleManager _scheduleManager = new ScheduleManager();
   
   ...
   
       private void OpenSchedule(string timesheetFilename)
       {
           if (!_scheduleManager.OpenSchedule(timesheetFilename))
           {
               _isOpen = false;
               return;
           }
   
           _isOpen = false;
       }
   }

   Собственное творчество. ^_^

   Запостил: spbAngel, 29 Января 2010

   Tweet

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

   • Altravert 30.01.2010 05:43 # 0

     Та не говнокод это.. Ну, спутал true с false, и че? Не интересно..

     Ответить

     • spbAngel 30.01.2010 06:41 # 0

       Ну тогда можно было ограничиться без ветвления, чем-то подобным: _isOpen = _scheduleManager...
       Да и я зря не уточнил - это единственное место, где "изменяется" _isOpen. И при этом, программа работает нормально.

       Ответить

       • spbAngel 30.01.2010 07:09 # +1

         Господи! Что я делаю... Пытаюсь доказать, что пишу тру-говнокод. о_О

         Ответить

         • Altravert 30.01.2010 07:14 # 0

           Тогда я мог бы расценить как "сначала этот атрибут был нужен, потом код соответственно изменился и переменная стала не нужна..." блаблабла. Короче, ниче странного, особенного или страшного.

           Ответить

           • spbAngel 30.01.2010 07:23 # 0

             атрибут? o_O
             Видимо, всем пора спать. ^_^

             Ответить

             • Altravert 31.01.2010 08:05 # 0

               Это на меня семестр проектирования uml херово действует) Я имел ввиду член класса "_isOpen"

               Ответить

         • wuhu 14.05.2010 22:35 # 0

           ну и зачем было это постить? не понимаю

           Ответить

   • Altravert 30.01.2010 07:14 # −7

     показать все, что скрытоГде тут Java?
     Ы))

     Ответить

     • guest 30.01.2010 18:59 # +1

       выше по генеалогическому дереву))

       Ответить

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

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

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

   Помни, guest, за тобой могут следить!

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

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

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