Boolean Array in

[adriano-celenta]

Hallo Freunde,

ich arbeite momentan an dem Projekt "Schiffe versenken"

Ich bin an der Stelle wo ich momentan auf meinem Feld die Schiffe setze:
D.h. Ich setze momentan nur Felder. Die verschiedenen Schiff-Typen habe ich noch nicht definiert.

Ich möcht die Felder die ich auf meinem Spiel-Feld setze in eine eigene Klasse schreiben. Dort ist dein ein Boolean Array, dass dann die belegten Felder auf true mitschreibt.

Im kursiven (********) Code kommt dann folgende Fehlermeldung "Syntax error on token "(", Expression expected after this token" :autsch:

for (int x = 0; x <= 99; x++) {

			final JButton btnNewButton_x = new JButton("");

			btnNewButton_x.setActionCommand(Integer.toString(x)); // Button ID
																	// wird
// gespeichert

			panel.add(btnNewButton_x);
			btnNewButton_x.addActionListener(new ActionListener() {

				public void actionPerformed(ActionEvent e) {
					// TODO Auto-generated method stub
					String cmd = e.getActionCommand(); // Wievielter Button
					int buttonNo = Integer.parseInt(cmd);
					Component c = panel.getComponent(buttonNo);
					JButton b = (JButton)c;
				    // x,y Koordinaten im Schiffe Feld Modell
				      int y = buttonNo/ 10;
				      int x = buttonNo- (y*10);
				      c.setBackground(Color.red);
				      //System.out.println("ButtonNummmer " +  buttonNo + " Koordinaten " + y + x );
				     Schiffe_setzen field = new Schiffe_setzen();
				     *********[code]if (field.setField([x][y]))

{
// Schiff setzen
c.setBackground(Color.gray);
}
}

});
}
[/code]

Meine Schiff_setzen Klasse
Anmerkung: zum Testen habe ich schon paar Werte in das Array hinein geschrieben und habe versucht auch diese Werte abzufragen. Dies ist auch an diesem Fehler gescheitert.

public class Schiffe_setzen {
	boolean field[][] = new boolean [10][10];
	
	Schiffe_setzen(){
		field[0][0] = true;
		field[1][0] = true;
		field[2][0] = true;
		field[3][0] = true;
	}
	public boolean [][] getField() {
		return field;
	}
	public void setField(boolean [][] field) {
		this.field = field;
	}
	
}

Ich bedanke mich schon im Voraus für eure Antworten.

LG

Adriano_Celentan

P.S. Für eventuelle Fragen: Ich kann erst am Nachmittag antworten, da ich arbeiten muss. ;(

 

[Mujahiddin]

Hallo...
Die Art, wie du vorgehst, ist ziemlich seltsam.
Du erzeugst 100 mal einen JButton, in welchen die Zahlen von 0 bis 99 stehen. Ich denke, das ist so geplant.
Was jetzt aber merkwürdig ist, ist erstens, dass du den actionCommand setzt. Du wandelst einen Integer in einen String (und tust diesen in deinen Button rein), den du später eh wieder in einen Integer wandeln wirst.
Dies würde einfacher gehen mit

final int buttonNo = x;

, im ActionListener kannst du dann auf diese Kopie zugreifen und musst nichts umwandeln (und dein Code ist lesbarer...).

Component c = panel.getComponent(buttonNo);
JButton b = (JButton)c;

Dieser Code ist auch sehr unpraktisch, um nicht zu sagen schlecht.
Wenn du vor den Buttons irgendeine Komponente hinzufügst (aus welchem Grund auch immer), wirst du diesen Fehler stundenlang verzweifelt suchen, weil es keine Fehlermeldung geben sollte (na gut, bestenfalls wird eine ClassCastException geschmissen, wenn es sich um keinen Button handelt)...
Dennoch, wieso brauchst du überhaupt diese Methode? Du kannst doch im ActionListener einfach auf dein JButton aus der for-Schleife zugreifen...

Als nächstes: Jedes Mal, wenn ein Button geklickt wird, erzeugst du ein neues Objekt der Klasse "Schiffe_setzen". Das ist höchstwahrscheinlich nicht so geplant. Du solltest irgendwo eine Referenz (evtl. Klassenvariable oder einen final Parameter) haben, auf die du zugreifst - und auf die alle Buttons zugreifen.

Liebe Grüße

 

[hexx]

Was ist denn die Signatur der Methode

field.setField([x][y])

. Der Aufruf sieht zielmich falsch aus.

[]

ist der Array-Zugriffsoperator. Den kannst du nicht als Argument in einem Methodenaufruf benutzen.

 

[Mujahiddin]

Achso, ja...
Übrigens..
Mach mal am besten sowas:

public void setField(int x, int y, boolean b) {
    field[x][y] = b;
}
public boolean getField(int x, int y){
    return field[x][y];
}

und dann in deinem ActionListener:

Schiffe_setzen field = getSchiffeSetzen(); /* auf eine bereits erstellte Instanz zugreifen */
if(field.getField(x, y))
    button_new.setBackground(Color.GRAY);