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

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

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

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

1. JavaScript / Говнокод #15580

   +151

   1. 1

   var $resetBtn = $("<a class=\"filter-reset-btn resetBtn\"/>");

   Это нормально вообще?

   Запостил: farit_slv, 26 Марта 2014

   Tweet

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

   • KonardinoHuyardino 26.03.2014 16:54 # −9

     показать все, что скрытоvanished

     Ответить

     • farit_slv 26.03.2014 16:55 # −2

       это было в js...

       Ответить

       • KonardinoHuyardino 26.03.2014 16:56 # −7

         показать все, что скрытоvanished

         Ответить

         • KonardinoHuyardino 26.03.2014 16:59 # −7

           показать все, что скрытоvanished

           Ответить

           • farit_slv 26.03.2014 17:02 # −1

             я не знаю =)

             Ответить

         • farit_slv 26.03.2014 17:01 # −3

           Суть в том это было в js файле, а почему бы не сделать например вот так: var $resetBtn = $("a.filter-reset-btn.esetBtn"); ??

           Ответить

           • KonardinoHuyardino 26.03.2014 17:03 # −6

             показать все, что скрытоvanished

             Ответить

           • roman-kashitsyn 26.03.2014 17:10 # +4

             > $("a.filter-reset-btn.esetBtn");
             Ваш код и код в топике делает разные вещи
             В топике создаётся новый нод дерева
             Ваш код ищет существующие селектором

             Ответить

             • farit_slv 26.03.2014 17:14 # −2

               Спасибо, теперь понял.

               Ответить

             • farit_slv 26.03.2014 17:15 # −2

               А это тоже нормально?
               $button = $div.find(".resetBtn");
               
               if ($button[0].onclick === null){
               $button[0].onclick = function(){
               var $select = $("#"+inst.id),
               $first = $select.children().first();
               $select.selectbox("change", $first.val(), $first.text());
               }
               }

               Ответить

               • Lokich 26.03.2014 18:30 # 0

                 ну, говно конечно какое-то, но особого криминала я не вижу :)

                 Ответить

               • Vindicar 26.03.2014 19:08 # 0

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

                 Ответить

   • eandr67 27.03.2014 11:32 # +2

     Абсолютно стандартное использование jQuery. Единственное: стоит строку заключить в одиночные кавычки - чтобы избавиться от экранирования двойных кавычек внутри. Ну и на мой взгляд, более красиво: var $resetBtn = $('<a>').addClass('filter-reset-btn resetBtn');

     Ответить

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

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

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

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

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

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

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