Почему click срабатывает только один раз?

Question

[Alexey1307]

Есть календарь, хочу чтоб по нажатию на кнопки "назад', и 'вперед' менялся месяц.
Сейчас кнопки меняют месяц, но только 1 раз, то есть с Июня на Июль поменяет, а на Август уже нет!
Плюс если нажать 'назад' то вместо возврата в июнь, календарь отрисует Май!
Вот код функции переключения стрелок:

function switchMounth(){
	let prev = document.querySelector('.prev');
  let next = document.querySelector('.next');
	
	prev.addEventListener('click', ()=>{
    for(let i =0; i<=monthsName.length; i++){
      if(i == curentMonth){
        let a = i-1;
            for(let k of monthsName){
              if(k == months.innerHTML){
                createCalendar(calendar, curentYear, realMonth-1);
                months.innerHTML = monthsName[a];
              }
            }      
          }  
      }  
	})

    next.addEventListener('click', ()=>{
    for(let i =0; i<=monthsName.length; i++){
      if(i == curentMonth){
        let a = i+1;
            for(let k of monthsName){
              if(k == months.innerHTML){
                createCalendar(calendar, curentYear, realMonth+1);
                months.innerHTML = monthsName[a];
              }
            }      
         }  
      }  
  })
}

Календарь здесь

Answer 1

[GeminiX]

У вас, как минимум, после клика у currentMonth не обновляется значение

Comments

[Alexey1307]

И правда!спасибо тебе милый человек)) Уже есть с чем работать!

Answer 2

[black1277]

У вас две переменные currentMonth и realMonth не меняются после клика, а должны увеличиваться или уменьшаться. Посмотрите лог перед каждым вызовом ф-и, чтобы понять

console.log('prev', calendar, curentYear, realMonth-1);
createCalendar(calendar, curentYear, realMonth-1);

Comments

[Alexey1307]

Спасибо большое что наводите на правильные мысли)
рад что у нас есть такое js сообщество, где можно попросить помощи:)

[black1277]

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

[Alexey1307]

black1277, Добавил изменения переменных строчками:
для стрелки назад:
curentMonth = curentMonth -1;
realMonth = realMonth - 1;

и для стрелки вперед соответственно:
curentMonth = curentMonth +1;
realMonth = realMonth +1;

Получилось в итоге вот

Кнопка назад заработала как надо, а вперед почему то не хочет..

[Alexey1307]

black1277, Ок! про пограничные значения тоже учту! спасибо!

[black1277]

Alexey1307, Вообще, если по правильному делать, то нужно избавляться от двух переменных - т.е. использовать одну и изменять одну - чтобы был единственный источник правды. А иначе, можете получить расхождение в данных, если одну переменную измените, а другую забудете.

[Alexey1307]

Спасибо все получилось! Есть кое какие новые глюки, например при возврате надо 2 раза кнопку вперед нажать, но я думаю поборим! В целом уже работает переключение:)

prev.addEventListener('click', ()=>{
    for(let i =0; i<=monthsName.length; i++){
      if(i == curentMonth){
        let a = i-1;
        console.log(curentMonth);
        createCalendar(calendar, curentYear, curentMonth);
        curentMonth = curentMonth - 1;
        console.log(curentMonth);
        months.innerHTML = monthsName[a];
          }  
      }  
  })

    next.addEventListener('click', ()=>{
    for(let i =monthsName.length; i>=0; i--){
      if(i == curentMonth){
        console.log(curentMonth);
        createCalendar(calendar, curentYear, curentMonth);
        curentMonth = curentMonth + 1;
        console.log(curentMonth);
        months.innerHTML = monthsName[i];
          }  
      }  
  })