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 ответ
для чего этот код и зачем его улучшать?
вот и первый ответ как улучшить код: сделать его понятным для окружающих
второе: вы 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: переформулировал ответ
-
Надо бы наверное переформулировать, т.к.
elems.item(j)
тоже обращение к DOM. В оригинальном говнокоде запрашивался NodeList по условию, что очень толсто. 7 июн 2012 в 19:45 -
плохо знаком с принципами работы
.getElementsByTagName
, аналогичная операция на jQuery закешировала бы объект и обращения вeach
, например, не были бы обращениями к DOM– Spectre8 июн 2012 в 5:47