js アクション ジャンプ

Question

いまjsでアクションゲームを作っていてジャンプを完成させられたのですがだんだんジャンプのスピードが速くなって行ってしまいます
どうすればよいでしょうか

エラーコード

window.addEventListener("keydown",function(event){
        if (event.key === "w") { 
        カオスダークドラゴン = 0
        var jamp = function () {
           if (カオスダークドラゴン == 0) {
            y--    
           }
           if (y < 250) {
            カオスダークドラゴン = 1      
         }
         if(カオスダークドラゴン == 1){
                y++ 
        if (y > 330) {
          clearInterval(jamp)
         y = 330
           }
        }    
     }    
    }setInterval(jamp,10)
 })  

（変数名は気にしないでください）

Comments

[gasbombe]

・yが何なのかわからない
・wキーを押したときに、やりたい動作がわからない。（1回だけジャンプ／繰り返しジャンプなど）
・「ジャンプのスピードが速く」とは、インターバルが短くなるのか、動作が速くなるのかわからない
・そもそもジャンプできるソースコードに見えない
などの理由で、何をしようとしてどこでつまづいているのかがわかりません。

あと、ソースコードのインデントや改行が汚いと読みにくいので回答がつきにくいです。

[masao1234]

すみません　でも解決しました　ありがとうございます!!

Answer 1

var timerId;
clearInterval(timerId);
timerId=setInterval(jamp,10);

Comments

[masao1234]

ありがとうございます
できれば詳しい説明いいですか？

[masao1234]

これってどこにつければいいですか

[gasbombe]

clearInterval()の引数がおかしいという指摘ですね。
timerId=setInterval(jamp,10); のように取得したタイマーIDを指定します。

[KazuhiroHatano]

clearIntervalにタイマーのIDが渡されてないので
タイマーが止まらず多重にjampが実行されてる状態です

clearIntervalの引数は登録したコールバックではなく
止めたいタイマーのIDを指定します

タイマーのIDはsetIntervalが返してくれてます

MDNで使い方を確認してみましょう

https://developer.mozilla.org/ja/docs/Web/API/WindowOrWorkerGlobalScope/setInterval

[masao1234]

あいざいます