0
function cet() {
    for (var j = 1; j < 15; ++j) {
        var el = document.getElementsByTagName("li")[j]
        el.onmouseover = function () {
            el.childNodes[1].style.display = "block"
        }
    }
}
1
  • 6
    для чего этот код и зачем его улучшать?
    – sercxjo
    7 июн 2012 в 16:36

1 ответ 1

4

для чего этот код и зачем его улучшать?

вот и первый ответ как улучшить код: сделать его понятным для окружающих

второе: вы 15 раз запрашиваете NodeList - никогда в циклах не запрашивайте NodeList!

var elems = document.getElementsByTagName("li");
for (var j = elems.length; j >= 0; j--) {
    var el = elems[j];
    el.onmouseover = function () {
        el.childNodes[1].style.display = "block"
    }
}

UPD: переформулировал ответ

2
  • Надо бы наверное переформулировать, т.к. elems.item(j) тоже обращение к DOM. В оригинальном говнокоде запрашивался NodeList по условию, что очень толсто.
    – karmadro4
    7 июн 2012 в 19:45
  • плохо знаком с принципами работы .getElementsByTagName, аналогичная операция на jQuery закешировала бы объект и обращения в each, например, не были бы обращениями к DOM
    – Spectre
    8 июн 2012 в 5:47

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.