Карусель JavaScript движется слишком быстро сама по себе

ProgramBox

Карусель JavaScript движется слишком быстро сама по себе

Post author:admin
Запись опубликована:21 марта, 2023
Post category:Вопросы по программированию

#javascript #html

#javascript #HTML

Вопрос:

Когда я нажимаю next кнопки previous или или точки, картинки начинают двигаться слишком быстро сами по себе. Я не могу найти проблему.

Это карусель с четырьмя изображениями, и она работает, но не так, как должна. Я думаю, что проблема в JavaScript, но я не смог ее идентифицировать.

 var slideIndex = 1; 
showSlides(slideIndex);
function plusSlides(n) {
  showSlides(slideIndex  = n);
}
function currentSlide(n) {
  showSlides(slideIndex = n);
}
function showSlides(n) {
  var i;
  var slides = document.getElementsByClassName("mySlides");
  var dots = document.getElementsByClassName("dot");
  if (n > slides.length) {slideIndex = 1;}
  else 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";
}
var slideIndex = 0;
showSlides();
function showSlides() {
    var i;
    var slides = document.getElementsByClassName("mySlides");
    for (i = 0; i < slides.length; i  ) {
        slides[i].style.display = "none";
    }
    slideIndex  ;
    if (slideIndex > slides.length) {slideIndex = 1;}
    slides[slideIndex-1].style.display = "block";
    setTimeout(showSlides, 3000); 
}

 <!DOCTYPE html>
<html lang="en">
  <head>
  <meta charset="UTF-8">
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <meta http-equiv="X-UA-Compatible" content="ie=edge" /> 
     <link rel="stylesheet" href="styleP.css" />
    <script src="https://kit.fontawesome.com/a076d05399.js"></script>  
    <script src="JavaScript.js"></script>
    <title>Smart Brothers</title>
  </head>
   <body>
   <div class="slideshow-container">
   <div class="mySlides fade">
      <img src="final.jpg" style="width:100%">
      <div class="text">Project One</div>
    </div>
    <div class="mySlides fade">
      <img src="final2.jpg" style="width:100%">
      <div class="text">Project Two</div>
    </div>
    <div class="mySlides fade">
      <img src="final3.jpg" style="width:100%">
      <div class="text">Project Three</div>
    </div> 
    <div class="mySlides fade">
      <img src="final4.jpg" style="width:100%">
      <div class="text">Project Four</div>
    </div> 
    <a class="prev" onclick="plusSlides(-1)">amp;#10094;</a>
  <a class="next" onclick="plusSlides(1)">amp;#10095;</a>
</div>
<br>
<div style="text-align:center">
  <span class="dot" onclick="currentSlide(1)"></span>
  <span class="dot" onclick="currentSlide(2)"></span>
  <span class="dot" onclick="currentSlide(3)"></span>
  <span class="dot" onclick="currentSlide(4)"></span>
</div>
</body>
</html>

1. У вас есть две функции с одинаковым именем. —> showSlides Это не поможет.

2. Хорошо, значит, я должен изменить имя функции? Или каковы ваши предложения, пожалуйста?

3. Не вдаваясь в подробности множественного this wrong в этом коде, конкретная вещь, которая, по-видимому, контролирует время слайдов, это setTimeout(showSlides, 3000);

4. Я все еще изучаю JavaScript =) Это работает, но когда я нажимаю кнопки NEXT или PREVIOUS, она ломается, и изображения начинают быстро перемещаться.

Ответ №1:

Чтобы ответить, что быстрое перемещение вашей карусели JavaScript overriding не поддерживает overloading , это означает, что если вы определяете две функции с одинаковыми именами, последней определенной будет override ранее определенная версия, и каждый раз, когда будет выполняться вызов функции, будет выполняться последняя определенная.

 var slideIndex = 1;
showSlides(slideIndex);

function plusSlides(n) {
  showSlides(slideIndex  = n);
}

function currentSlide(n) {
  showSlides(slideIndex = n);
}

function showSlides(n) {
  var i;
  var slides = document.getElementsByClassName("mySlides");
  var dots = document.getElementsByClassName("dot");
  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";
}
var slideIndex = 0;
showSlides1();

function showSlides1() {
  var i;
  var slides = document.getElementsByClassName("mySlides");
  var dots = document.getElementsByClassName("dot");
  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", "");
  }
  slideIndex  ;
  if (slideIndex > slides.length) {
    slideIndex = 1;
  }

  slides[slideIndex - 1].style.display = "block";
  dots[slideIndex - 1].className  = " active";
  setTimeout(showSlides1, 3000);
}

 * {
  box-sizing: border-box
}

body {
  font-family: Verdana, sans-serif;
  margin: 0
}

.mySlides {
  display: none
}

img {
  vertical-align: middle;
}


/* Slideshow container */

.slideshow-container {
  max-width: 1000px;
  position: relative;
  margin: auto;
}


/* Next amp; previous buttons */

.prev,
.next {
  cursor: pointer;
  position: absolute;
  top: 50%;
  width: auto;
  padding: 16px;
  margin-top: -22px;
  color: white;
  font-weight: bold;
  font-size: 18px;
  transition: 0.6s ease;
  border-radius: 0 3px 3px 0;
  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);
}


/* Caption text */


/* Number text (1/3 etc) */

.numbertext {
  color: #f2f2f2;
  font-size: 12px;
  padding: 8px 12px;
  position: absolute;
  top: 0;
}


/* The dots/bullets/indicators */

.dot {
  cursor: pointer;
  height: 15px;
  width: 15px;
  margin: 0 2px;
  background-color: #bbb;
  border-radius: 50%;
  display: inline-block;
  transition: background-color 0.6s ease;
}

.active,
.dot:hover {
  background-color: #717171;
}


/* Fading animation */

.fade {
  -webkit-animation-name: fade;
  -webkit-animation-duration: 1.5s;
  animation-name: fade;
  animation-duration: 1.5s;
}

@-webkit-keyframes fade {
  from {
    opacity: .4
  }
  to {
    opacity: 1
  }
}

@keyframes fade {
  from {
    opacity: .4
  }
  to {
    opacity: 1
  }
}


/* On smaller screens, decrease text size */

@media only screen and (max-width: 300px) {
  .prev,
  .next,
  .text {
    font-size: 11px
  }
}

 <!DOCTYPE html>
<html lang="en">

<head>
  <meta charset="UTF-8">
  <meta name="viewport" content="width=device-width, initial-scale=1.0" />
  <meta http-equiv="X-UA-Compatible" content="ie=edge" />
  <link rel="stylesheet" href="styleP.css" />
  <script src="https://kit.fontawesome.com/a076d05399.js"></script>
  <script src="JavaScript.js"></script>
  <title>Smart Brothers</title>
</head>

<body>
  <div class="slideshow-container">
    <div class="mySlides fade">
      <img src="final.jpg" style="width:100%">
      <div class="text">Project One</div>
    </div>
    <div class="mySlides fade">
      <img src="final2.jpg" style="width:100%">
      <div class="text">Project Two</div>
    </div>
    <div class="mySlides fade">
      <img src="final3.jpg" style="width:100%">
      <div class="text">Project Three</div>
    </div>
    <div class="mySlides fade">
      <img src="final4.jpg" style="width:100%">
      <div class="text">Project Four</div>
    </div>
    <a class="prev" onclick="plusSlides(-1)">amp;#10094;</a>
    <a class="next" onclick="plusSlides(1)">amp;#10095;</a>
  </div>
  <br>
  <div style="text-align:center">
    <span class="dot" onclick="currentSlide(1)"></span>
    <span class="dot" onclick="currentSlide(2)"></span>
    <span class="dot" onclick="currentSlide(3)"></span>
    <span class="dot" onclick="currentSlide(4)"></span>
  </div>
</body>

</html>