按键时循环功能

Question

我正在尝试使用Javascript中的两个键上下移动div。其想法是，当按下某个键时，函数每次都会循环并添加到div的"top"样式值中。基本功能可以工作，但我无法使它循环，也无法对按键做出任何响应。

在Javascript中很难找到关于按键处理的信息，似乎大多数人都使用jQuery来处理。

我使用do while循环正确吗？有更好的方法来处理keydown和keyup事件吗？

这是我的代码：

var x = 0;
console.log(x);
function player1MoveDown() {
            var value = document.getElementById("player1").style.top;
            value = value.replace("%", "");
            value = parseInt(value);
            value = value + 1;
            value = value + "%";
            document.getElementById("player1").style.top = value;
            console.log(value);
        }    //moves paddle down; adds to paddle's 'top' style value
function player1MoveSetting() {
    x = 1;
    do {
        setInterval(player1MoveDown(), 3000);
    }   
    while (x == 1);
    console.log(x);
} //paddle moves while x=1; runs player1MoveDown function every 3 seconds
 function player1Stop() {
    x = 0;
 }

这是HTML的相关部分：

<div class="paddle" id="player1" style="top:1%" onkeydown="player1MoveSetting()" onkeyup="player1Stop()"></div>

Answer 1

不能将keydown事件附加到div，除非它具有tabindex:

<div class="paddle" id="player1" 
     onkeydown="player1MoveSetting()"
     onkeyup="player1Stop()"
     tabindex="1"
>
</div>

---

你可以替换所有这些代码：

var value = document.getElementById("player1").style.top;
value = value.replace("%", "");
value = parseInt(value);
value = value + 1;
value = value + "%";
document.getElementById("player1").style.top = value;

…这个：

var p1= document.getElementById('player1');
p1.style.top= parseInt(p1.style.top)+1+'%';

---

这调用player1MoveDown:的返回结果

setInterval(player1MoveDown(), 3000);

由于player1MoveDown不返回任何内容，因此它相当于

setInterval(null, 3000);

要每隔3秒调用函数，请执行以下操作：

setInterval(player1MoveDown, 3000);

---

这创建了一个无限循环：

x = 1;
do {
  setInterval(player1MoveDown, 3000);
}   
while (x == 1);

即使keyup将全局x设置为0，它也永远不会运行，因为循环永远不会结束。

相反，创建一个timer变量，该变量在keydown上设置，在keyup上清除。

---

完整的JavaScript代码

var timer;
function player1MoveDown() {
  var p1= document.getElementById('player1');
  p1.style.top= parseInt(p1.style.top)+1+'%';
  console.log(p1.style.top);
}
function player1MoveSetting() {
  if(timer) return;
  timer= setInterval(player1MoveDown, 100);
}
function player1Stop() {
  clearInterval(timer);
  timer= null;
}
document.getElementById('player1').focus();

工作Fiddle