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

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

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

1. PHP (5703)
2. C++ (2739)
3. Куча (2337)
4. JavaScript (2030)
5. C# (1929)
6. Java (1503)
7. Си (1102)
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 (31)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   +126

   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

   if (_documentsCount < 0)
               {
                   var docs = new HashSet<int>();
   
                   _manager.WalkThroughConcepts(delegate(Concept concept, CancelEventArgs cargs)
                                                    {
                                                        foreach (var doc in concept.Documents)
                                                            docs.Add(doc);
                                                    });
   
                   _documentsCount = docs.Count;
               }
   
               return _documentsCount;

   Как сосчитать количество документов в дереве? очень просто - надо всех их добавить в один список (и для этого сделать цикл в каждом узле, разумеется) а потом вернуть количество элементов в списке.

   Запостил: eval_2009, 06 Января 2011

   Tweet

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

   • Lure Of Chaos 06.01.2011 19:28 # 0

     мда. просто посчитать, конечно же, слишком просто для такого крутого кодера.

     Ответить

     • eval_2009 06.01.2011 19:35 # 0

       На самом деле все еще хуже - у человека были проблемы с дубликатами - он некорректно создавал структуру, и некоторые документы принадлежали нескольким узлам. А hashset не кидает ошибок ежели в него пытаться добавить уже существующий элемент. То есть имеем не только медленный код (счет идет на десятки а то и сотню тысяч) но и сокрытие существующей проблемы вместо ее исправления.

       Ответить

       • Lure Of Chaos 06.01.2011 19:38 # 0

         > hashset не кидает ошибок ежели в него пытаться добавить уже существующий элемент
         а старый при этом не выкинется?

         Ответить

         • eval_2009 06.01.2011 19:45 # 0

           Нет
           http://msdn.microsoft.com/en-us/library/bb353005.aspx
           полезный тип если применять правильно

           Ответить

           • Lure Of Chaos 06.01.2011 19:46 # 0

             > true if the element is added to the HashSet(Of T) object; false if the element is already present.
             
             все правильно, новый не добавится. Т.е. дубликатов не будет

             Ответить

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

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

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

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

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

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

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