在嵌套循环中创建事件处理程序的效率:我在这里创建了1440个函数吗?

时间:2012-05-22 20:10:52

标签: javascript

我刚开发了一些代码来创建一个24x60的表。我想在<td>上打印每个mouseover的ID:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Untitled Document</title>
<style type="text/css">
table {
    background-color:blue;
}
td {
    width: 2px;
    height: 2px;
    background-color:red;
}
</style>
</head>
<body>
<table id="time-table"></table>
<script type="text/javascript">
var table = document.getElementById( "time-table" );
for ( var r = 0; r < 24; r++ ) {
    var row = document.createElement( "tr" );
    for ( var c = 0; c < 60; c++ ) {
        var td = document.createElement( "td" );
        td.id = "td-" + r + "-" + c;
        td.onmouseover = function ( e ) {
            console.log( this.id );
        }
        row.appendChild( td );
    }
    table.appendChild( row );
}
</script>
</body>
</html>

代码有效,但现在我担心它是否已经过优化?我是否在嵌套循环中创建了1440个事件处理函数?或者JavaScript解释器是否足够聪明,只能创建一个函数并将其分配给1440个<td>元素?

4 个答案:

答案 0 :(得分:13)

不,JavaScript不会进行任何优化(好吧,也许某些实现会这样做,但你不应该依赖它)。你真的在创造那么多功能。

更好地定义一次功能并重复使用它:

var handler = function() {
    console.log(this.id);
}

for ( var r = 0; r < 24; r++ ) {
    var row = document.createElement( "tr" );
    for ( var c = 0; c < 60; c++ ) {
        var td = document.createElement( "td" );
        td.id = "td-" + r + "-" + c;
        td.onmouseover = handler;
        row.appendChild( td );
    }
    table.appendChild( row );
}

或者考虑使用事件委托,即将处理程序绑定到单元格的祖先:

table.onmouseover = function(event) {
    event = event || window.event;
    var target = event.target || event.srcElement;

    if(target.nodeName === 'TD') {
        console.log(target.id);
    }
};

这很有效,因为事件会冒出DOM树,在某些浏览器中可能会提升性能。

了解事件处理的一个很好的资源是the articles at quirksmode.org

答案 1 :(得分:1)

为了安全起见,一个小小的改变:

var myFunc = function (e) {
    console.log( this.id );
};

var table = document.getElementById( "time-table" );
for ( var r = 0; r < 24; r++ ) {
    var row = document.createElement( "tr" );
    for ( var c = 0; c < 60; c++ ) {
        var td = document.createElement( "td" );
        td.id = "td-" + r + "-" + c;
        td.onmouseover = myFunc;
        row.appendChild( td );
    }
    table.appendChild( row );
}

答案 2 :(得分:1)

我建议在表上放置一个事件处理程序,并使用事件冒泡在一个地方处理它:

var table = document.getElementById( "time-table" );
for ( var r = 0; r < 24; r++ ) {
    var row = document.createElement( "tr" );
    for ( var c = 0; c < 60; c++ ) {
        var td = document.createElement( "td" );
        td.id = "td-" + r + "-" + c;
        row.appendChild( td );
    }
    table.appendChild( row );
}
table.addEventListener('mouseover', function(e) {
    console.log(e.target.id);
}, false);

对于旧版本的IE,您可以使用attachEvent而不是addEventListener。

答案 3 :(得分:1)

是的,你为那里的每个元素定义了一个未命名的函数,但是你可以简单地在循环之外定义函数并引用它。

var printMyId = function(e) {
  console.log(e.srcElement.id);
};
var table = document.getElementById("time-table");
for (var r = 0; r < 24; r++) {
  var row = document.createElement("tr");
  for (var c = 0; c < 60; c++) {
    var td = document.createElement("td");
    td.id = "td-" + r + "-" + c;
    td.onmouseover = printMyId;
    row.appendChild(td);
  }
  table.appendChild(row);
}