2

我在试图弄清楚为什么我动态添加的 onclick 事件没有做任何事情时遇到了问题。我已经搜索了很多网站，但我尝试过的都没有。了解自己这可能是某种愚蠢的错误，但我真的很想知道我做错了什么。这是我的代码的一部分，包括相关功能：

        function ChangeNumber(line){   //this is just a test function so far :)
            document.getElementById('maincol').innerHTML += line + "<br/>"; //just adds "something to the end of a div"
        }

        function ChangeSize()
        {
            var rows, cols;
            rows = document.getElementById('rows').value;
            cols = document.getElementById('cols').value;

            var tbody = document.getElementById('model');
            tbody.innerHTML = "";

            for (var i = 0; i < rows; i++){
                var tr = document.createElement('tr');
                for (var j = 0; j < cols; j++){
                    var td = document.createElement('td');
                    td.setAttribute('name', (i * cols) + (j + 1));
                    td.onclick = function() {ChangeNumber('something'); };
                    td.innerHTML = "0";
                    tr.appendChild(td);
                }
                tbody.appendChild(tr);
            }

        }

表的创建工作正常，从静态创建的 onclick 调用函数也是如此，ChangeNumber()但是当我单击动态创建的 td 时没有任何反应。有人可以向我澄清这个问题吗？

4

5 回答 5

1

一个快速的 JSFiddle 表明基本方法很好——至少在 Chromium 中是这样。

tbody_x = document.getElementById('x');
tbody_x.innerHTML = '';

for (row = 0; row < 5; row++)
{
  tr = document.createElement('tr');

  for (col = 0; col < 10; col++) 
  {
    td = document.createElement('td');
    td.setAttribute('name', 'r' + row + 'c' + col);
    td.onclick = function() { ChangeNumber('hi'); };
    td.innerHTML = '0';
    tr.appendChild(td);
  }

  tbody_x.appendChild(tr);
}

必须打破别的东西。（尝试启动 Firebug 或类似工具并在控制台中查找 JS 错误。）

于 2013-10-19T20:30:39.923 回答
1

您还可以使用：

td.onclick = ChangeNumber;

这ChangeNumber是一个也可以绑定到点击的函数，如果你把这个函数放在里面，你可以为每个点击创建一个自定义函数。

我知道他面临什么问题，只是 onclick 函数不会弹出，它不显示任何日志，如果你使用它，td.onclick = ChangeNumber();那么它只会在加载这个脚本时触发这个函数一次，

于 2016-03-22T08:49:28.987 回答
0

你的方法似乎很好，请在这里找到：（jsfiddle.net/fa9SV/）

请交叉检查文档中是否存在所有 ID。希望这可以帮助！

于 2013-10-19T20:54:01.833 回答
0

创建 HTML 元素时，您必须使用 setAttribute 来设置它。此外，您必须将其设为字符串。

所以，像：

td.setAttribute('onclick', 'function(){change("Something")}');

或者，你可以这样写：

td.setAttribute('onclick', 'changeSomething();');

然后在全局对象的某处定义脚本，如下所示：

changeSomething = function() {
// do the change
}

第三种选择是使用事件库。然后你就可以做类似的事情：

Event.on(td, 'click', changeSomething);

看看这个问题，看看我的意思。

PS这超出了这个问题的范围，但最好不要在全局对象上绑定东西。使用模块或其他东西。

于 2013-10-19T20:17:39.890 回答
0

您必须将 设置onclick为属性。所以，而不是

td.onclick = function() {ChangeNumber('something'); };

你会想要类似的东西

td.setAttribute('onclick', "function() {ChangeNumber('something');}");

于 2013-10-19T20:09:47.480 回答