IndexoutofBoundException

[DaSt]

Hallo,

bekomme an folgender Stelle eine I.o.o.B.Exception. Warum?

public class Matrix {
    int n=0; //zeilen
    int m=0; //spalten
   
    public Matrix(int n, int m){
        this.n =n;
        this.m =m;
    }
   
    int arr[][] = new int[n][m];
     
    //Matrix mit 0 initialisieren
    public void Init(){
       
        for(int i=0; i<n; i++){
            for(int y=0; y<m; y++){
                this.arr[i][y]=0; //Hier kommt die Exception
            }
        }
       
    }

Aufruf in der Main:

...
Matrix ma = new Matrix(3,3);
ma.Init();

//Es soll also eine 3*3 Matrix erstellt werden und alle Werte mit 0 initialisiert

Danke

 

[Robat]

int arr[][] = new int[n][m];

An der Stelle ist dein n und m für den Kompiler noch 0. Daher hast du auch nur ein Array von der Dimension 0 x 0.

Initialisiere dein Array im Konstruktor dann sollte es klappen.

Gruß Robert

 

[Soloeco]

Dem kann ich so nur zustimmen. Das Programm läuft schließlich von oben nach unten ab und somit wird das zwei dimensionale Array erst nach Festlegung der Matrix Größe initialisiert.

 

[DaSt]

ok danke, ist mir auch gerade aufgefallen. Da ich ein Jahr nicht mehr mit Java gearbeitet habe, bin ich eingerostet. Wie weise ich dem Array im Kunstruktor die größe zu? (Also von der Syntax her, will gerade nicht klappen)
so ist es ja ausserhalb nicht mehr sichtbar

public Matrix(int n, int m){
        this.n =n;
        this.m =m;
        int arr[][]= new int[n][m];
       
    }

 

[Robat]

Genau so wie du es bei der globalen Variable auch gemacht hast

private int[][] arr;

public Test()
{
     arr = new int[2][2];
}

Gruß Robert

 

[DaSt]

ok, jetzt klappts, mein Fehler war:
arr[][]= new int[n][m]; anstelle arr= new int[n][m];



DAnke