Почему игнорируется условие if?

Question

[sharpsss]

в общем не понимаю почему вот это условие просто не хочет отрабатывать, я уверен что я красавчик всё правильно делаю, но увы чёта оно не пашет, почему понять не могу

if (parseInt(number.innerHTML) == 0){
  let step = 3;
  } else {
  let step = parseInt(number.innerHTML) - 1;
  }

https://codepen.io/sharpprahs/pen/rNrwvBv

Answer 1

[Вадим]

1. Если что-то не работает как задумано, первое что нужно сделать это открыть инструменты разработчика на вкладке console и посмотреть нет ли там никаких ошибок. Если есть то загуглить их и посмотреть где они возникают. Попытаться осознать и исправить. А ещё лучше научиться отладке собственного кода https://learn.javascript.ru/debugging-chrome
2. Перед тем как пользоваться тем или иным способом объявления переменных (var\let\const) стоит почитать про него и ознакомиться с нюансами.
https://medium.com/nuances-of-programming/%D0%B2-%...

А именно:

if (parseInt(number.innerHTML) == 0){
  let step = 3;
  } else {
  let step = parseInt(number.innerHTML) - 1;
  }

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

Comments

[Kentavr16]

И как работающий пример дополню

function hides(){
  let popUp = document.querySelector('.popUp');
  popUp.style.display ="flex";
  function hidden() {
  popUp.style.display ="none";
}
  
 let step;
  
function countdown(){
  let number = document.querySelector('b');
  if (parseInt(number.innerHTML) == 0){
    step=3;
  clearInterval(myInterval)
  } else {
  step = parseInt(number.innerHTML) - 1;
  }
  number.innerHTML = step;
  number = null;
  step = null;
}
setTimeout(hidden, 3000);
let myInterval = setInterval(countdown, 1000);
}


let input = document.querySelector('input');
input.addEventListener("click", hides);

Советую обратить внимание на использование setInterval и обьявление переменных. В остальном целиком поддерживаю ответ выше.

[sharpsss]

понял спасибо

Answer 2

[YK21]

разберитесь с областью действия переменных

в каждой ветке условия у Вас своя переменная step

Comments

[sharpsss]

точно