4

我正在遍历一组 16 个 id 并eventListener为每个 id 分配一个。我想向我的 php 文件发送一个数字（第一个 id 为 1，第二个 id 为 2，等等），但它似乎i比我想要的更动态。每个 id 发送17.

klasses.forEach(function(klass){
    var svgElement = svgDoc.getElementById(klass); //get the inner element by id
    svgElement.addEventListener("mouseup",function(){
        $.ajax({
            type: "POST",
            url: "buildService.php",
            data: { "service" : i}
        }).done(function(msg){
            alert(lameArray[i]);
            $("#modalSpan").html(msg);
            $("#modmodal").modal();
        });
    });
    i++;
});

如何将每个设置为特定数字？我也试过：

var lameArray = [1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16];
...
data: { "service" : lameArray[i]}
4

3 回答 3

10

是什么i？问题是它i是一个全局变量，或者是forEach循环之外的变量，所以当mouseup事件被触发时，使用的是iin that istant 的值，而不是定义事件侦听器时的值。

请注意，由于您使用forEach的是，回调函数实际上是使用第二个参数调用的，即计数器。所以你可以使用：

klasses.forEach(function(klass, i) {
    ...
});

现在，i是forEach范围内的一个变量，可以满足您的目的。（forEach也使用第三个参数调用回调函数，即集合本身 -klasses在您的情况下。）

注意：由于您使用的是 jQuery，因此您应该使用更“类似 jQuery”的样式进行编码。所以改变你的代码是这样的：

$.each(klasses, function(i, klass) {
    $("#" + klass).mouseup(function(){
        $.ajax({
            type: "POST",
            url: "buildService.php",
            data: {service: i + 1}
            ...
        });
    });
});
于 2012-07-11T22:08:10.790 回答
1

试试这个：

klasses.forEach(function(klass){
    (function(i) {
        var svgElement = svgDoc.getElementById(klass); //get the inner element by id
        svgElement.addEventListener("mouseup",function(){
            $.ajax({
                type: "POST",
                url: "buildService.php",
                data: { "service" : i}
            }).done(function(msg){
                alert(lameArray[i]);
                $("#modalSpan").html(msg);
                $("#modmodal").modal();
            });
        });
    })(i);
    i++;
});
于 2012-07-11T22:07:22.097 回答
1

当您需要迭代器索引时，不要使用 forEach。并避免使用 JQuery 的 .each。在绝大多数情况下它是完全没有必要的，它在每次迭代时都会触发一个回调函数，所以它在 IE 中要慢得多。您可以使用 while 编写一个完全惰性/紧凑的循环。

var outerI = klasses.length;

while(outerI--){
    (function(i){
        i+=1;//doesn't affect outerI and you wanted 1-length so we add one.
        //I would personally just add 1 but it also adds clarity to the example

        //crap inside your forEach loop but without the i++
    })(outerI)
}

发生了什么：您告诉事件侦听器从外部范围引用 i。所以无论我在那个事件开始时是什么，都是你得到的。

解决方案：将 i 的值传递到函数的范围内，它成为本地 var。带括号的业务只是定义、评估和执行匿名函数的一种懒惰方式，看起来像是一步。该函数通过第一个括号进行评估（使其可触发），因此第二个集合就像您放入函数定义的内部“i”参数中的 arg。您基本上通过将其传递给新的本地变量来锁定您想要的值。

注意 while 循环：while(0) 评估为假，停止循环。如果您考虑一下，这很奇怪，因为长度为 1 比您想要的少一。使用 while(i--) 然而 i 被评估，然后 i 递减，所以在块内你得到长度 1 到 0，这对于数组表示法来说是完美的。要在其他运算符击中它之前减少 i ，您通常会这样做 --i 但它在惰性/高效的 while 循环中很方便。

于 2012-07-11T23:04:27.010 回答