Java:尝试更改矩形位置时的ArrayIndexOutOfBoundsException

时间:2012-04-07 04:09:30

标签: java arrays

我正在开发Java游戏,目前正在编写地图制作者。我可以制作地图并绘制图块,但我需要能够更改这些图块的位置,以便角色可以看到地图的不同位置。当我尝试更改它时,在moveMap()方法中,它给了我这个错误:

Exception in thread "Thread-2" java.lang.ArrayIndexOutOfBoundsException: 570
at Base.moveMap(Base.java:88)
at Base.run(Base.java:55)
at java.lang.Thread.run(Unknown Source)

我不知道为什么会这样 - 请有人帮我理解这个问题。有没有其他方法来移动瓷砖?

这是我的代码......

public class Base extends JPanel implements Runnable {

    private static String[] line = { 
        "wwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwww",
        "wwwwwwwwwwwwwwwwfffffffffffwwwwwwwwwww", 
        "wwwwwwffwwwwwwwwfwwwwwwwwwwwwwwwwwwwww",
        "wwwwwwfffffffwwwfwwwwwwwwwwwwwwwwwwwww",
        "wwwwwwffwwwffffffwwwwwwwwwwwwwwwwwwwww",
        "wwwwwwffwwwffffffwwwwwwwwwwwwwwwwwwwww",
        "wwwwwwfffffffwwwwwwwwwwwwwwwwwwwwwwwww",
        "wwwwwwffwwwwwwwwwwwwwwwwwwwwwwwwwwwwww", 
        "wwwwwwffwwwwwwwwwwwwwwffffffffwwwwwwww",
        "wwwwwwffwwwwwwwwwwwwwwwwwwwwffwwwwwwww",
        "wwwwffffffwwwwwwwwwwwwwwwwwwffwwwwwwww",
        "wwwwffffffffffffffffffffffffffwwwwwwww",
        "wwwwffffffwwwwwwwwwwwwwwwwwwwwwwwwwwww",
        "wwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwww",
        "wwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwww",};

    private Rectangle[] colRect;

    private int tileWidth = 30;
    private int tileHeight = 30;

    public Base() {
        colRect = new Rectangle[line.length * line[0].length()];
        for (int i = 0; i < line.length; i++) {
            for (int f = 0; f < line[i].length(); f++) {
                colRect[counter] = new Rectangle(f * tileWidth, i * tileHeight,tileWidth, tileHeight);
                if (counter != colRect.length) {
                    counter += 1;
                }
            }
        }
    }

    public void moveMap(){
        for(int i = 0; i <= colRect.length; i++){
            colRect[i].setLocation(colRect[i].x+1, colRect[i].y+1);
        }
    }
}

2 个答案:

答案 0 :(得分:3)

你几乎肯定是这个意思:

for (int i = 0; i < colRect.length; i++) {

而不是:

for(int i = 0; i <= colRect.length; i++){

请记住,如果数组的长度为n,则索引从0到n - 1。

答案 1 :(得分:0)

我建议您只修改几何体的访问器,而不是地图本身。因为看起来,函数moveMap可能会有某种“循环”行为

getLocation (int index, offset = 0){
    int accIndex = (index + offset) % colRect.length;
    // ... probably better to modify your data-structure to simplify the handling
}