Wert in ArrayList ändert sich ohne wissentliches zutun

[Nikolas]

Ich baue gerade an meinem Minesweeper und habe ein 2Dimensionales Array dafür gebaut.

static List<List<MineField>> grid = new ArrayList<List<MineField>>();

MineField ist eine Klasse aus ein paar integern und booleans.

Dieses grid habe ich als Klasse gebaut und mir dabei etwa solche Funktionen gegeben:

	public MineField get(int x, int y) {
		
		// Wenn das zu holende Element nicht da ist:
		try {return grid.get(x).get(y);}
		catch (IndexOutOfBoundsException e ) {
			System.out.println("IndexOutOfBounds bei grid.get: x="+x+ "y="+y);
			return null;
		}
	}

Im eigentlich Spiel initalisiere ich das array mit

	// Die Minen werden gelegt:
		for (int i=0; i<countX; i++) {
			for (int j=0;j<countY; j++) {
				
				grid.get(i,j).flagged = false;
				grid.get(i,j).opened=false;
				if (r.nextDouble() < mineProb) {
					grid.get(i,j).mined=true; // Bombe
					bombCount++;}
				else 
					grid.get(i,j).mined=false; // keine Bombe
			} //  for y
		} // for x

Diese Werte werden auch angenommen und korrekt gelesen, so weit kein Problem. Wenn ich jetzt aber auf eine Bombe klicke, führe ich diesen Code aus:

	// eine Mine wurde an der übergebenen Position aufgedeckt
	public static void handleDefeat(int x, int y) {
		
		System.out.println("defeat");
		
		f.setTitle("verloren");
		
		for (int i=0; i<countX; i++) {
			for (int j=0;j<countY; j++) {
				
			// Bei nicht verminten Feldern werden die Nachbarbomben angezeigt		
			if (grid.get(x,y).mined) {
				if (grid.get(x,y).flagged)
					Buttons[i][j].setText("(B)"); // markierte Bombe
				else
					Buttons[i][j].setText("B"); // nicht markierte Bombe
			} else { // Feld ist nicht vermint
				int cnt = grid.get(x,y).mineCount;
				
				if (grid.get(x,y).flagged) {
					Buttons[i][j].setText("F"); // Falsch markiert
					continue;
				}
				
				if (cnt==0)
					Buttons[i][j].setVisible(false);
				else
					Buttons[i][j].setText(""+cnt);
			}				
			} // schleife y
		} // schleife x
		
	// Die falsche Entscheidung wird angezeigt:	
	Buttons[x][y].setText("X");
		} // handledefeat

Das Problem liegt jetzt darin, dass sämtlich Felder mit einem B markiert sind. (ausser dem an Stelle x,y). An keiner Stelle ausser bei der initialisierung habe ich ein grid.get(x,y).mined=true stehen, so dass mich das verhalten etwas wundert.
Möglicherweise mache ich sonst etwas falsch, ich programmiere erst seit ein paar Wochen mit Java, davor hab ich mit Pascal gearbeitet und heute habe ich die Lists zum ersten Mal benutzt.

 

[SlaterB]

tja, nix offensichtliches in deinem Code, aber du beschreibst auch nicht mal ein konkretes Problem,

allgemein bleibt zu sagen:
mache deinen Code 'kontrollierter' und baue mehr Ausgaben ein,
dann kannst du jedes einzelne bit verfolgen und nichts kann dich mehr überraschen,

du hast Problem mit mined?
mache mined private und verwende

public boolean getMined() {
  System.out.println("ich bin Objekt "+hashCode()+", ich returne minded = "+this.minded);
  return this.minded;
}

public void setMined(boolean mined); {
  System.out.println("ich bin Objekt "+hashCode()+", ich war minded = "+this.minded+", bin nun mined: "+minded);
  this.mined = mined;
}

so kannst du für jedes Objekt von Anfang an bis zum Ende aller Tage den Zustand von mined loggen,
falls die Ausgaben zu viele werden: fange eh erstmal mit einem 1x1, 1x2 oder 2x2-Feld an..

 

[Guest]

if (grid.get(x,y).flagged) 
               Buttons[i][j].setText("(B)"); // markierte Bombe 
            else 
               Buttons[i][j].setText("B"); // nicht markierte Bombe

Ist das richtig, dass beides Mal "B" gesetzt wird?

Fred

 

[Fatal Error]

Ich baue gerade an meinem Minesweeper und habe ein 2Dimensionales Array dafür gebaut.

2 Dimensionales Array? Das was du da hast ist ne Liste^^ kein Array....

 

[Nikolas]

@Slater: Ich beschreibe doch recht genau mein Problem... Im obigen Code ist zu sehen, wie .mined gesetzt ist und unten sage ich, dass diese Variable einen anderen Wert hat. Das mit den Funktionen werde ich mal ausprobieren.

@Gast: Wer lesen kann, ist klar im Vorteil.

@Fatal Error: Oh tschuldigung, den Ausdruck bin ich noch von Pascal gewohnt. Vielleicht spürt Java diesen Ausdrucksfehler und macht deswegen komische Sachen

 

[André Uhres]

In "handleDefeat" vergleichst du mit

if (grid.get(x,y).mined) {...}

immer dasselbe Feld (x,y sind ja die Parameter der Methode).
Wenn dieses Feld vermint ist, wird natürlich überall B gesetzt!

 

[Nikolas]

oh. Da hast du recht. Ich hatte in einer ersten Version statt der Liste ein statisches Array und die aufrufe habe ich dann per find&replace ersetzt. Anscheinend habe ich da mit dem falschen ersetzt und keine Fehlermeldung bekommen, weil auch die anderen parameter vorkommen.

Vielen Dank.