创建一个垂直的侧卷轴,遇到了arraylists和逻辑(Java)的问题

时间:2014-09-04 00:22:51

标签: java arraylist logic collision-detection side-scroller

我是编程的第二年,只知道Java。但是对于我的班级,我们必须创建一个垂直的侧卷轴。我的设置是我有一个用于敌舰的ArrayList,以及一个用于玩家火箭的ArrayList。我在渲染(Graphics2D g)方法中在玩家船上的精确点上绘制2枚火箭,并且敌人以相同的方法运送。火箭ArrayList由

填充
arsenal.add(r);
arsenal.add(r2);

其中r和r2是带有参数(x,y,w,h)的火箭,而阿森纳是火箭的arraylist。我把它们设置在宇宙飞船不同端的非常特定的地方。在update()方法中,我有这个方法:

if (arsenal.size() < 0){
        arsenal.add(new Rocket(p.currentX(), p.getHalfY(),30,30));
        arsenal.add(new Rocket(p.getHalfX(), p.getHalfY(), 30, 30));
    }

这是为了确保当arraylist变空时,火箭重新添加并重新绘制到与宇宙飞船末端相对应的坐标。火箭消失的唯一方法是当他们离开屏幕或与敌人发生碰撞时。我已将此代码放入以检查:

for(int x = army.size()-1; x >= 0; x--){
        if (army.get(x).getR().y > 1024){
            army.remove(x);
        }
        if(p.intersects(army.get(x).getR())){
            army.remove(x);
            p.changeH(10);
        }
        for (int q = arsenal.size()-1; q>=0;q++){
            if (arsenal.get(q).intersects(army.get(x).getR())){
                score+=20;
                army.remove(x);
                arsenal.remove(q);
            }
        }
}

这整个方法通过敌舰的arraylist“军队”并检查(1)敌舰是否超过设定高度,(2)如果玩家船与敌舰发生碰撞,(3)如果阿森纳arraylist中的火箭与敌人发生碰撞。所有碰撞都是使用矩形完成的。 p.changeH(10);命令是将玩家的生命值降低10,分数+ = 20;是增加分数。对于所有这些检查,我确保敌人从arraylist中移除并因此变得未拉伸,然后如果火箭离开屏幕或碰撞,他们也将自己从arraylist中移除并假设自己画下来。最后我想要的东西提到的是,armlust arralust变成了:

if (frameCount % 100 == 0){
            Random r = new Random();
            int randX = r.nextInt(width - 10);
            army.add(new EnemyShip(randX, -200 , 47, 47, enemyP1));
}

它接收参数(x,y,w,h,Image)并首先在屏幕外绘制,使其看起来像一个侧卷轴。 x变量是随机的,因此船只并不都出现在同一个地方。每次更新时frameCount都会增加1,而update()方法每秒调用60次。

在对我的项目进行了大量介绍之后,我的问题是,无论何时我运行游戏,游戏都会在几秒后停止,而eclipse会在代码中给我一个错误:

if (arsenal.get(q).intersects(army.get(x).getR())){

我无法弄清楚问题。任何帮助是极大的赞赏。如果需要更多游戏代码,请告诉我!谢谢!

编辑:

GameScreen.update(GameScreen.java:108)

带我上线:

 if (arsenal.get(q).intersects(army.get(x).getR())){

是循环的一部分:

for (int q = arsenal.size()-1; q>=0;q++){
            if (arsenal.get(q).intersects(army.get(x).getR())){
                score+=20;
                army.remove(x);
                arsenal.remove(q);
            }
}

在构造函数中我放置命令

arsenal.add(r);
arsenal.add(r2);

这会将它们添加到arraylist中。在render(Graphics2D g)方法中,我使用:

绘制它们
for (Rocket r :arsenal){
            r.draw(g);
}

在火箭构造函数中,p.currentX()是玩家当前的x坐标,p.getHalfY()和p.getHalfX()是自解释的。对它们进行编码,使得火箭r出现在一侧,火箭r2出现在另一侧。我还创建了两个布尔变量rMove和r2move。这些有助于火箭的平稳运动,并告诉它何时移动:

if (!r2move){
        r2.setP(p.getR().x + 40, currentY + 12);
}
    if(!rMove){
        r.setP(p.currentX() - 10, currentY + 12);
}

setP(x,y)将火箭的位置设置为原始位置。我和老师谈过这件事,他说我有点使用arraylists和单独的对象,但是我不明白这个问题。如果您需要任何更相关的代码,请告诉我们。 这行代码还告诉火箭什么时候移动,那就是当空格按钮被触发时:

if (rMove && r2move){
    for (Rocket p:arsenal){
        p.move();
    }
}
if(r.getR().y < 0){     
        rMove = false;
        space = false;
}
if (r2.getR().y < 0){
        r2move = false;
        space = false;
}

1 个答案:

答案 0 :(得分:1)

异常告诉你的是它有ArrayList,其中包含2个元素,这意味着它的有效索引是0和1.但是你的代码在这个ArrayList上调用.get()方法索引值为2,显然不存在并导致此异常。

查看代码,问题似乎在于如何在以下行中初始化和使用循环变量q

for (int q = arsenal.size()-1; q>=0;q++){
            if (arsenal.get(q).intersects(army.get(x).getR()))

假设arsenal中包含2个元素,因此大小为2.因此,q将使用值1进行初始化。 将调用if语句,并将调用arsenal.get(1)。这是arsenal Arraylist的有效索引,因此此代码将通过而不会出现任何错误。

在循环的下一次传递中,q将增加1并将变为2.根据循环检查条件2 > 0,执行将继续到下一行。将再次调用if语句,并将调用arsenal.get(2)。由于2是arsenal的无效索引,只有2个项目,因此会看到您看到的异常。

您可以通过递减 q的值来解决此问题,而不是在for循环中递增它。