[java] ArrayIndexOutOfBoundsException

[DonnyR]

Hey,

Ik ben bezig met een project genaamd Sokoban.
Dit is een spelletje waarmee je met een poppetje door een level loopt en dozen op een eindveld moet schuiven.

Oke, ik wil een 2 dimensional array vullen met verschillende Veld objecten.
Maar nu krijg ik de onderstaande foutmelding:
Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: 9

Als ik het goed heb begrepen houdt dit in dat ik een index in de array willen vullen die niet bestaat.

Dus dan zou of mijn array demension of mijn for loop niet goed moeten zijn.

Maar zie echt niet waar ik de fout ben in gegaan.

Als iemand de moeite zou willen nemen om even een kijkje te nemen in mij code zou ik dat erg op prijs stellen.

ps: in de string map zouden nog wat spaties moeten staan maar die haalt hij eruit dus vandaar dat het zo rommelig eruit ziet

Code:

Java:

private Veld veld [][];

public void maakLevel(){

        String map= "#############\n"+
                    "#           #\n"+
                    "#  $  ##    #\n"+
                    "#  $  ##    #\n"+
                    "#       $   #\n"+
                    "#  @  ##    #\n"+
                    "#     ##    #\n"+
                    "#...        #\n"+
                    "#############\n";
        String regels[] = map.split("\n");
        veld = new Veld[regels.length][regels[0].length()];
        int row = 0;

        for (String regel : regels){
            for (int c = 0; c < regel.length(); c++){

            char item = regel.charAt(c);

            if(item == '#'){
                veld[row][c] = new Muurveld(row,c);

            }
            row ++;
        }
    
    }
}

Staat nog een lap code onder voor de andere tekens zoals (@,$, enz) maar die kan ik zelf wel fixen zodra de 1e lukt.

Donny

[NMe]

Wat zegt je debugger op die regel waar het fout gaat? Welke index van welk array probeer je aan te spreken? Hoe groot is die array? Hoe groot is je index?

Verder: Waar hoort mijn topic? Dit is een programmeerprobleem, niet een ontwerpprobleem; ik verplaats je topic dan ook. En voor het posten van code: Hoe post je code? / Hoe gebruik je de code tag?

Nog een algemene tip: zorg voor goede indenting. Ik moest drie keer kijken om te zien onder welke loop die row ++ op regel 27 nou viel.

[Voor 3% gewijzigd door NMe op 02-06-2011 15:12]

[DonnyR]

NMe schreef op donderdag 02 juni 2011 @ 15:11:
Wat zegt je debugger op die regel waar het fout gaat? Welke index van welk array probeer je aan te spreken? Hoe groot is die array? Hoe groot is je index?

Verder: Waar hoort mijn topic? Dit is een programmeerprobleem, niet een ontwerpprobleem; ik verplaats je topic dan ook. En voor het posten van code: Hoe post je code? / Hoe gebruik je de code tag?

Nog een algemene tip: zorg voor goede indenting. Ik moest drie keer kijken om te zien onder welke loop die row ++ op regel 27 nou viel.

Bedankt voor de feedback, ik heb het probleem gevonden.
Ging even kijken naar me indenting en zag dat me row++ nog binnen de 2e for zat.
Dus vandaar die index out of bounds.
Domme fout die me 2 uur zoeken heeft gekost
Maar nu is er nog iets niet in orde, hij zit blijkbaar in een oneindige for loop

[Voor 4% gewijzigd door DonnyR op 02-06-2011 15:24]

[Soultaker]

Of het je oneindige lus oplost weet ik niet, maar je veldbeschrijving is momenteel niet bepaald rechthoekig. Dat lijkt me ook niet de bedoeling! Zo is de eerste regel bijvoorbeeld korter dan de tweede, waardoor je geheid problemen gaat krijgen aangezien de breedte van je veld-array bepaald wordt door de lengte van de eerste regel.

[DonnyR]

Soultaker schreef op donderdag 02 juni 2011 @ 15:29:
Of het je oneindige lus oplost weet ik niet, maar je veldbeschrijving is momenteel niet bepaald rechthoekig. Dat lijkt me ook niet de bedoeling! Zo is de eerste regel bijvoorbeeld korter dan de tweede, waardoor je geheid problemen gaat krijgen aangezien de breedte van je veld-array bepaald wordt door de lengte van de eerste regel.

Hij was in me code wel gewoon recht, heb hem hier ook even aangepast.

[NetForce1]

Als ik het zo snel zie lijkt je oneindige loop niet in dit stuk code te zitten. Dmv een debugger, of output naar de console printen kun je snel genoeg zien welk stuk code de oorzaak is.

[DonnyR]

NetForce1 schreef op donderdag 02 juni 2011 @ 17:04:
Als ik het zo snel zie lijkt je oneindige loop niet in dit stuk code te zitten. Dmv een debugger, of output naar de console printen kun je snel genoeg zien welk stuk code de oorzaak is.

Klopt ja,
Zodra ik de method tekenObjecten aanroep in paintComponent(), wordt me spel zwart en kan ik hem enkel afsluiten via taakbeheer.

Java:

private static final int Hokje = 20;

    public void tekenObjecten(Graphics g){
        for (int row = 0; row < veld.length; row++){
            for (int col = 0; col < veld[0].length;){
                                
                if (veld[row][col] instanceof Muurveld){
                    g.setColor(Color.RED);
                    g.fill3DRect(veld[row][col].getxPosition(), veld[row][col].getyPosition(), Hokje, Hokje, true);
                }

            }
        }
    }

[Caelorum]

DonnyR schreef op donderdag 02 juni 2011 @ 18:33:
[...]

Geen idee waar de fout zit, maar mocht je ooit per ongeluk niet rechthoekige maps krijgen, dan lijkt het me dat dit wel eens een probleem gaat opleveren:

Java:

for (int col = 0; col < veld[0].length;){

Je kan het beter die 0 veranderen in row

Java:

for (int col = 0; col < veld[row].length;){

[DonnyR]

Caelorum schreef op donderdag 02 juni 2011 @ 18:46:
[...]

Geen idee waar de fout zit, maar mocht je ooit per ongeluk niet rechthoekige maps krijgen, dan lijkt het me dat dit wel eens een probleem gaat opleveren:

Java:

for (int col = 0; col < veld[0].length;){

Je kan het beter die 0 veranderen in row

Java:

for (int col = 0; col < veld[row].length;){

Bedankt voor de tip en moeite

[Soultaker]

Mist daar op die regel met de for-lus niet gewoon nog een col++ op het eind?

[DonnyR]

Soultaker schreef op donderdag 02 juni 2011 @ 20:02:
Mist daar op die regel met de for-lus niet gewoon nog een col++ op het eind?

Wow je bent een held!
Dat was het!
Altijd iets heel erg logisch doe ik fout...

[Caelorum]

Haha ik had em ook niet gezien. Mooi gespot soultaker