Wieso funktioniert dieses Array nicht?

[RememberdiAlamo]

Hi! Versuch grad eine 5x4 Matrix zu erstellen in der die Zahlenfolge von 1 bis 20 angezeigt werden soll. Hab das schonmal codiert, aber es kommen nur lauter Nullfolgen raus, weiß nicht wieso. Könnte mal jemand rübergucken was da schief gelaufen ist?

class Aufgabe93{
	public static void main (String[] args){
	int [][] feld;
	feld = new int [5][4];
	int zahl = 1;
	
	
	for (int i = 0; i< feld.length; i++){
		for (int j = 0; j < feld[i].length; j++)
		System.out.print(feld[i][j] +"\t");
		System.out.println();
	}
	
	
	
	}
}

 

[Chloroplast]

du greifst niemals wieder auf die einzelnen Elemente zu, deshalb sind sie noch alle 0. die Variable zahl wird nicht genutzt.

 

[TheSorm]

weil du dir das nur ausgeben läst du hast da ja noch nichts reingeschrieben.

 

[RememberdiAlamo]

jaaaau stimmt!:bloed:
also dann noch

feld [i][j] = zahl++;

oder?

 

[Chloroplast]

würde gehen. ich würde allerdings ++zahl nehmen, es kommt zwar denke ich hier nicht auf performance an, aber bei zahl++ würdest du dir in ner hilfsvariable den wert zwischenmerken. müsstest halt bei 0 anfangen für gleiche ergebnis
[EDIT]und vergiss die klammern nicht 🙂[/EDIT]

 

[RememberdiAlamo]

Jetzt kommt aber error can not find symbol j

Damit komm ich leider überhaupt nicht klar, Sackgasse:noe:
Wenn mir niemand dies Aufgabe vorlöst, dann raff ichs wohl überhaupt nicht

class Aufgabe93{
	public static void main (String[] args){
	int [][] feld;
	feld = new int [5][4];
	int zahl = 1;
	
	
	
	for (int i = 0; i < feld.length; i++){
		for (int j = 0; j < feld.length; j++)
			
			feld [i][j] = zahl++;
			
		System.out.print(feld[i][j] +"\t");
		System.out.println();
	}
	
	}
}

 

[Chloroplast]

i und j sind dort nichtmehr sichtbar, nur innerhalb der schleife. ich verrate dir jetzt mal nicht das du die klammern vergessen hast :bae:

 

[TheSorm]

class Aufgabe93{
	public static void main (String[] args){
	int [][] feld;
	feld = new int [5][4];
	int zahl = 1;
	
	
	
	   for (int i = 0; i < feld.length; i++){
		for (int j = 0; j < feld.length; j++){
			
			feld [i][j] = zahl++;
			
		System.out.print(feld[i][j] +"\t");
		System.out.println();
                }
	   }
	
	}
}

 

[RememberdiAlamo]

danke TheSorm, kann das zwar jetzt compilieren aber bekomme eine out of bounds exception... Ich bin mir sicher dass ist ein kleiner Fehler drin, aber ich finde den einafch nicht. Da fehlt was:rtfm:


EDIT: Ok, der Fehler scheint darin zu liegen weil es nicht auf das aaray zugreift. habs jetzt so:

    class Aufgabe93{
        public static void main (String[] args){
        int [][] feld;
        feld = new int [5][4];
        int zahl = 1;
       
       
       
           for (int i = 0; i < feld.length; i++){
            for (int j = 0; j < feld[i].length; j++){
               
                feld [i][j] = zahl++;
               
            System.out.print(feld[i][j] +"\t");
            System.out.println();
                    }
           }
       
        }
    }

nur zeigt es jetzt nicht die MAtrix 5x4 an, sondern eine Liste ???:L

 

[Phoenixz]

Weil du das System.Out.println falsch gesetzt hast:

 for (int i = 0; i < feld.length; i++){
            for (int j = 0; j < feld[i].length; j++){
                feld [i][j] = zahl++;
                System.out.print(feld[i][j] +"\t");
              }
           System.out.println();
           }

 

[RememberdiAlamo]

Super! :toll:
Hast mir den Schlaf gerettet Phoenixz :applaus:
Danke und danke euch allen! Sind doch immer die kleinen Dinge auf die es ankommt.
Sollte mal im Script eine Section geben wie "Top 10 Dinge bei denen man sich immer vertut"...

 

[Chloroplast]

Ich persönlich finde ein top-10 liste eindeutig zu kurz, und das allein bei mir schon 😀

 

[RememberdiAlamo]

Ich habe da noch nicht so viel Erfahrung damit, daher... aber wenn der werter photsynthese betreibender Plastid das so sagt, dann werd ich wohl noch einige Probleme damit haben :autsch:
😀

 

[Chloroplast]

ist nur meine persönliche meinung :bae: