ArrayIndexOutOfBoundsException

[guilhelm]

ich habe einige Themen über dieses Problem im Forum gelesen, ich habe aber keine Lösung für mein Problem gefunden.

Wir programmieren das Spiel "4 Gewinnt" ohne graphische Darstellung und haben ein Problem, wenn das Programm überprüfen soll, ob der Spieler gewonnen hat.

Beim Compilieren gibts kein Problem, nur wenn man spielt kommt so was:

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: -2
at Logik.gewonnen(Logik.java:50)
at Spiel.spielen(Spiel.java:65)
at Start.main(Start.java:22)

public class Start
{
	public static void main(String[] args)
	{
		System.out.print("Wie heisst der 1. Spieler? ");
		String name1 = Tools.stringEingabe();
		System.out.print("Wie heisst der 2. Spieler? ");
		String name2 = Tools.stringEingabe();
		
		char[][] spielfeld = new char[6][7];
		for(int j = 0; j <= 6; j++)
		{
			for(int k = 0; k <= 5; k++)
			{
				spielfeld[k][j] = ' ';
			}
		}
		
		Spiel meinSpiel = new Spiel();
		
		meinSpiel.ausgabe(spielfeld);
		meinSpiel.spielen(spielfeld, 1, name1, name2);
	}
}

public class Spiel
{
	public char stein;
	public int dran;
	public int zeile;
		
	public void spielen(char[][] spielfeld, int dran, String n1, String n2)
	{
		Logik l = new Logik(false);
		boolean g = false;
		
		for(int i = 1; i <= 42 && g == false; i++)
		{
			if(dran == 1)
			{ 
				stein = 'X';
				
				System.out.println(n1+" ist dran!");
				System.out.print("In welche Spalte? ");
				int spalte = Tools.intEingabe();
				
				if(spielfeld[5][spalte] == ' ')
				{
					spielfeld[5][spalte] = stein;
				}
				else
				{
					for(int z = 0; spielfeld[z][spalte] == ' ' && z <= 4; z++)
					{
						zeile = z;
					}
					spielfeld[zeile][spalte] = stein;
				}
				//Diese Zeile sollte die Zeile 65 sein (haben viele unwichtige Zeilen gelöscht)
				g = l.gewonnen(stein, spielfeld, zeile, spalte);

public class Logik
{
	boolean gewonnen;
	
	public Logik(boolean gewonnen)
	{
		this.gewonnen = gewonnen;
	}
	
	boolean gewonnen(char stein, char[][] sf, int z, int s)
	{
		int l = s - 1;
		int ll = s - 2;
		int lll = s - 3;
		int r = s + 1;
		int rr = s + 2;
		int rrr = s + 3;
		int u = z + 1;
		int uu = z + 2;
		int uuu = z + 3;
		int o = z - 1;
		int oo = z - 2;
		int ooo = z - 3;

		System.out.println("Zeile = "+z+" Spalte = "+s);
		
		//horizontal
		if(s > 1 && s < 6 && sf[z][ll] == stein && sf[z][l] == stein && sf[z][r] == stein)
		{ gewonnen = true;}
		else if(s > 0 && s < 5 && sf[z][l] == stein && sf[z][r] == stein && sf[z][rr] == stein)
		{ boolean gewonnen = true;}
		else if(s > 2 && sf[z][lll] == stein && sf[z][ll] == stein && sf[z][l] == stein)
		{ boolean gewonnen = true;}	
		else if(s < 4 && sf[z][r] == stein && sf[z][rr] == stein && sf[z][rrr] == stein)
		{ boolean gewonnen = true;}
		
		//vertikal
		else if(z < 3 && sf[z][u] == stein && sf[z][uu] == stein && sf[z][uuu] == stein)
		{ boolean gewonnen = true;}
		
		//diagonal obenlinks - untenrechts
//Diese Zeile sollte die Zeile 50 sein:
		else if(s > 1 && s < 6 && z < 5 && sf[oo][ll] == stein && sf[o][l] == stein && sf[u][r] == stein)
		{ boolean gewonnen = true;}
		else if(s > 0 && s < 5 && z < 4 && sf[o][l] == stein && sf[u][r] == stein && sf[uu][rr] == stein)
		{ boolean gewonnen = true;}
		else if(s > 2 && sf[ooo][lll] == stein && sf[oo][ll] == stein && sf[o][l] == stein)
		{ boolean gewonnen = true;}
		else if(s < 4 && z < 3 && sf[uuu][rrr] == stein && sf[uu][rr] == stein && sf[u][r] == stein)
		{boolean gewonnen = true;}
		
		//diagonal obenrechts - untenlinks
		else if(s > 0 && s < 5 && z < 5 && sf[oo][rr] == stein && sf[o][r] == stein && sf[u][l] == stein)
		{boolean gewonnen = true;}
		else if(s > 1 && s < 6 && z < 4 && sf[o][r] == stein && sf[u][l] == stein && sf[uu][ll] == stein)
		{ boolean gewonnen = true;}
		else if(s < 4 && sf[ooo][rrr] == stein && sf[oo][rr] == stein && sf[o][r] == stein)
		{ boolean gewonnen = true;}
		else if(s > 2 && z < 3 && sf[u][l] == stein && sf[uu][ll] == stein && sf[uuu][lll] == stein)
		{ boolean gewonnen = true;}
		else { boolean gewonnen = false;}
		return gewonnen;
	}

 

[mikachu]

:shock: ???:L

viel spass beim reindenken :bahnhof:

auf jeden fall greifst du irgendwo auf ein element zu, was entweder noch nicht initialisiert wurde, oder was es gar nicht gibt...

angenommen du kriegst z = 1 übergeben, und oo = z - 2 ergibt dann eine -1 !
das ist für einen index ein unzulässiger integer-wert.

laut deiner fehlerausschrift, ist der parameter z auf 0 gesetzt, darum auch ArrayIndexOutOfBoundsException -2

 

[thE_29]

Die Exception sagt aber das der übergeben Index (also -2) nicht erreichbar ist!

Nur wie kommt er auf -2?

 

[guilhelm]

ja also das habe ich ja beschränkt!

else if(s > 1 && s < 6 && z < 5 && sf[oo][ll] == stein && sf[o][l] == stein && sf[u][r] == stein)
      { boolean gewonnen = true;}

er sollte eigentlich nicht weiter als die Spalte 0 und Spalte 6, wie die Zeile 5! ich verstehe gar nicht wie er -2 kriegt!

 

[mikachu]

Nur wie kommt er auf -2?

...wenn z den wert 0 hat... hat oo den wert -2 :!:
desweiteren hat o den wert -1 :!:

 

[guilhelm]

jetzt dasselbe aber in einer anderen Zeile:

Zeile = 2, Spalte = 3 und meinArray[6][7]

else if(s > 1 && s < 6 && z < 5 && z > 1 && sf[oo][ll] == stein && sf[o][l] == stein && sf[u][r] == stein)
		{
			gewonnen = true;
			return gewonnen;
		}

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: -1
at Logik.gewonnen(Logik.java:56)

:x WARUM?! :x

 

[mikachu]

gleiches prob...

du versuchst irgendwo mit 'ooo' zuzugreifen => ooo = -1 :!:

 

[guilhelm]

ja aber ich habe es beschränkt mit

else if(s > 1 && s < 6 && z < 5)

und überall anders genau so, auch von oben habe ich eine Grenze gesetzt

das programm sollte dann, gar nicht prüfen ob irgendein Wert "out of bounds" ist oder nicht, wenn die Zeile und Spalten mit der Einschränkung nicht stimmen, oder?

hab' immer gelernt dass A && B = false ist, wenn A = false ist ohne zu prüfen ob B false oder true ist, im Gegensatz zu "&", das heisst wenn s schone kleiner gleich 1 wäre, würde das Programm was noch kommt überhaupt nicht prüfen...

 

[mikachu]

iss richtig... aber

else if(s > 2 && sf[ooo][lll] == stein && sf[oo][ll] == stein && sf[o][l] == stein)
      { boolean gewonnen = true;}

z = 2; (wie oben)
ooo = z - 3 = 2 - 3 = -1; :!:

 

[guilhelm]

danke für eure zeit, ich habe inzwischen die loesung gefunden: gab ein problem bei der vertikal loesung, das programm hat es einfach falsch verstanden, und eine andere zeile gezeigt...

 

[bygones]

nebenbei einen Tipp für die Helfer, als auch für dich:

boolean gewonnen(char stein, char[][] sf, int z, int s) 
   { 
      int l = s - 1; 
      int ll = s - 2; 
      int lll = s - 3; 
      int r = s + 1; 
      int rr = s + 2; 
      int rrr = s + 3; 
      int u = z + 1; 
      int uu = z + 2; 
      int uuu = z + 3; 
      int o = z - 1; 
      int oo = z - 2; 
      int ooo = z - 3;

als allgemeiner Tipp: Nutze sprechende Variablen... in dem ganzen wust ist dir irgendwann nicht mehr klar, was sf[ll] ist bzw sf[uuu]... Bad Practice !! und was ist s und z ?!

ich halte auch ausarten ala l ll lll für unsinnig... das eine ist s-1, das andere s-2 und das dritte s-3, warum ihnen noch so unwirre namen geben.

keine Ahnung sagen wir s soll die x Koordinate sein. Dann nenn s => xCoordinate und in den arrays dann einfach -1 -2 bzw -3

Es lebe die Übersicht ;-)

 

[mikachu]

this.finalize();