仅调用多个 select 元素的最后附加的 onchange 函数

Question

我正在以编程方式创建多个 select 元素，每个 select 元素中有多个选项。我想以编程方式为每个 select 元素分配自己的 onchange 函数，该函数会提醒我 select 元素的相应 ID。

$(document).ready(function () {

    // Headers indicate how many select elements I need.
    // In this example, they are only used to initialize a unique select.id
    for (var ndx = 0; ndx < headers.length; ndx++) {
        var header = headers[ndx];
        const rowName = header.innerHTML;
        var select = document.createElement("select");
        var options = ["Contains", "Equals", "Starts with", "More than", "Less than", "Between", "Empty",
            "Doesn't contain", "Doesn't equal", "Doesn't start with", "Is not more than", "Is not between", "Is not empty"];

        select.id = rowName + "-select";
        options.forEach(function(option) {
            var element = document.createElement("option");
            element.value = option;
            element.innerHTML = option;
            select.appendChild(element);
        });
        select.onchange = function () {
            alert("select.id: " + select.id);
        }
    }
}

但是，在更改任何 select 元素的选项后，只有最后一个 select 元素的 ID 会显示在警报 中。这可能是我不熟悉的 javascript 引用问题吗？感谢您的帮助。

Answer 1

javascript 中的变量被提升到声明它们的函数范围的开头。实际上，这意味着在每次迭代中，您都将变量 select 替换为新的 select 框。这意味着所有侦听器都将引用相同的 select 变量，该变量在循环结束时将包含对最后创建的 select 的引用。

解决方案

将侦听器注册包装在 IIFE（立即调用的函数表达式）中，将其作为参数传递 select。这样 select 的值将仅绑定到当前迭代。

(function(select) {
    select.onchange = function() {
        alert(select.id);
    }
})(select);

编辑：

实现侦听器的更正确方法是使用传递给它的事件对象：

select.onchange = function(e) {
    alert(e.target.id); //e.target refers to the element that originated the event
}

但我也将保留上述答案，因为您的代码中也存在与 hoisting/scoping 相关的问题，这是一个非常常见的错误。

仅调用多个 select 元素的最后附加的 onchange 函数

Only the last-appended onchange function for multiple select elements is called

html

javascript

onchange

node.js

web