Попытка заставить div перемещаться по экрану при нажатии клавиш - PullRequest
Новая
       27

Попытка заставить div перемещаться по экрану при нажатии клавиш

0 голосов
/ 02 марта 2019

Я пытаюсь создать игру, в которой игроку приходится перемещать мышь по лабиринту, чтобы найти кусок сыра.У меня возникли проблемы с тем, чтобы заставить div двигаться, как я хочуПроблема в том, что когда вы нажимаете клавишу со стрелкой вниз (клавиша, которую я использую для проверки), она не будет уменьшаться с шагом в десять раз, как я хочу.Я пробовал петли, весь шебанг, но я просто не могу понять, как заставить это работать. 

document.onkeydown = function(key){
    if (key.keyCode == 38){
        console.log("up");
    } else if (key.keyCode == 39){
        console.log("right");
    } else if (key.keyCode == 37){
        console.log("left");
    } else if (key.keyCode == 40){
        mouse.style.top = 10 + "px";
    };
};

Ответы [ 3 ]

0 голосов
/ 02 марта 2019

Вы можете использовать объект Player и изменять его свойства x и y.
Создать также объект K, чтобы установить логическое значение используемых в настоящее время ключей чисел.
Чем внутри цикла, как requestAnimationFrame переместить игрока: 

const Player = {
  el: document.getElementById('mouse'),
  x: 200,
  y: 100,
  speed: 2,
  move() {
    // If no collision with map or tiles... (TODO!)
    this.el.style.transform = `translate(${this.x}px, ${this.y}px)`;
  }
};
 
const K = {
  fn(ev) {
    ev.preventDefault();
    const k = ev.which;
    if (k >= 37 && k <= 40) K[k] = ev.type === "keydown"; // If is arrow
  }
};
  
const update = () => {
    // 1 / Math.sqrt(2) = 0.7071.. (Diagonal)
    let dist = K[38] && (K[37] || K[39]) || K[40] && (K[37] || K[39]) ? 0.707 : 1;
    dist *= Player.speed;
    if (K[37]) Player.x -= dist;
    if (K[38]) Player.y -= dist;
    if (K[39]) Player.x += dist;
    if (K[40]) Player.y += dist;
    Player.move();
}

document.addEventListener('keydown', K.fn);
document.addEventListener('keyup', K.fn);

(function engine() {
  update();
  window.requestAnimationFrame(engine);
}());
#mouse{
  position: absolute;
  left: 0;  top: 0;
  width: 20px;  height: 20px;
  background: #000;  border-radius: 50%;
}
Click here to focus, and use arrows
<div id="mouse"></div>

A Более простой пример, который перемещает элемент на Npx шагами и повторяется какпока вы удерживаете нажатой клавишу (как при наборе текста) будет: 

const Player = {
  el: document.getElementById('mouse'),
  x: 0,
  y: 0,
  step: 10,
  move() {
    this.el.style.transform = `translate(${this.x}px, ${this.y}px)`;
  }
};

const keysHandler = (ev) => {
  const k = ev.which;
  if (k>=37 && k<=40) {
    ev.preventDefault();
    ({
      '37' () {Player.x -= Player.step},
      '38' () {Player.y -= Player.step},
      '39' () {Player.x += Player.step},
      '40' () {Player.y += Player.step},
    }[k]());
    Player.move();
  }
}

document.addEventListener('keydown', keysHandler);
#mouse{
  position: absolute;
  left: 0;  top: 0;
  width: 20px;  height: 20px;
  background: #000;  border-radius: 50%;
}
Click here to focus, and use arrows
<div id="mouse"></div>
0 голосов
/ 02 марта 2019

Даже если другой ответ использовал parseInt, лучше хранить переменные внутри кода 

var x = 0;
var y = 0;

document.onkeydown = function(key){
    if (key.keyCode == 38){
        y -= 10;
    } else if (key.keyCode == 39){
        x += 10;
    } else if (key.keyCode == 37){
        x -= 10;
    } else if (key.keyCode == 40){
        y += 10;
    };
    
    document.getElementById("block").style.left = x+"px";
    document.getElementById("block").style.top = y+"px";
};
#block
{
  top: 0;
  left: 0;
  position: absolute;
  width: 30px;
  height: 60px;
  background: teal;
}
<div id="block">
0 голосов
/ 02 марта 2019

Вы устанавливаете значение 10 каждый раз, сначала используйте split, чтобы получить значение: 

else if (key.keyCode == 40){
    mouse.style.top = parseInt(mouse.style.top.split("px")[0]) + 10 + "px";
}
...