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

    +163

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    var z=fggh.getElementsByTagName("*");
for(iii=0; iii<z.length; z++)
if(z[iii].parentNode==fggh){
//z[iii] - непосредственный потомок fggh, и мы с ним что-то делаем!
…
}

    Раз народ решил, что я незаслуженно обижаю юниора, то вот вам ещё более шокирующий пример подобного кода от программиста, с которым мы вместе работали несколько лет назад в одной московской фирме. Тут, к сожалению, воспроизвожу по памяти, но очень близко к оригиналу.

    Запостил: torbasow, 04 Апреля 2013

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

    • ava WGH 04.04.2013 20:02 # +6

      Зато чистый DOM, безо всяких jQuery!
      Ответить

      • ava Vindicar 05.04.2013 09:36 # 0

        Ну если быть придирчивым, то z[iii].childNodes - это тоже чистый DOM. =)
        Ответить

        • ava guest 05.04.2013 11:55 # +1

          Если уж на то пошло, то для полного соответствия надо еще фильтровать текстовые узлы и прочие комментарии.
          Ответить

          • ava torbasow 05.04.2013 15:36 # 0

            Вот я, к сожалению, уже не помню, было ли там что-нибудь такое… Зато до сих пор с содроганием вспоминаю неизменный метод выбора наименований переменных путём случайной пробежки пальцами по клавиатуре.
            Ответить

            • ava WGH 05.04.2013 16:44 # +1

              Как раз так фильтровать не нужно, селектор "левые" узлы не поймает.
              А если писать через childNodes - тогда да, нужно.
              Ответить

      • ava torbasow 05.04.2013 15:34 # 0

        Да, автор кода был почему-то поборником чистого Javascript.
        Ответить

        • ava scriptin 06.04.2013 01:22 # 0

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

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