动态数组创建导致ArrayIndexOutOfBoundsException&空指针异常

时间:2013-11-15 18:42:54

标签: java arrays nullpointerexception indexoutofboundsexception

好的。因为我不能自己看到这个而感到非常愚蠢,但我被困住了。第一个代码块与发生错误的其余代码相关。提前致谢! :d

此代码确定将出现多少怪物(数组的大小)。

    public static byte monsters()
{
    byte b = -7;                                //identifies how many monsters will be found

    chance = (byte)d20.nextInt(30);     //dice roll

    if(chance >= 24)
    {
        b = 4;          //quadruple monsters
    }           
    else if(chance >= 18)
    {
        b = 3;          //triple monsters
    }
    else if(chance >= 12)
    {
        b = 2;          //double monsters
    }
    else if(chance >= 6)
    {
        b = 1;          //single monster
    }
    else
    {
        b = 0;          //no monster
    }
    return b;
}//end monsters()

此代码确定并生成要放入数组中的怪物。第一部分从上面的代码中获取输出以确定大小。第二部分产生怪物。当抛出“NullPointerException”时,这是它指向的代码,特别是“for(x = 0; x)

public void determineMons()
{
    byte x = 0;                         //counter

    switch(monsters())                  //defines the array
    {
        case 4:
            monsters = new Monster[4];
            break;

        case 3:
            monsters = new Monster[3];
            break;

        case 2:
            monsters = new Monster[2];
            break;

        case 1:
            monsters = new Monster[1];
            break;
    }//end switch

    for(x=0;x<monsters.length;x++)          //populates the array
    {
        chance = (byte)d20.nextInt(20);     //dice roll
        if(chance >= 15)
        {
            monsters[x] = new NazRuel();
        }           
        else if(chance >= 10)
        {
            monsters[x] = new GiantSnake();
        }
        else if(chance >= 5)
        {
            monsters[x] = new Yeti();
        }
        else
        {
            monsters[x] = new Zombie();
        }
    }//end fill For
}//end determineMons()

这是“ArrayOutOfBoundsExceptions”代码。不过,错误在不同情况之间反弹,每次错误行都是“monster =”行。

        determineMons();
    switch(Cell.monsters())
    {
        case 4:
            monster = "There is a " + monsters[0] + ", a " + monsters[1] + ", a " + monsters[2] + ", and a " + monsters[3] + " in this area!";
            break;

        case 3:
            monster = "There is a " + monsters[0] + ", a " + monsters[1] + ", and a " + monsters[2] + " in this area!";
            break;

        case 2:
            monster = "There is a " + monsters[0] + ", and a " + monsters[1] + " in this area!";
            break;

        case 1:
            monster = "There is a " + monsters[0] + " in this area!";
            break;
    }//end Monster block

1 个答案:

答案 0 :(得分:1)

NullPointerException

如果monsters()返回0,则交换机不会初始化变量怪物(没有case 0也不会default)并且当您执行NullPointerException时会生成monsters.length在for循环中{1}}。

此外,您应该更改开关:

switch(monsters())                  //defines the array
{
    case 4:
        monsters = new Monster[4];
        break;

    case 3:
        monsters = new Monster[3];
        break;

    case 2:
        monsters = new Monster[2];
        break;

    case 1:
        monsters = new Monster[1];
        break;
}

为:

int b = monsters();
if (b > 0) {
    monsters = new Monsters[b];
}

这不是一个错误,但这会让你的代码更清晰。

越界异常

您拨打monsters()两次。换句话说,您通过第一次调用monsters来创建数组monsters(),并使用另一个调用monsters()来读取monsters的内容,以确定元素的数量。

例如,假设在第一次调用monsters()时,返回的值为3。此时,您将创建一个包含3个元素的数组。当您要打印monsters的内容时,monsters()会返回4。因此,在尝试阅读第4个元素时,您将发出一个出界错误。

提醒一下,monsters()返回的值是随机的:

chance = (byte)d20.nextInt(30);     //dice roll

要解决此问题,请尝试更改:

switch(Cell.monsters())

switch(monsters.length)