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

    +207.3

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    $('div').each(function () {
if ($(this).attr('id') == 'blok') {
$(this).html('');
}
});

    Хочется взять и уебать…

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

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

    • ava striker 12.01.2010 18:14 # +1

      код реален? откуда?
      p.s. плюсанул!
      Ответить

    • ava Dreyk 12.01.2010 19:51 # +1

      Бугога)
      Это кем надо быть...

      +1 откуда код?
      Ответить

    • ava TarasB 13.01.2010 01:20 # +2

      Можно пояснения для нубов?
      Ответить

    • ava vov4ik 13.01.2010 02:57 # 0

      мда, поржал )))
      Ответить

    • ava Cr@ZyBoY 13.01.2010 16:22 # +7 

      document.getElementById("blok").innerHTML = "";

      - а так даже jquery не нужен. Люди совсем программировать разучились.
      Ответить

      • ava guest 13.01.2010 18:39 # −4

        Ай малаца!
        Ответить

      • ava guest 13.01.2010 19:02 # 0

        Это очень правильно. Надо иметь ввиду еще, что в jQuery много вещей через жопу сделано. Например, для получения текстового содержимого элемента (метод text()) не используется короткий кроссбраузерный путь: elem.textContent || elem.innerText; А делается перебор всех чайлдов и проверка это текстовая нода или нет, и соответсвенно собирается строка из этого. Такой подход глючит например, если у элемента есть чайлд элемент object или canvas.
        Ответить

        • ava guest 14.01.2010 10:45 # 0

          Видимо на то есть свои причины.
          Впрочем, я сам видел такой говнокод: 
          merge: function( first, second ) {
        // We have to loop this way because IE & Opera overwrite the length
        // expando of getElementsByTagName
        var i = 0, elem, pos = first.length;
        // Also, we need to make sure that the correct elements are being returned
        // (IE returns comment nodes in a '*' query)
        if ( !jQuery.support.getAll ) {
            while ( (elem = second[ i++ ]) != null )
                if ( elem.nodeType != 8 )
                    first[ pos++ ] = elem;

        } else
            while ( (elem = second[ i++ ]) != null )
                first[ pos++ ] = elem;

        return first;
    }
          Ответить

        • ava guest 15.01.2010 11:11 # 0

          Сделал бы всем лучше, если бы написал патч.
          Ответить

      • ava guest 14.01.2010 05:49 # +5

        Айайай, а если элемента id = blok не существует?

        Мне кажется, что на самом деле у в данной задаче был не один элемент с id = blok, что, безусловно, тоже неправильно. Перебор всех div'ов на странице, конечно, брутальный метод.
        Ответить

        • ava guest 14.01.2010 15:57 # +1

          А если элемента нет, то если по нормальному, то должна вывестись ошибка. И если у человека в коде несколько элементов с одним id, то это чисто его проблемы, что он не знает стандартов.
          Ответить

        • ava guest 04.02.2010 15:07 # +1

          id может быть только один.
          Ответить

        • ava guest 05.02.2010 08:08 # 0

          Rules with overly qualified selectors
          For example:

          ul#top_blue_nav {...}
          form#UserLogin {...}

          ID selectors are unique by definition. Including tag or class qualifiers just adds redundant information that needs to be evaluated needlessly.
          Ответить

      • ava Dreyk 14.01.2010 15:31 # +1

        Угу, даешь проекты на чистом JS, долой всякие джыквери мутулзы и эксты с ними же.
        Ответить

      • ava Johnny 16.01.2010 00:28 # 0

        собственно, скорее всего аффтор подразумевал поиск нескольких дивов, геБайОйди не выручит...
        Ответить

        • ava guest 16.01.2010 17:34 # +1

          А использовать классы вера не позволяет?
          Ответить

          • ava Johnny 17.01.2010 01:36 # 0

            А меня то че спрашиваешь? Это к автору вопрос.
            Ответить

    • ava sbb 14.01.2010 03:00 # −2

      Вспомнился бойан с 10 страницы:
      $('div[id="myid"]')
      Ответить

    • ava guest 26.01.2010 16:11 # +2

      Из той же серии:

      $('*).each(function () {
if ($(this).attr('tagName') == 'div') {
if ($(this).attr('id') == 'blok') {
$(this).html('');
}}
});
      Ответить

    • ava guest 01.02.2010 02:38 # 0

      ыыы я так пишу )
      Ответить

    • ava guest 18.02.2010 13:04 # 0

      тут правильнее юзать
      $('#blok').empty();

      в этом случае "Other jQuery constructs such as data or event handlers are erased as well."
      зы плюсанул))
      Ответить

    • ava guest 24.05.2010 17:59 # −1

      А может
      var block = $('#blok');
      if(block.length)
      {

      }
      Ответить

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