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

    +145

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    var detailsTable = document.getElementById("detailsTable").tBodies[0];
    var childs = [], j=0;
    for (var i = 0; i < detailsTable.childNodes.length; i++)
    	if  (detailsTable.childNodes[i].nodeType == 1){
    		childs[j++] = detailsTable.childNodes[i];
    	}
    childs[0].childNodes[1].innerHTML = ''

    Наткнулся в нашем проекте. Так юниор получал доступ к ячейкам таблицы. Видно, он не знал про коллекции rows и cells.

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

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

    • Ему-то хоть сказали что есть rows и cells? Или сразу на ГК постить?
      Ответить
    • Я думаю, теперь он уже знает. Он видел, как это у меня делается.
      Ответить
      • Из контекста этого не видно, и создается впечатление, что вы без причины гнобите новичка.
        Ответить
        • При чём тут вообще фантазии на тему отношений в нашем коллективе?
          Ответить
          • Просто, код написанный джуниорами и студентами, по умолчанию считается не достойным говнокода. Это как если бы искусствоведы критиковали рисунки детсадовцев, смысла мало. Вот если бы вы написали, что это написал программист с опытом в 100500 лет, позиционирующий себя как гуру javascript, тогда бы поймали кучу плюсов. Ибо не что так не повышает ЧСВ посетителей сего ресурса, как говнокод от "профессионалов" ))
            Ответить
            • > Ибо не что так не повышает
              Ничто же.
              Ответить
            • В самом деле? Сейчас проверим. Ибо был аналогичный случай…
              Ответить
              • Ну вот же.
                Ответить
                • Поразительно. А ведь мне казалось, что эти отрывки кода смешны одним и тем же…
                  Ответить
                  • Комментарий к коду тоже имеет значение. По нему определяется контекст. @santa_microbe правильно говорит.
                    Ответить
    • не смертельно
      Ответить

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