Question

我似乎无法让以下工作：我得到了这个方法checkEnemyHealth（）通过更改精灵和添加杀戮等来照顾敌人的健康。但是，我需要更改特定Playertank的杀戮对象，所以我尝试了以下代码：

public void collisionOccured(GameItem collidedItem)
{
    if(collidedItem instanceof Playertank) //Botsing met player
    {
        enemyHealth = 0.0;      
    }       
    if (collidedItem instanceof Weapon) //Botsing met wapen
    {
        lowerEnemyHealth(Weapon.weaponDamage);
        System.out.println("loweredddd");
    }
    if ((collidedItem instanceof Weapon) || (collidedItem instanceof Playertank))
    {
    checkEnemyHealth((GameItem)collidedItem);
    }
}

    public void checkEnemyHealth(GameItem collidedItem)
{
    if (enemyHealth >= (2.0/3.0 * startupEnemyHealth))
    {
        this.setFrame(0); //Green Enemy
        this.setEnemyStage(0);
    }
    else if (enemyHealth < (2.0/3.0 * startupEnemyHealth) && enemyHealth >= (1.0/3.0 * startupEnemyHealth))
    {
        this.setFrame(1); //Orange Enemy
        this.setEnemyStage(1);
    }
    else if (enemyHealth < (1.0/3.0 * startupEnemyHealth) && enemyHealth > 0)
    {
        this.setFrame(2); //Red Enemy
        this.setEnemyStage(2);
    }
    else if (enemyHealth <= 0 && (getEnemyStage() != 3))
    {
        setImage("/images/death.png", 27, 33); //Dead enemy     
        enemyOnDeath();
        if (collidedItem instanceof Playertank)
        {
            ((Playertank)collidedItem).raisePlayerKills(1); 
            mygame.setKillsonDashboard(((Playertank)collidedItem).getPlayerKills());
            ((Playertank)collidedItem).raisePlayerLandmines(1);
            mygame.setLandminesonDashboard(((Playertank)collidedItem).getPlayerLandmines());
        }
        mygame.setTimer(10, 33, this);
    }
    printHealth();
    System.out.println(this.getEnemyStage());
}

但是，即使实例是武器，我也需要执行raisePlayerKills（）。我似乎无法找到解决方案。

更多信息：

Playertank.java creates the gameplayer and extends GamePlayer.java
GamePlayer.java extends MoveableGameItem.java
Enemytank.java extends MoveableGameItem.java
MoveableGameItem.java extends GameItem.java

我希望它足以解决这个问题， Thx提前火焰

Answer 1

首先，我建议在调试器中浏览代码。这很可能会告诉你什么是null。似乎碰撞的项目为空。您确定要将实例化对象传递给方法checkEnemyHealth吗？

Answer 2

此强制转换无效：checkEnemyHealth((GameItem)collidedItem); - collidedItem已经是GameItem类型。

此向下广播保证会失败：checkEnemyHealth((Playertank)collidedItem);因为我认为Playertank不是Weapon的子类，反之亦然。

总的来说，您发布的确切代码无法引发NullPointerException，因为没有代码路径可以使用checkEnemyHealth调用null - 这是获得NPE之后的唯一途径' beforreee'，但不是'afterrrr'之后。那么让我问你这个问题：你的代码会抛出什么异常？我怀疑是ClassCastException：

((Playertank)collidedItem).raisePlayerKills(1);

这是对Playertank的未经检查的贬值。

将对象作为参数传递

2 个答案: