0

我正在创建一个“幻灯片”，如果你会这样称呼它，用于页面导航。

我正在尝试使用 forloop 将事件侦听器添加到所有菜单项。它似乎不起作用。基本上我有一个基于参数显示新页面的功能。

function slidePFer(page_side){

通过像这样手动设置事件监听器，这可以完美地工作：

showPF  = document.getElementsByClassName('showPF');

showPF[0].addEventListener('click', function(){ slidePFer(0); }, false);
showPF[1].addEventListener('click', function(){ slidePFer(1); }, false);
showPF[2].addEventListener('click', function(){ slidePFer(2); }, false);
showPF[3].addEventListener('click', function(){ slidePFer(3); }, false);

然而，这看起来非常混乱。所以我试图用不同的循环来运行它。而且我无法让它工作，这是我尝试过的更有希望的循环之一：

for(var i=0, n=showPF.length; i<n; i++){
    showPF[i].addEventListener('click', function(){ slidePFer(i); }, false);
}

任何人都可以看到我似乎不明白的内容吗？

4

3 回答 3

1

从函数（i）中删除 i 也许这会解决它

于 2013-10-17T10:00:22.833 回答
0

使用 jQuery ( http://jquery.com )，您可以这样简化：

html

<img slideNo="0" class="showPF" src="slide_0_preview.jpg" />
<img slideNo="1" class="showPF" src="slide_1_preview.jpg" />
<img slideNo="2" class="showPF" src="slide_2_preview.jpg" />

javascript

<script type="text/javascript" src="http://code.jquery.com/jquery-1.10.2.min.js"></script>
<script type="text/javascript">
    $(document).ready(function() {
        $('.showPF').click(function() {
            slidePFer($(this).attr('slideNo'));
        });
    });
    function slidePFer(slideNo) {
        // todo
    }
</script>
于 2013-10-17T10:09:34.907 回答
0

我终于为任何感兴趣的人找到了解决方案，这就是我的做法：

showPF          = document.getElementsByClassName('showPF');

var i = showPF.length;
    while(i--){
        showPF[i].addEventListener('click', function(x){ return function(){ slidePFer(x); }; }(i),false);
    }

信用在哪里信用： http ://brackets.clementng.me/post/24150213014/example-of-a-javascript-closure-settimeout-inside-a

于 2013-10-17T12:15:44.477 回答