Collections String[] als value in HashMap

[Ceromanc]

Hallo, ich hab da ein Problem. Ich versuche in einer for-Schleife Schritt für Schritt die values von den keys zu aktualisieren, leider werden auch die vorherige values überschrieben, so, dass am Ende alle values gleich sind. Hier ist die Code:

for (int i=0; i<3; i++) {
		DatenGelesen[0] = buff.readLine();
		DatenGelesen[1] = buff.readLine();
		Ergebnisse.put(Spieler[i], DatenGelesen);
		if (DatenGelesen[0] == null) break;
		if (DatenGelesen[1] == null) break;
		String ww[] = Ergebnisse.get("Spieler 1");
		System.out.println(Arrays.toString(ww));
		ww = Ergebnisse.get("Spieler 2");
		System.out.println(Arrays.toString(ww));
		ww = Ergebnisse.get("Spieler 3");
		System.out.println(Arrays.toString(ww));
	}

Raus kommt:
Erste Runde:

[Michael, 99]
[keine Daten, 9999]
[keine Daten, 9999]

Zweite Runde:

[Viktoria, 102]
[Viktoria, 102]
[keine Daten, 9999]

Dritte Runde:

[Marcel, 150]
[Marcel, 150]
[Marcel, 150]

Was mache ich falsch und was muss man ändern?

Auch wenn ich so was schreibe:

DatenGelesen[0] = buff.readLine();
DatenGelesen[1] = buff.readLine();
Ergebnisse.put(Spieler[i], DatenGelesen);
DatenGelesen[0] = buff.readLine();
DatenGelesen[1] = buff.readLine();

befinden sich im HashMap die dritte und vierte Zeilen aus meiner Datei :bahnhof:

 

[jgh]

mmmh kannst du mal alle datentypen zu den Variablen zeigen...btw. CodeConvention googeln!!!!

ansonsten befindest du dich in einer for-Schleife und benutzt nicht einmal den Zählindex der Forschleife...schon sehr merkwürdig.

 

[Ceromanc]

Also hier habe ich versucht, alles was dazugehört reinzustecken:

...
Map<String, String[]> Ergebnisse = new HashMap<String,String[]>();
String [] DatenGelesen = new String[2];
String [] Default = {"keine Daten","9999"};
String [] Spieler = {"Spieler 1","Spieler 2","Spieler 3"};
...
public void DateiLesen(){
		
	FileReader myFile = null;
	BufferedReader buff = null;

		for (int i=0; i<3; i++) {
			Ergebnisse.put(Spieler[i], Default);
		}
		try {
			myFile=new FileReader("scores.txt");
			buff = new BufferedReader(myFile);
			try {	
				for (int i=0; i<3; i++) {
					DatenGelesen[0] = buff.readLine();
					DatenGelesen[1] = buff.readLine();
					Ergebnisse.put(Spieler[i], DatenGelesen);
					if (DatenGelesen[0] == null) break;
					if (DatenGelesen[1] == null) break;
					String ww[] = Ergebnisse.get("Spieler 1");
					System.out.println(Arrays.toString(ww));
					ww = Ergebnisse.get("Spieler 2");
					System.out.println(Arrays.toString(ww));
					ww = Ergebnisse.get("Spieler 3");
					System.out.println(Arrays.toString(ww));
				}
					
			} catch (IOException e2) {
				e2.printStackTrace();
			}
				
			try {
				myFile.close();
			} catch (IOException e1) {
				// TODO Auto-generated catch block
				e1.printStackTrace();
			}
				
			}catch (FileNotFoundException e2){
				JOptionPane.showMessageDialog(Hauptfenster,"Es ist ein Problem beim Lesen der Datei aufgetretten!\n" +
						"Entweder existiert sie nicht oder der Zugriff ist verweigert\n"+"Moeglicherweise stehen fruehere Daten nicht mehr zur Ferfuegung",
						"Fehler", JOptionPane.PLAIN_MESSAGE);
		}
			
}

 

[njans]

Ohne weiter zu gucken:

String [] Default = {"keine Daten","9999"};

for (int i=0; i<3; i++) {
            Ergebnisse.put(Spieler[i], Default);
        }

Ganz schlechte Idee.
Sobald du etwas an dem String[] veränderst, veränderst du es an ALLEN! Das ist ein Objekt und du packt bei jedem Spiel dieses eine Objekt als Value in die Map.

Mach lieber:

for (int i=0; i<3; i++) {
            Ergebnisse.put(Spieler[i], new String[]{"keine Daten","9999"});
        }

Jedoch würde ich mir das dann noch mal überdenken, warum ist da denn überhaupt eine Zahl drin, wenn du eigentlich keine Daten hast, sollt das dann nicht lieber gleich null sein?
Außerdem solltest du, wie schon vorher erwähnt, einmal die Java Conventions einhalten. Variablen schreibt man klein mit camelCase (dasIstMeineVariable). Ebenso ist die Verwendung von Java Schlagwörtern (default) eher schlecht.

 

[Ceromanc]

Danke für das Antwort. Mit dem default bin ich einverstanden, das war nur zur Probe. Ich werde das alles morgen noch Mal ausprobieren, sonnst bricht mein Kopf bald , jetzt kommen bei mir überall null'en.

Das Programm ist meine Abschlussklausur in der Hochschule, das ist ein Schiebepuzzle, und ich hatte vorher keine Colloections drin, aber der Professor meinte, dass ich sie unbedingt benutzen muss, weil wir das hatten. Die einzige Möglichkeit wo ich sie benutzen kann, ist die Bearbeitung von Ergebnissen des Spiels. Meine Idee ist, dass ich drei keys mache und in jedem key wird Name und Anzahl von Gängen gespeichert. Zu dem Fall, dass die Datei statt drei nur zwei oder ein Ergebnis hat, habe ich mein HashMap vorher mit "keine Daten","9999" ausgefüllt, so habe ich immer drei Ergebnisse.

 

[Marco13]

Sowas wie
Ergebnisse.put(Spieler, DatenGelesen.clone);
würde wohl auch den gewünschten Effekt bringen, aber vermutlich ist der "Fehler", dass "DatenGelesen" ... so... existiert wie es das offenbar tut. Der Array wird ja NUR dort lokal gebraucht, dann kann man ihne auch da deklarieren (!) und neu erstellen (!)

 

[Ceromanc]

Vielen Dank für die Antworten!!! Ich habe es so gemacht, wie njans geschrieben hat. Jetzt ist es so wie es sein soll :toll:. Ich habe auch verstanden was ich falsch gemacht habe: ich habe nicht das Inhalt von DatenGelesen geschrieben, sondern den Array selbst und das ist dann logisch, wenn ich Array ändere, dann wird es überall geändert :lol:.

Danke noch Mal für die Antworten.