Почему этот объект не двигается?

Я новичок в Javascript и пытался сделать простую игру, в которой есть блок, и он может двигаться. Прямо сейчас я только сделал так, чтобы он двигался только влево, но это не так.

Я проверил опечатки и проверил консоль с помощью console.info, и она работает, но мой объект не перемещается.

const ctx=document.getElementById("ctx").getContext("2d");

var one = {
  width:30,
  height:30,
  x:250,
  y:200,
  color:"rgb(63, 154, 244)"
};
function draw(){
  ctx.save();
  ctx.fillStyle=one.color;
  ctx.fillRect(one.x,one.y,one.width,one.height);
  ctx.restore();
}
function move(){
  document.onkeydown=function(event){
    if (event.keyCode==65){
      one.x=one.x-15;
      console.info("test");
    }
  }
}

function startGame(){
  draw();
  setInterval(move,2000);
}
startGame();
<canvas id = "ctx" width = "1400" height = "500" style = "border:0.01em solid #fff"></canvas>

Я думал, что по нажатию "а" блок сдвинется влево на 15, но он не двигается. Он по-прежнему отображается синим цветом, но просто не двигается.

Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
Улучшение производительности загрузки с помощью Google Tag Manager и атрибута Defer
В настоящее время производительность загрузки веб-сайта имеет решающее значение не только для удобства пользователей, но и для ранжирования в...
Безумие обратных вызовов в javascript [JS]
Безумие обратных вызовов в javascript [JS]
Здравствуйте! Юный падаван 🚀. Присоединяйся ко мне, чтобы разобраться в одной из самых запутанных концепций, когда вы начинаете изучать мир...
Система управления парковками с использованием HTML, CSS и JavaScript
Система управления парковками с использованием HTML, CSS и JavaScript
Веб-сайт по управлению парковками был создан с использованием HTML, CSS и JavaScript. Это простой сайт, ничего вычурного. Основная цель -...
JavaScript Вопросы с множественным выбором и ответы
JavaScript Вопросы с множественным выбором и ответы
Если вы ищете платформу, которая предоставляет вам бесплатный тест JavaScript MCQ (Multiple Choice Questions With Answers) для оценки ваших знаний,...
0
0
41
1
Перейти к ответу Данный вопрос помечен как решенный

Ответы 1

Ответ принят как подходящий

Вот пример изменения вашего кода, вы можете его улучшить.

const ctx=document.getElementById("ctx").getContext("2d");

var one = {
  width:30,
  height:30,
  x:250,
  y:200,
  color:"rgb(63, 154, 244)"
};

function draw(){
  ctx.save();
  ctx.fillStyle=one.color;
  ctx.fillRect(one.x,one.y,one.width,one.height);
  ctx.restore();
}

// we're gonna register our event handler just one time.
document.onkeydown=function(event){
  if (event.keyCode==65){
    moveLeft();
  }
}

function moveLeft() {
  // I've added this line of code to erase the previous square from the canvas
  // Since we're gonna move your square to another location.
  // You'll probably want to move this code somewhere else
  // Can be inside a redraw function? Idk.
  ctx.clearRect(one.x, one.y, one.width, one.height);
  one.x=one.x-15;
  console.info("test");
}

function startGame(){
  // Let's call draw every 50ms
  setInterval(draw,50);
}

startGame();

Я переместил настройку обратного вызова onkeydown наружу, чтобы она была там с самого начала.

И в startGame(), вместо повторного вызова прежней функции move с интервалом, я вызвал draw, так как мы хотим перерисовать холста для визуального обновления игры.

Проблема с вашим кодом была:

  1. В вашем startGame вы неоднократно вызывали move каждые 2 секунды, в которых функция просто регистры обработчика событий для document.onkeydown, на самом деле ничего не делает.

  2. Поскольку обработчик регистрируется, вы можете быть уверены, что значение one.x действительно обновляется при нажатии клавиши А, вы не можете просто проверить его, потому что холст никогда не обновляется.

Вот рабочий JSFiddle: https://jsfiddle.net/8qjcfya9/

ОБНОВИТЬ

Я просто отрефакторил часть кода, все равно работает так же.

Другие вопросы по теме