2017-09-28 67 views
-2

我做了一個簡單的簡單的滑塊和現在我已經達到了一個問題我的變量變量是沒有得到迭代

我已經意識到,以後每隔「前進」按鈕水龍頭「點擊」增加變量只有1,它永遠不會變得更大。我試圖使用for語句,但它也沒有奏效。如果有人能告訴我錯誤在哪裏,我將非常感激。

乾杯

這裏是我的JS代碼

const fw = document.querySelector(".test-button_forward"); 
const back = document.querySelector(".test-button_backwards"); 
const img1 = document.querySelector('.images_first.active'); 
const img2 = document.querySelector(".images_second"); 
const img3 = document.querySelector(".images_third"); 

let clicks = 0; 


window.addEventListener('load', function() { 
     console.log(clicks); 

     fw.addEventListener('click', function() { 
      for (clicks = 0; clicks < 2; clicks++) { 
       if (clicks = 1) { 
        console.log(clicks); 

        img1.classList.remove('active'); 
        img1.classList.add('hidden'); 

        img2.classList.remove('hidden'); 
        img2.classList.add("active"); 

        console.log(clicks); 
        clicks = clicks + 1; 

       } else if (clicks = 2) { 
        img2.classList.remove('active'); 
        img2.classList.add("hidden"); 

        img3.classList.remove('hidden'); 
        img3.classList.add('active'); 

        console.log(clicks); 
       } 
      } 
     }) 
    } 
) 

Link to slider

+0

如果(點擊= 1) – epascarello

+0

讓自己的linting工具內置到IDE – epascarello

回答

1

在您的代碼:

if (clicks = 1) { 

當你的意思是檢查是否要設置點擊等於一個它等於。所以做:

if (clicks === 1) { 

我認爲這是你正在處理的問題。

+0

感謝您接受的答案OP。我繼續做了一些改變,可能會簡化一些https://jsbin.com/munulakehu/edit?html,css,js,output – JonahAaron

0

如果你想要去2需要改變你的循環:

for (clicks = 0; clicks <= 2; clicks++) 

,並沒有設置點擊到1:

if (clicks === 1) {