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

    +169

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    if(navigator.appName == "Microsoft Internet Explorer") {
	for(var i=0;i<$('.product_documents').length;i++)  if( ( (firstLaunch_onChangeDocs) && ($('.product_documents')[i].selectedIndex==0)) || (!firstLaunch_onChangeDocs) )
	{
	  document.getElementById($('.product_documents')[i].id).innerHTML = '';
	  document.getElementById($('.product_documents')[i].id).outerHTML = document.getElementById($('.product_documents')[i].id).outerHTML.replace("</SELECT>", jsInternalDocuments + '</select>')
	}
  } else
	for(var i=0;i<$('.product_documents').length;i++)  if( ( (firstLaunch_onChangeDocs) && ($('.product_documents')[i].selectedIndex==0)) || (!firstLaunch_onChangeDocs) )
	  $('.product_documents')[i].innerHTML = jsInternalDocuments;

    Очень альтернативная техника использования jquery.

    Запостил: clauclauclau, 06 Декабря 2012

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

    • ava Vindicar 06.12.2012 09:20 # +7

      Пять поисков элементов с классом product_documents за каждую итерацию для IE и три - для остальных.
      O(N^2) на пустом месте - это волшебно.
      Ответить

    • ava Vasiliy 06.12.2012 11:46 # +1

      >document.getElementById($('.product_doc uments')[i].id).innerHTML = '';
      За это вырезать всю семь до 5 колена. Это же просто не возможно.
      Ответить

    • ava torbasow 06.12.2012 15:08 # +1

      Юмористическое программирование? Трудно поверить, что такое писали всерьёз.
      Ответить

      • ava clauclauclau 06.12.2012 19:44 # 0

        Увы, всерьез. Это b2b сайт крупного производителя промышленной продукции. Исключительные юмористы писали.
        Ответить

      • ava Vindicar 07.12.2012 00:31 # 0

        По-моему, человек заучил что $('.product_documents') - это список элементов с классом product_documents. Вот он и работает с ним как со списком. А то, что этот список строится заново при каждом вызове, осталось за кадром.
        Т.е. имеем знание внешних атрибутов без понимания сути происходящего.
        Ответить

        • ava torbasow 07.12.2012 05:49 # 0

          А он не кэшируется? Я не очень в курсе начинки jQuery, но меня смутило намерение брать по идентификатору элемент, ссылка на который уже получена.
          Ответить

          • ava Vindicar 07.12.2012 08:37 # +1

            . - это не идентификатор, это класс. jQuery позволяет использовать CSS нотацию при запросах.
            И нет, он не кэшируется. DOM-структура может поменяться, а отслеживать это... даже если можно, дешевле просто строить каждый раз список заново.
            Ответить

            • ava torbasow 07.12.2012 09:40 # +1

              «. - это не идентификатор, это класс.»
              Нет, я имею в виду то, что дальше делается: у узла берётся айдишник и по нему мы получаем тот же узел. Если автора смущает, что сначала он получает не совсем узел, а jQuery-объект, и что с ним делать, он не знает, так просто .get(0) же.
              Ответить

              • ava Vindicar 07.12.2012 12:03 # 0

                А, сразу не вкурил. И впрямь второй уровень говна.
                Ответить

            • ava PascalGovno 07.12.2012 09:47 # 0

              Поправка: не jQuery, а SizzleJS

              > а отслеживать это
              https://developer.mozilla.org/en-US/docs/DOM/Mutation_events
              Ответить

              • ava Vindicar 07.12.2012 12:05 # 0

                И городить такую красоту просто ради того чтобы скрыть фундаментальные ошибки чьего-то говнокода?
                Не, ну можно конечно... Но нужно ли?
                Ответить

                • ava mac-t 09.12.2012 00:03 # 0

                  >>фундаментальные ошибки чьего-то говнокода?
                  JQuery? Маздай!
                  Ответить

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