javascriptのカウントダウンタイマーの止め方について

Question

前提・実現したいこと

カウントダウンタイマーのカウントが0になるとタイマーが停止する機能の実装。

発生している問題・エラーメッセージ

javascriptでカウントダウンタイマーを制作してますが、setIntervalとclearIntervalを使ってもタイマーが止まりません。

該当のソースコード

ソースコード
var PassSec;
var CountDownValue;
var min = 0;
var sec = 0;
var PassageID = null;

function timerStop(){
	clearInterval(PassageID);
}

// タイマー処理の中身
function showPassage() {
	
	CountDownValue--;   // カウントダウン
	min = Math.floor(CountDownValue / 60);
	sen = Math.floor(CountDownValue % 60);
	PassageArea.innerHTML =min +':' + sen ;
         
	if (sen<10) {
		if (min<10){
		PassageArea.innerHTML ='0' + min + ':' + '0' + sen;
		}else {
			PassageArea.innerHTML =min + ':' + '0' + sen;
		}
		}else if(min<10){
		PassageArea.innerHTML ='0' + min + ':' + sen;
	}
		else{
		PassageArea.innerHTML =min + ':' + sen;
	};
}

// タイマー開始
function startShowing() {
    CountDownValue = 60;   // カウンタのリセット
	PassageID = setInterval('showPassage()',1000);   
	// タイマーをセット
	if(CountDownValue < 0){
		timerStop();
	}
	
}

Answer 1

startShowingの中でif(CountDownValue < 0){しても意味がないです。
インターバルの中でチェックします

<script>
var PassSec;
var CountDownValue;
var min = 0;
var sec = 0;
var PassageID = null;

function timerStop(){
    clearInterval(PassageID);
}

function showPassage() {
  CountDownValue--;   // カウントダウン
  if(CountDownValue <= 0){
    timerStop();
  }
  min = Math.floor(CountDownValue / 60);
  sen = Math.floor(CountDownValue % 60);
  PassageArea.innerHTML =min +':' + sen ;
  
  if (sen<10) {
    if (min<10){
      PassageArea.innerHTML ='0' + min + ':' + '0' + sen;
    }else {
      PassageArea.innerHTML =min + ':' + '0' + sen;
    }
  }else if(min<10){
    PassageArea.innerHTML ='0' + min + ':' + sen;
  }
  else{
    PassageArea.innerHTML =min + ':' + sen;
  };
}

// タイマー開始
function startShowing() {
  CountDownValue = 60;   // カウンタのリセット
  PassageID = setInterval(showPassage,1000);   

}
window.addEventListener('DOMContentLoaded', ()=>{
  PassageArea=document.querySelector('#hoge');
  document.querySelector('#start').addEventListener('click',startShowing);
  document.querySelector('#stop').addEventListener('click',timerStop);
});
</script>
<input type="button" value="start" id="start">
<input type="button" value="stop" id="stop"><br>
<div id="hoge">01:00</div>

Comments

[fateinboku]

回答通りにコードを書き換えたら解決しました！
ありがとうございました！！

Answer 2

setInterval('showPassage()',1000); 

　
単純に文法ミスなのでは？
余計なシングルクォーテーションが付いているから関数が動いていないかと

setInterval(showPassage,1000);