Array zum Speichern von Würfen

[Lacritz]

Heyho
bin derzeitig an einem Programm zum Speichern von Würfen. man würfelt mit 2 würfeln gleichzeitig , wobei wurf 1 = zeilen , wurf 2 = spalte bedeutet , also wenn ich (2;1) habe bedeutet es das das feld in zeile 2 , spalte 1 um eins erhöht wird.

Das array soll 6x6 groß sein , (klar da es hier um herkömmliche würfel mit 6 Augen geht)

habe folgendes Programm bisher geschrieben, finde aber leider den Fehler bisher nicht!
Bitte um Hilfe

import java.util.Arrays;

import javax.swing.JOptionPane;

public class zaehlenvondoppelwuerfen {

	public static int rollDice() {
		return (int) Math.floor(Math.random() * 6 + 1);
	}

	static int[][] arrayFeld = new int[6][6];
	static int wuerfe;
	static String testString;

	static int[][] fillArray(int wuerfe) {

		int counter = 0;
		int wurfeins;
		int wurfzwei;
		while (counter <= wuerfe) {
			wurfeins = rollDice();
			wurfzwei = rollDice();

			arrayFeld[wurfeins][wurfzwei] = +1;
			counter++;
		}
		return arrayFeld;
	}

	public static void main(String[] args) {
		wuerfe = Integer.parseInt(JOptionPane
				.showInputDialog("Anzahl ihrer Wuerfe="));
		arrayFeld = new int[6][6];
		arrayFeld = fillArray(wuerfe);
		System.out.print(Arrays.toString(arrayFeld));
	}
}

MfG Lacritz

 

[diggaa1984]

Wäre gut gewesen, den Fehler auch zu beschreiben, oder eine Ausgabe anzuhängen, aber ok:

Folgendes:

while (counter <= wuerfe) {
    wurfeins = rollDice();
    wurfzwei = rollDice();
    arrayFeld[wurfeins][wurfzwei] = +1;
    counter++;        
}

• counter <= wuerfe
  
  Hier würfelst du immer einmal mehr als angegeben, da counter mit 0 startet.
  counter = 0 und wuerfe = 1, würde diese Schleife 2x ausführen
  ​
• arrayFeld[wurfeins][wurfzwei] = +1;
  
  Hier schreibst du immer nur eine 1 an die entsprechende Stelle. Richtig wäre zB.

  += 1;

  , statt
  

  = +1;

  Weiterhin arbeitest du eventuell mit falschen Indizes, das habe ich jetzt nicht genauer überprüft. Aber angenommen, das du eine 6 Würfelst, dann greifst du eventuell auf den 6. Index zu, welcher aber nicht existiert, da dein Feld nur 6 Einträge hat (0-5).
  ​

Welche Fehler konnten damit gelöst werden?

 

[rme]

(entfernt, diggaa1984 war schneller und hat mehr gesagt)

 

[Lacritz]

Danke für die Schnelle Hilfe!

import java.util.Arrays;

import javax.swing.JOptionPane;

public class zaehlenvondoppelwuerfen {

	public static int rollDice() {
		return (int) Math.floor(Math.random() * 6 + 1);
	}

	static int[][] arrayFeld = new int[6][6];
	static int wuerfe;

	static int[][] fillArray(int wuerfe) {

		int counter = 1;
		int wurfeins;
		int wurfzwei;
		while (counter <= wuerfe) {
			wurfeins = rollDice()-1;
			wurfzwei = rollDice()-1;

			arrayFeld[wurfeins][wurfzwei] ++;
			counter++;
		}
		return arrayFeld;
	}

	public static void main(String[] args) {
		wuerfe = Integer.parseInt(JOptionPane
				.showInputDialog("Anzahl ihrer Wuerfe="));
		arrayFeld = new int[6][6];
		arrayFeld = fillArray(wuerfe);
		System.out.print(Arrays.toString(arrayFeld));
	}
}

Hab den Code mal umgeändert , als ausgabe (Bei anzahl der Wuerfe=1000) kommt folgendes raus [[I@2f1050d, [I@1d8299fd, [I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4] - was für mich keinen Sinn macht.

MfG Lacritz

 

[diggaa1984]

Macht sinn, weil du das Array so nicht ausgeben kannst.
Du gibst quasi für alle Zeilen die Spalten-Arrays aus und das ist per default nur der Hash des Arrays.

Du musst schrittweise mit einer Schleife die Elemente ausgeben, anders ist das bei Listen, die überschreiben die toString entsprechend.

[EDIT]ah ok, das mit den wuerfen war korrigiert, nicht gesehen [/EDIT]

 

[Ikaron]

Spoiler

Der Fehler ist nur ein Kleiner. Du schreibst:
[c]arrayFeld[wurfeins][wurfzwei] = +1;[/c]
Das Plus gehört noch zu der eins dazu, ist also das gleiche wie:
[c]arrayFeld[wurfeins][wurfzwei] = 1;[/c]
=> Du setzt nach einem Wurf den Wert auf eins, anstatt ihn um eins zu erhöhen. Dies ginge mit +=
[c]arrayFeld[wurfeins][wurfzwei] += 1;[/c]

Ach ja, und was die Ausgaben betrifft, hat mein Vorninja recht.

Es wurde schon alles gesagt. Ich sollte die Seite vor dem Absenden noch mal neu laden.

 

[Lacritz]

für wuerfe=1 kommt "[[I@2f1050d, [I@1d8299fd, [I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4]"

Wie wäre würde ich denn eine korrekte Ausgabe schreiben?
Gibt es eigentlich eine Homepage wie Self.HTML auch für java?

MfG Lacritz

 

[diggaa1984]

for (int i=0; i<arrayFeld.length; i++) {
    for (int j=0; j< arrayFeld[i].length; j++) {
        System.....
    }
}

So ginge das.
Zeilenumbruch darfst selber noch einfügen

 

[rme]

Gibt es eigentlich eine Homepage wie Self.HTML auch für java?

Ja, die gibt es: Java SE Technical Documentation und insbesondere Java Platform Standard Edition 7 Documentation

 

[Lacritz]

	public static void main(String[] args) {
		wuerfe = Integer.parseInt(JOptionPane
				.showInputDialog("Anzahl ihrer Wuerfe="));
		arrayFeld = new int[6][6];
		arrayFeld = fillArray(wuerfe);
		for (int i = 0; i < arrayFeld.length; i++) {
			for (int j = 0; j < arrayFeld[i].length; j++)

				System.out.println(Arrays.toString(arrayFeld));
		}
	}
}

so wäre es dann , Ausgabe ist jedoch diesmal :[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]
[[I@694bc02d, [I@6c36005c, [I@4d51cf84, [I@360f0ac4, [I@158be596, [I@138de5e7]

Ich weiß nichtmal woher die werten kommen , noch woher soviele Spalten / Zeilen kommen, sollten doch eigentlich nur 6x6 sein?

 

[rme]

Du gibst 36 mal das ganze arrayFeld aus, statt die jeweis interessante Zelle

Edit: Etwas bearbeitet, um nicht zu viel zu verraten

 

[diggaa1984]

Ich wollte dir nicht alle verraten.
Die innerste Schleife läuft bei einem 6*6 Array insgesamt 36 mal!
Was du ausgibst ist jeweils das was du vorher schon ausgegeben hattest.

Was du brauchst ist:
Ausgabe der AKTUELLEN ZELLE in der innersten Schleife ohne Zeilenumbruch, eventuell ein Komma oder so rein, als Trennzeichen.
Und Einfügen eines Zeilenumbruchs, an passender Stelle.

[EDIT]Ach mist, jetzt wurde es verraten [/EDIT]

 

[Lacritz]

Ich komm leider garnicht weiter :shock: ;(

 

[diggaa1984]

wie kannst du eine einzige Zelle des Arrays ausgeben??
Genau das muss dann in die innerste Schleife (da wo du momentan die Ausgabe machst)

 

[Lacritz]

Ist mein zweites Programm mit Arrays , leider heißt es bei uns an der Uni meist Zuerst wird Programmiert , danach bekommt man das Wissen vermitelt.

	for (int i = 0; i < arrayFeld.length; i++) {
			for (int j = 0; j < arrayFeld[i].length; j++)

				System.out.println(Arrays.toString(arrayFeld[j]));

Ausgabe macht mehr sinn , ist jedoch zu lang für 1000:

[30, 27, 29, 33, 30, 33]
[23, 19, 31, 32, 21, 35]
[26, 22, 29, 22, 28, 19]
[36, 31, 36, 15, 21, 36]
[22, 29, 30, 31, 21, 28]
[22, 27, 31, 33, 25, 37]
[30, 27, 29, 33, 30, 33]
[23, 19, 31, 32, 21, 35]
[26, 22, 29, 22, 28, 19]
[36, 31, 36, 15, 21, 36]
[22, 29, 30, 31, 21, 28]
[22, 27, 31, 33, 25, 37]
[30, 27, 29, 33, 30, 33]
[23, 19, 31, 32, 21, 35]
[26, 22, 29, 22, 28, 19]
[36, 31, 36, 15, 21, 36]
[22, 29, 30, 31, 21, 28]
[22, 27, 31, 33, 25, 37]
[30, 27, 29, 33, 30, 33]
[23, 19, 31, 32, 21, 35]
[26, 22, 29, 22, 28, 19]
[36, 31, 36, 15, 21, 36]
[22, 29, 30, 31, 21, 28]
[22, 27, 31, 33, 25, 37]
[30, 27, 29, 33, 30, 33]
[23, 19, 31, 32, 21, 35]
[26, 22, 29, 22, 28, 19]
[36, 31, 36, 15, 21, 36]
[22, 29, 30, 31, 21, 28]
[22, 27, 31, 33, 25, 37]
[30, 27, 29, 33, 30, 33]
[23, 19, 31, 32, 21, 35]
[26, 22, 29, 22, 28, 19]
[36, 31, 36, 15, 21, 36]
[22, 29, 30, 31, 21, 28]
[22, 27, 31, 33, 25, 37]

 

[diggaa1984]

Achso, in dem Fall nimm die J-Schleife raus und schreibe die Ausgabe in die I-Schleife (und ersetze arrayFeld[j] mit arrayFeld)

Und, lerne die Schleifen-Konstrukte zu deuten!
Dann ergibt auch die lange Ausgabe für dich Sinn.

 

[Lacritz]

Es Funktioniert ! Dank dir... habe leider was Java angeht noch einen weiten weg vor mir...

 

[diggaa1984]

So haben wir doch alle angefangen, nur nicht entmutigen lassen.