[英]N+1 in javascript not working as intended
我從 Stack Overflow 的另一個問題中得到了這個 img 畫廊,我添加了一些按鈕,以便能夠通過+1
和-1
更改當前的 img。
但是它不太有效,我做錯了什么?
如果您想自己嘗試一下,這也是一個小提琴:
https://jsfiddle.net/k5r98foh/1/
<div class="container3 wrapper">
<div class="mySlides">
<div class="bb" style="background-image:url(https://lh3.googleusercontent.com/-NREkKmTtXX0/AAAAAAAAAAI/AAAAAAAAADg/w2IRnCo5AwQ/photo.jpg?sz=328);"></div>
<!-- The above div is the problem -->
</div>
<div class="mySlides">
<img src="https://i.stack.imgur.com/FyV3d.gif" style="width:100%">
</div>
<div class="row">
<div class="column">
<!-- pass this to handler -->
<img class="demo cursor" src="https://lh3.googleusercontent.com/-NREkKmTtXX0/AAAAAAAAAAI/AAAAAAAAADg/w2IRnCo5AwQ/photo.jpg?sz=328" onclick="currentSlide(this,1)" width="100%" alt="text">
</div>
<div class="column">
<!-- pass this to handler -->
<img class="demo cursor" src="https://i.stack.imgur.com/FyV3d.gif" width="100%" onclick="currentSlide(this, 2)" alt="text">
</div>
</div>
<a class="prev" onclick="plusSlides(-1)">❮</a>
<a class="next" onclick="plusSlides(1)">❯</a>
</div>
-Javascript
var slideIndex = 1;
let wrappers = document.querySelectorAll(".wrapper");
wrappers.forEach(function(el) {
showSlides(slideIndex, el);
});
// Next/previous controls
function plusSlides(n) {
showSlides(slideIndex += n);
}
// Thumbnail image controls
function currentSlide(el, n) {
// Get the wrapper element and pass it as a variable to showSlides
let wrapper = el.closest(".wrapper");
showSlides(slideIndex = n, wrapper);
}
function showSlides(n, wrapper) {
var i;
// find only the slides and dot that are decedents of current wrapper
var slides = wrapper.querySelectorAll(".mySlides");
var dots = wrapper.querySelectorAll(".demo");
if (n > slides.length) {
slideIndex = 1
}
if (n < 1) {
slideIndex = slides.length
}
for (i = 0; i < slides.length; i++) {
slides[i].style.display = "none";
}
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";
}
plusSlides() 中的plusSlides()
showSlides()
function 調用缺少包裝上下文。 將其更改為:
function plusSlides(el,n) {
showSlides(slideIndex += n, el.closest('.wrapper'));
}
並將對元素本身的引用 ( this
) 添加到您的<a>
標記中的 function 調用中:
<a class="prev" onclick="plusSlides(this,-1)">❮</a>
<a class="prev" onclick="plusSlides(this,1)">❯</a>
var slideIndex = 1; let wrappers = document.querySelectorAll(".wrapper"); wrappers.forEach(function(el) { showSlides(slideIndex, el); }); // Next/previous controls function plusSlides(el,n) { showSlides(slideIndex += n,el.closest('.wrapper')); } // Thumbnail image controls function currentSlide(el, n) { // Get the wrapper element and pass it as a variable to showSlides let wrapper = el.closest(".wrapper"); showSlides(slideIndex = n, wrapper); } function showSlides(n, wrapper) { var i; // find only the slides and dot that are decedents of current wrapper var slides = wrapper.querySelectorAll(".mySlides"); var dots = wrapper.querySelectorAll(".demo"); if (n > slides.length) { slideIndex = 1 } if (n < 1) { slideIndex = slides.length } for (i = 0; i < slides.length; i++) { slides[i].style.display = "none"; } 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"; }
.container3 { position:absolute; height:100px; width:20%; margin:auto; right:0; left:0; }.bb{ height:100px; background: center center no-repeat; background-size: cover; }.mySlides { display: none; }.cursor { cursor: pointer; }.row { width:100%; display:flex; }.demo { opacity: 0.6; }.active, .demo:hover { opacity: 1; } /* Next & previous buttons */.prev, .next { cursor: pointer; position: absolute; top: 40%; width: auto; padding: 16px; margin-top: -50px; color: white; font-weight: bold; font-size: 20px; border-radius: 0 3px 3px 0; user-select: none; -webkit-user-select: none; } /* Position the "next button" to the right */.next { right: 0; border-radius: 3px 0 0 3px; } /* On hover, add a black background color with a little bit see-through */.prev:hover, .next:hover { background-color: rgba(0, 0, 0, 0.8); }
<div class="container3 wrapper"> <div class="mySlides"> <div class="bb" style="background-image:url(https://lh3.googleusercontent.com/-NREkKmTtXX0/AAAAAAAAAAI/AAAAAAAAADg/w2IRnCo5AwQ/photo.jpg?sz=328);"></div> <:-- The above div is the problem --> </div> <div class="mySlides"> <img src="https.//i.stack.imgur.com/FyV3d:gif" style="width:100%"> </div> <div class="row"> <div class="column"> <.-- pass this to handler --> <img class="demo cursor" src="https.//lh3.googleusercontent?com/-NREkKmTtXX0/AAAAAAAAAAI/AAAAAAAAADg/w2IRnCo5AwQ/photo,jpg:sz=328" onclick="currentSlide(this.1)" width="100%" alt="text"> </div> <div class="column"> <.-- pass this to handler --> <img class="demo cursor" src="https.//i.stack,imgur,com/FyV3d,gif" width="100%" onclick="currentSlide(this, 2)" alt="text"> </div> </div> <a class="prev" onclick="plusSlides(this,-1)">❮</a> <a class="next" onclick="plusSlides(this,1)">❯</a> </div>
聲明:本站的技術帖子網頁,遵循CC BY-SA 4.0協議,如果您需要轉載,請注明本站網址或者原文地址。任何問題請咨詢:yoyou2525@163.com.