JavaScript removeEventListenerが動作しない

Question

シューティングゲームを作っています。
クリックを連打できないようにaddEventListenerとremoveEventListenerを使っていてそこではしっかり動作していますが、タイムオーバー時にremoveEventListenerをしてイベントを削除する、というコードを書いているのですがここがうまく動作しません。
(keydownでも同じような動作をしたいです。)

調べてみたのですが解決できませんでした。
ご教授宜しくお願いいたします。

//タイマー
let timeId = setInterval(function () {
      cnt--;
      timer.text(cnt);
      dt = new Date(); //現在時刻を取得
      if (dt.getTime() >= endDt.getTime()) {
        // 現在日時と終了日時を比較
        clearInterval(timeId);
        //タイムアップ
        timeUpFunc();
      }
    }, 1000);


gameSheet.addEventListener("click", handleClick);
addEventListener("keydown", keydown);


let clickFlag = true;
let keyFlag = true;
//クリックイベント
function handleClick() {
   if (clickFlag) {
      goFire(levelUpHitCount);
      clickFlag = false;
      gameSheet.removeEventListener("click", handleClick);
    }
    setTimeout(function () {
      clickFlag = true;
      gameSheet.addEventListener("click", handleClick,false);
    }, 1000)
}


//enterキーのイベント 
function keydown(event) {
　 let ck = event.keyCode;
   if (ck === 13) {
     if (keyFlag) {
       console.log(keyFlag);
       keyFlag = false;
       goFire(levelUpHitCount);
       removeEventListener("keydown", keydown);
        }
      }
      setTimeout(function () {
        console.log(keyFlag);
        keyFlag = true;
        addEventListener("keydown", keydown);
      }, 1000);
    }


//タイムアップ時の関数
function timeUpFunc() {
      createjs.Ticker.removeEventListener("tick", handleTick);
      gameSheet.removeEventListener("click", handleClick);
      removeEventListener("keydown", keydown);
}

Comments

[m.ts10806]

clickFlag という変数を持っているならaddEvent内でその変数を見て何もしないreturnだけという対応でも良いように思いますが、如何でしょうか。

[aaaa-08]

返答ありがとうございます。
初学者なので理解が間違っていたらすみません...
clickFlagがfalseの時に何もないreturnってことでよいしょうか？
一応これで解決いたしました！

Answer 1

falseの時にreturnするだけでできました。
もうちょっと自分で考えれば導けそうな答えだったので、もっとロジックや考える力をつけていきたいと思っています。

let clickFlag = true;
function handleClick() {
   if (clickFlag) {
    goFire(levelUpHitCount);
    clickFlag = false;
   }else{
     return;
   }
   setTimeout(function () {
  　 clickFlag = true;
   }, 1000)
    }