Необходимо изменить код JavaScript для запуска различных сгенерированных функций в цикле - PullRequest
Новая
       8

Необходимо изменить код JavaScript для запуска различных сгенерированных функций в цикле

0 голосов
/ 30 декабря 2018

Модификация из "https://www.w3schools.com/howto/howto_js_slideshow.asp":

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';
}

Я вызываю эти функции из php, как указано ниже: 

for($j=0; $j < 2; $j++) {
    for($i=0; $i < 10; $i++) {
    echo "<div>
      <!-- Next and previous buttons -->
      <a class="prev" onclick="plusSlides(-1)">&#10094;</a>
      <a class="next" onclick="plusSlides(1)">&#10095;</a>
    </div>";
    }
}

Я хочу изменить приведенный выше код в приведенный ниже код, который не являетсяработает для меня: 

for($j=0; $j < 2; $j++) {
    for($i=0; $i < 10; $i++) {
    echo "<div>
      <!-- Next and previous buttons -->
      <a class="prev" onclick="plusSlides_".$i."(-1)">&#10094;</a>
      <a class="next" onclick="plusSlides_".$i."(1)">&#10095;</a>
    </div>";
    }
}

Я хочу 2 отдельных ползунка с 10 изображениями в каждом и отдельные элементы управления для обоих ползунков.

Как мне заставить работать приведенный выше код? Что я должен изменить?

Я пытался использовать следующее для вызова функции: 

window["plusSlides".concat("_",$i)](slideIndex);

и следующее для определения функции: 

window["plusSlides".concat("_",$i)] = function(n) {}

и функцию eval, но они этого не сделалиработа для меня.

1 Ответ

0 голосов
/ 30 декабря 2018

Предложение

Вместо того, чтобы изменять свой внутренний код, почему бы вам просто не изменить свой внешний код?На мой взгляд, это намного проще, и не говоря уже о том, что, возможно, все проще.

Во-вторых, почему бы вам не сделать какую-то функцию для создания слайдера, а не использовать цикл?Я просто думаю, конечно, что будет проще?Пример:

Редактировать 1

Имейте в виду, я не говорю, что вы должны использовать этот конкретный код PHP, я просто приводил пример того, что вы могли бы сделать, чтобы сделать вашу жизнь проще в долгосрочной перспективе. PS Я написал этот код PHP на лету, поэтому я понятия не имею, работает он или нет, это скорее псевдокод, чем формальное решение.

Редактировать 2

Это просто простое обновление, содержащее структуры данных, которые я имел в виду для массивов изображений.Пожалуйста, имейте в виду, что я не пробовал и не тестировал этот код, это больше псевдокод, чем что-либо еще, это просто здесь, чтобы помочь вам в решении вашего решения.

<?php

  // This is just a POPO - Plain old php object.
  // Aka a data/value object. 
  class Image {
    public $src;
    public $caption;
    public $index;

    function __construct ($src, $caption, $index) {
      $this->src = $src;
      $this->caption = $caption;
      $this->index = $index;
    }
  }

  // How you do this is up to you, but this is an EXAMPLE... 
  $imageArray1 = array(
    new Image('../image.png', 'This is an awesome image', '1/10')
  );
?>

Тогда это оригинальный код, который я написал в этом ответе ... 

<?php
    function slideshow (images) {
        ob_start();

        <div class="slideshow-container">

          <?php foreach ($images as $image) : ?>
              <div class="mySlides fade">
                <div class="numbertext"><?php echo $image->index ?></div>
                <img src="<?php echo $image->src?>" style="width:100%">
                <div class="text"><?php echo $image->caption ?></div>
              </div>       
          <?php endforeach; ?>

          <a class="prev">&#10094;</a>
          <a class="next">&#10095;</a>
        </div>

        return ob_get_clean();  
    }

    echo slideshow($imageArray1);
    echo slideshow($imageArray2);
?>

Кроме того, в качестве примечания, смотрите на то, что вы пытаетесь достичь, ичто вы в настоящее время реализуете, ваш подход полностью отключен?Конечно, было бы разумнее создать множество элементов с именем класса slideshow-container.Это не только кажется более логичным с точки зрения разметки, но также означает, что ваш JavaScript может сделать с ним больше.В качестве примера, вот моя идея, касающаяся того, как вы будете кодировать свой JavaScript, вместо того, чтобы многие функции JavaScript создавались на стороне сервера, просто сделать его более универсальным и гибким, это просто?

С этим решением, как вы можете видеть, вместо множества функций JavaScript у вас есть только две, start и slideshow, ничего сложного, простого и гибкого.Кроме того, если вы хотите добавить больше слайд-шоу на более поздний срок, я думаю, что этот подход будет легче управлять и поддерживать.

const slideshow = (el) => {

  let slideIndex = 1;
  const plusSlides = n => showSlides(slideIndex += n);
  const currentSlide = n => showSlides(slideIndex = n);

  function showSlides(n) {
    const slides = el.querySelectorAll('.mySlides');

    if (n > slides.length) slideIndex = 1;
    else if (n < 1) slideIndex = slides.length;

    for (let i = 0; i < slides.length; i++) slides[i].style.display = 'none';
    slides[slideIndex - 1].style.display = 'block';
  }

  showSlides(slideIndex);

  el.querySelectorAll(".prev").forEach(i => {
    i.addEventListener('click', x => plusSlides(-1));
  });

  el.querySelectorAll(".next").forEach(i => {
    i.addEventListener('click', x => plusSlides(1));
  });
};

const start = () => {
  document.querySelectorAll(".slideshow-container").forEach(el => slideshow(el));
};

setTimeout(start, 250); // A super lazy implementation of document ready...
/*
  NO CHANGES HAVE BEEN MADE HERE... 
*/

* {
  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 & 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 */

.text {
  color: #f2f2f2;
  font-size: 15px;
  padding: 8px 12px;
  position: absolute;
  bottom: 8px;
  width: 100%;
  text-align: center;
}


/* 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
  }
}
<!-- First slideshow -->
<div class="slideshow-container">
  <div class="mySlides fade">
    <div class="numbertext">1 / 3</div>
    <img src="https://www.w3schools.com/howto/img_nature_wide.jpg" style="width:100%">
    <div class="text">Caption Text</div>
  </div>

  <div class="mySlides fade">
    <div class="numbertext">2 / 3</div>
    <img src="https://www.w3schools.com/howto/img_snow_wide.jpg" style="width:100%">
    <div class="text">Caption Two</div>
  </div>

  <div class="mySlides fade">
    <div class="numbertext">3 / 3</div>
    <img src="https://www.w3schools.com/howto/img_mountains_wide.jpg" style="width:100%">
    <div class="text">Caption Three</div>
  </div>
  
  <a class="prev">&#10094;</a>
  <a class="next">&#10095;</a>
</div>
<br>

<!-- Just to show where the code splits. -->
<hr>

<!-- Second slideshow. -->
<div class="slideshow-container">
  <div class="mySlides fade">
    <div class="numbertext">1 / 3</div>
    <img src="https://www.w3schools.com/howto/img_nature_wide.jpg" style="width:100%">
    <div class="text">Caption Text</div>
  </div>

  <div class="mySlides fade">
    <div class="numbertext">2 / 3</div>
    <img src="https://www.w3schools.com/howto/img_snow_wide.jpg" style="width:100%">
    <div class="text">Caption Two</div>
  </div>

  <div class="mySlides fade">
    <div class="numbertext">3 / 3</div>
    <img src="https://www.w3schools.com/howto/img_mountains_wide.jpg" style="width:100%">
    <div class="text">Caption Three</div>
  </div>

  <a class="prev">&#10094;</a>
  <a class="next">&#10095;</a>
</div>
<br>
...