Javascript HTML5 Canvas Mario Bros NES 克隆、碰撞和跳跃损坏

Question

我希望有人可以查看我一直在为简单的老式 Mario 克隆工作的 javascript 代码。我从几个教程中拼凑了我对 Canvas 的了解，但我无法正确处理与 block 的碰撞或跳跃。

跳跃似乎让马里奥陷入无限循环，一遍又一遍地弹跳，这看起来很有趣，但不太适合玩游戏!

       function Player() {
     this.srcX = 0;
     this.srcY = 0;
     this.drawX = gameWidth /2;
     this.drawY = 0;
     this.scaleWidth = 38;
     this.scaleHeight = 50;
     this.width = 48;
     this.height = 60;
     this.speed = 10;
     this.maxJump = 50;
     this.velY = 0;
     this.velX = 0;
     this.isJumpKey = false;
     this.isRightKey = false;
     this.isCrouchKey = false;
     this.isLeftKey = false;
     this.jumping = false;
     this.grounded = false;
    }


    Player.prototype.draw = function(){
      clearPlayer();
      this.checkKeys();
      ctxPlayer.drawImage(
        player,
        this.srcX,
        this.srcY,
        this.width,
        this.height,
        this.drawX,
        this.drawY,
        this.scaleWidth,
        this.scaleHeight);
    };
Player.prototype.checkKeys = function () {


 if(this.isJumpKey){

    if (!this.jumping && this.grounded ) {
        this.jumping = true;
        this.grounded = false;
        this.velY = -this.speed * 2;
    }

 }

 if(this.isRightKey){

   if (this.velX < this.speed) {
            this.velX++;
        }

 }
  if(this.isLeftKey){
  if (this.velX < this.speed) {
            this.velX--;
        }
 }
 if(this.isCrouchKey){
      player1.grounded = true;
      player1.jumping = false;
}


};

这是我现在所在位置的代码笔:http://codepen.io/AlexBezuska/pen/ysJcI

我非常感谢任何帮助，在此期间我将继续搜索和尝试，但是您可以提供的任何指示，甚至是关于格式化、原型(prototype)创建等的建议都非常感谢(我对 canvas 和原型(prototype))

Answer 1

在您的 checkKeyDown() 和 checkKeyUp() 函数中，您让它们检查不同的“跳转”键。来自 checkKeyDown():

if (keyID === 74) { //spacebar
    e.preventDefault();

    player1.isJumpKey = true;
}

来自 checkKeyUp():

if (keyID === 32) { // spacebar
    player1.isJumpKey = false;
    e.preventDefault();
}

所以 checkKeyUp() 没有正确重置 player1.isJumpKey。将它们设置为相同，对我来说效果很好。

一般而言，可能值得设置一个对象来保存代码中具有多个实例的所有参数。然后通过引用这个对象将它们写入你的代码中。这样您只需在一个地方更改它们:

CONSTS = {
    upKeyID: 32,
    etc.
}

// then later:

if (keyID === CONSTS.upKeyID) {
    player1.isJumpKey = false;
    e.preventDefault();
}