Я пытаюсь создать две функции, которые позволят мне прокручивать массив источников изображений и отображать их в моем представлении изображения: nextImage () и previousImage (). Они работают, если я использую исходную переменную num, но если я попытаюсь сделать ее более общей, чтобы я мог повторно использовать их в своем коде, я передаю переменную num в качестве параметра в моем окне. Загрузить это не работает, она просто достигает числа 1 и не идет назад. Пожалуйста, посмотрите на код. Заранее спасибо, Алессандро

var num = 0;

var imagesArray1 = ["img/campionatigiovanili2018/img1.jpg", "img/campionatigiovanili2018/img2.jpg", "img/campionatigiovanili2018/img3.jpg"];

window.onload = function(){
  let logo = document.getElementById('logo');
  logo.addEventListener("click", function(){
    window.location.href = "index.html";
  });



  document.getElementById('postImage1').setAttribute("src", imagesArray1[num]);


  //previous image
  document.getElementById('previousImage').addEventListener("click", function(){
    previousImage("postImage1", imagesArray1, num)
});

document.getElementById('nextImage').addEventListener("click", function(){
  nextImage("postImage1", imagesArray1, num);
});

};

function nextImage(postImage, imageArray, myVar){
  if (myVar < 2) {
    myVar = myVar+1
    document.getElementById(postImage).setAttribute("src", imageArray[myVar]);
    console.log(myVar);
  } else {
  }
};

function previousImage(postImage, imageArray, myVar){
  if (myVar > 0) {
    myVar = myVar-1
    document.getElementById(postImage).setAttribute("src", imageArray[myVar]);
    console.log(myVar);
  } else {
  };
}
0
Alessandro Gregolin 8 Окт 2018 в 22:11

2 ответа

Лучший ответ

Пожалуйста, обращайтесь к переменной num внутри функции, а не передавайте ее как параметр:

function nextImage(postImage, imageArray){
  if (num < 2) {
    num = num +1
    document.getElementById(postImage).setAttribute("src", imageArray[num]);
    console.log(num);
  } else {
  }
};

Поскольку myVar и num в этом случае - две разные переменные.

2
Sergii Vorobei 8 Окт 2018 в 19:22

Javascript всегда передает параметры по значению, если они примитивны (например, ваша переменная num)

Решение:

  • Просто напрямую отредактируйте num внутри обработчика событий, поскольку это глобальная переменная.

  • Объявите это как объект: var currentObj = { num:0};

2
Nhon Dinh 8 Окт 2018 в 19:25