Говнокод #15580 — JavaScript

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

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

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

+151

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

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

Запостил:

farit_slv, 26 Марта 2014

Комментарии (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');
Ответить

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

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

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

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