2

我一直在关注 w3schools 上的幻灯片教程来学习如何创建自己的幻灯片。

这是教程代码的链接。然而，我指的是这段代码。

<script>
  var slideIndex = 0;
  showSlides();

function showSlides() {
  var i;
  var slides = 
document.getElementsByClassName("mySlides");
var dots = document.getElementsByClassName("dot");
for (i = 0; i < slides.length; i++) {
   slides[i].style.display = "none";  
}
slideIndex++;
if (slideIndex > slides.length) {slideIndex = 1}    
for (i = 0; i < dots.length; i++) {
    dots[i].className = dots[i].className.replace(" 
active", "");
}
slides[slideIndex-1].style.display = "block";  
dots[slideIndex-1].className += " active";
setTimeout(showSlides, 2000); // Change image every 
2 seconds
}
</script>

我想为一个页面制作多个幻灯片，所以我做了它，所以这部分代码将采用一个参数，然后我可以插入我的图像列表。但是当我将代码更改为此...

 var images = document.getElementsByClassName("mySlides");
var imageNum = document.getElementsByClassName("dot");

showSlides(images, imageNum);

function showSlides(slides, dots) {
    var i;

for (i = 0; i < slides.length; i++) {
   slides[i].style.display = "none";  
}
slideIndex++;
if (slideIndex > slides.length) {slideIndex = 1}    
for (i = 0; i < dots.length; i++) {
    dots[i].className = dots[i].className.replace(" active", "");
}
slides[slideIndex-1].style.display = "block";  
dots[slideIndex-1].className += " active";

    setTimeout(showSlides, 2000);
}

完整代码可通过此链接获得。

它给了我长度未定义的错误属性，即使 .length 可以用于 HTML 集合。

为什么这个函数似乎不能接受 HTML 集合作为参数？

编辑：在查看反馈并注意到我让人们更难帮助我之后，我为我的问题创建了一个代码框链接。我将解决方案作为评论包含在内，因此人们可以看到前后。我还包括了上面的整个功能。

感谢您的反馈，这样我就可以更清楚地提出问题，并对问题感到抱歉！

4

1 回答 1

6

这是因为函数末尾的那一行

setTimeout(showSlides, 2000); // Change image every 2 seconds

它showSlides不带任何参数地调用

要为调用添加参数，您可以执行以下操作：

setTimeout(() => { showSlides(slides, dots) }, 2000);
// call from an anonymous function

// or

setTimeout(showSlides, 2000, slides, dots); 
// use additional params of setTimeout

这是文档setTimeout

于 2018-08-30T13:26:26.583 回答