Перекрытие не работает с использованием JavaScript в Phaser

Question

Я делаю игру, и в этой игре я хочу, чтобы, когда игрок перекрывал монету, монета исчезала, но это не по какой-то причине, и я не знаю почему. Я пробовал много способов, даже помещая монету в плитку, когда я делал карту, но даже тогда она все еще не работает.

Кто-нибудь может мне помочь, пожалуйста?

 var game = new Phaser.Game(800, 600, Phaser.CANVAS, 'phaser-example', { 
 preload: preload, create: create, update: update, render: render });

 function preload() {
game.load.tilemap('level12', 'res/level12.json', null, 
Phaser.Tilemap.TILED_JSON);
game.load.image('sprite12', 'res/sprite12.png');
game.load.spritesheet('dude', 'res/player.png', 64, 64);
game.load.spritesheet('droid', 'res/droid.png', 32, 32);
game.load.image('starSmall', 'res/star.png');
game.load.spritesheet('coin', 'res/coin.png',32,32 );
game.load.image('background', 'res/sprite3.png');

}

var map;
var tileset;
var layer;
var player;
var facing = 'left';
var jumpTimer = 0;
var cursors;
var jumpButton;
var bg;
var coin;


function create() {
game.physics.startSystem(Phaser.Physics.ARCADE);
game.stage.backgroundColor = '#000000';


bg = game.add.tileSprite(0, 0, 800, 600, 'background');
bg.fixedToCamera = true;

map = game.add.tilemap('level12');
map.addTilesetImage('sprite12');
map.addTilesetImage('coin');
map.setCollisionBetween(1, 12);
map.setCollisionByExclusion([ 13, 14, 15, 16, 46, 47, 48, 49, 50, 51 ]);

layer = map.createLayer('Tile Layer 1');
layer.resizeWorld();

coin = game.add.group();
coin.enableBody = true;
coin.physicsBodyType = Phaser.Physics.ARCADE;
createcoin();


game.physics.arcade.gravity.y = 200;

player = game.add.sprite(150, 900, 'dude');


game.physics.enable(player, Phaser.Physics.ARCADE);

player.body.bounce.y = 0.2;


player.body.collideWorldBounds = true;



player.animations.add('left', [3,2,1,0], 10, true);
player.animations.add('right', [4,5,6,7], 10, true);


game.camera.follow(player);

cursors = game.input.keyboard.createCursorKeys();
jumpButton = game.input.keyboard.addKey(Phaser.Keyboard.SPACEBAR);


game.physics.arcade.overlap(coin, player, killcoin(), null, this);

}
function killcoin(coin,player) {

coin.kill();
}



function  createcoin() {
coin = game.add.sprite(50,700, 'coin');
}

function update() {

game.physics.arcade.collide(player, layer);

player.body.velocity.x = 0;

if (cursors.left.isDown) {
    player.body.velocity.x = -150;

    if (facing != 'left') {
        player.animations.play('left');
        facing = 'left';
    }
}
else if (cursors.right.isDown) {
    player.body.velocity.x = 150;

    if (facing != 'right') {
        player.animations.play('right');
        facing = 'right';
    }
}
else {
    if (facing != 'idle') {
        player.animations.stop();

        if (facing == 'left') {
            player.frame = 0;
        }
        else {
            player.frame = 5;
        }

        facing = 'idle';
    }
}

if (jumpButton.isDown && player.body.onFloor() && game.time.now > jumpTimer) {
    player.body.velocity.y = -250;
    jumpTimer = game.time.now + 750;
}
}

function render () {



}

Answer 1

В создаваемой вами функции создания, в обновляющей функции вы обновляете, убийство монеты - это не начальное состояние, это то, что должно произойти в определенных условиях, эти две строки

game.physics.arcade.overlap(coins, player, killcoin, null, this); //killcoin is callback so no(), coins with "s" as you are calling the whole group
function killcoin(coin,player) {
    coin.kill();
}

принадлежит функции обновления. У вас все еще могут быть проблемы с областью действия.

также вы должны создать группу для монет, что-то вроде этого, и вы создаете их, чтобы они пошли, чтобы создать функцию

        coins= game.add.group();// group of many coins
        coins.enableBody = true;
        for (var i = 0; i<12; i++){
            var coin = coins.create(i*70,0, 'coin');// var individual coin
            coin.body.gravity.y = 6;
            coin.body.bounce.y = 0.7 +Math.random()*0.2;
        }

Кроме того, CamelCasing является отличным дополнением к вашему стилю, поскольку он улучшает читабельность (killCoin, createCoin...)

Answer 2

В вашем коде есть три проблемы. Прежде всего, именование переменных сбивает с толку. Вы создаете группу и назначаете ее coin но вы также добавляете новые спрайты и назначаете их coin, Итак, какова ценность coin при вызове функции перекрытия? Может быть, это работает правильно на техническом уровне, но просто чтобы избежать путаницы, я бы использовал разные имена переменных для разных вещей. Если ничего другого, это делает ваш код более читабельным.

Во-вторых, спрайты монет, которые добавляются в игру в createcoin() не добавляются в группу. И поскольку он не добавлен в группу, физическое тело спрайта никогда не включается.

Наконец, arcade.overlap функция вызывается только один раз в create(). Я думаю, что это должно быть вызвано в функции update(). Функция update() в Phaser.js является своего рода основным игровым циклом. Он вызывается для каждого обновления кадра.

Так что сложите все эти три исправления вместе, и я бы попробовал что-то вроде этого:

grpcoins = game.add.group();
grpcoins.enableBody = true;
grpcoins.physicsBodyType = Phaser.Physics.ARCADE;
//..

function  createcoin() {
    var c = game.add.sprite(50,700, 'coin');
    grpcoins.add(c);
}

function update() {
   //..
   game.physics.arcade.overlap(player, grpcoins, killcoin, null, this);
   //..
}

function killcoin(pl, cn) {
    cn.kill();
}