DefaultCellRendere - Zellehintergrundsfarbe anpassen

[adriano-celenta]

Hallo an alle,

Ich bin neu hier in diesem Forum, möchte aber gleich ohne groß rumschwafeln zu meinem Problem kommen.

Mein Problem:
Ich kann die Zellhintergrundfarbe einer selektierten Zelle in meiner JTable nicht durch einen Mausklick anpassen.

Meine Situation:
Ich versuche unter Java das Programm "Schiffe versenken" zu programmieren - Ist die Abschlussnote für meinen Programmierkurs.
Für das Feld, auf dem man die einzelnen Schiffe positionieren kann, habe ich mich für eine JTable entschieden.
Nach einer ausgedehnten Researche bin ich dann auf folgenden Link gekommen in eurem Forum:
http://www.java-forum.org/bilder-gui-damit-zusammenhaengt/7032-jtable-teil-4-darstellung-daten.html
Gibt einen guten Einblick in das Thema. Ich habe versucht mich an die Anleitung zuhalten, jedoch funkt es bei mir noch immer nicht. ???:L

Zur besseren Übersicht mein Code der JTable:

table = new JTable(rowData, columnName);
		table.setModel(new DefaultTableModel(
			new Object[][] {
				{null, null, null, null, null, "", null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
			},
			new String[] {
				"A", "B", "C", "D", "E", "F", "G", "H", "I", "J"
			}
		) {
			boolean[] columnEditables = new boolean[] {
				false, false, false, false, false, false, false, false, false, false
			};
			public boolean isCellEditable(int row, int column) {
				return columnEditables[column];
			}
		});
		table.getColumnModel().getColumn(0).setResizable(false);
		table.getColumnModel().getColumn(0).setPreferredWidth(50);
		table.getColumnModel().getColumn(1).setResizable(false);
		table.getColumnModel().getColumn(1).setPreferredWidth(50);
		table.getColumnModel().getColumn(2).setResizable(false);
		table.getColumnModel().getColumn(2).setPreferredWidth(50);
		table.getColumnModel().getColumn(3).setResizable(false);
		table.getColumnModel().getColumn(3).setPreferredWidth(50);
		table.getColumnModel().getColumn(4).setResizable(false);
		table.getColumnModel().getColumn(4).setPreferredWidth(50);
		table.getColumnModel().getColumn(5).setResizable(false);
		table.getColumnModel().getColumn(5).setPreferredWidth(50);
		table.getColumnModel().getColumn(6).setResizable(false);
		table.getColumnModel().getColumn(6).setPreferredWidth(50);
		table.getColumnModel().getColumn(7).setResizable(false);
		table.getColumnModel().getColumn(7).setPreferredWidth(50);
		table.getColumnModel().getColumn(8).setResizable(false);
		table.getColumnModel().getColumn(8).setPreferredWidth(50);
		table.getColumnModel().getColumn(9).setResizable(false);
		table.getColumnModel().getColumn(9).setPreferredWidth(50);
		table.setRowHeight(20);
		table.setBackground(SystemColor.desktop);
		table.setSelectionMode(ListSelectionModel.SINGLE_SELECTION ); 
		table.setColumnSelectionAllowed(false);
		table.setRowSelectionAllowed(false);
		table.setCellSelectionEnabled(true);
		table.setBounds(68, 125, 223, 200);
		
		table.addMouseListener(new MouseAdapter() {
			public void mouseClicked(MouseEvent arg0) {
				int x = table.getSelectedColumn();				
				int y = table.getSelectedRow();		
				
				[code] table.setDefaultRenderer( Color.class, new MyTableCellRenderer() );

System.out.println(x +" "+ y);
}
});
[/code]

Mein Code des DefaultCellRenderer

public class MyTableCellRenderer extends DefaultTableCellRenderer {

	@Override
	public Component getTableCellRendererComponent(JTable table, Object value,
			boolean isSelected, boolean hasFocus, int row, int column) 
	{
		super.getTableCellRendererComponent(table, value, isSelected, hasFocus,
				row, column);

		Color color = (Color) value;

		setText(color.getRed() + ", " + color.getBlue() + ", "
				+ color.getGreen());
		setForeground(color);
		setBackground(Color.green);

		return this;
	}

}

Ich weiß, dass das Thema schon einmal gepostet wurde. Ich würde aber euch bitten ein kurzen Blick darauf zu werfen, denn ich finde meinen Fehler nicht, den ich gemacht habe.
Gemäße dem Sprichwort:"Man sieht den Wald, vor lauter Bäume nicht" :autsch:

Vielen Dank für eure Hilfe.

LG

Adriano-Celentano

 

[adriano-celenta]

Sorry Leute! Irgendwie würde der Beitrag 2mal abgespeichert. Ich schliesse den 1. Beitrag. Danke!!

LG
Adriano-Celentano

 

[Michael...]

Da das ganze für eine Abschlussarbeit ist ein paar Hinweise:

table = new JTable(rowData, columnName);
		table.setModel(new DefaultTableModel(...

Die Übergabe von rowData und columnName in der obersten Zeile ist überflüssig, wenn Du eine Zeile später sowieso ein neues TableModel setzt. (wenn nicht sogar gefährlich, wenn man im späteren Ablauf versucht über diese Arrays das Datenmodel zu manipulieren)

		table.setModel(new DefaultTableModel(
			new Object[][] {
				{null, null, null, null, null, "", null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
			},
			new String[] {
				"A", "B", "C", "D", "E", "F", "G", "H", "I", "J"
			}

würde man diesen Konstruktor verwenden könnte man sich einige Zeilen Code sparen:

DefaultTableModel(Object[] columnNames, int rowCount)

		table.getColumnModel().getColumn(0).setResizable(false);
		table.getColumnModel().getColumn(0).setPreferredWidth(50);
		table.getColumnModel().getColumn(1).setResizable(false);
		table.getColumnModel().getColumn(1).setPreferredWidth(50);
		table.getColumnModel().getColumn(2).setResizable(false);
		table.getColumnModel().getColumn(2).setPreferredWidth(50);
		table.getColumnModel().getColumn(3).setResizable(false);
		table.getColumnModel().getColumn(3).setPreferredWidth(50);
		table.getColumnModel().getColumn(4).setResizable(false);
		table.getColumnModel().getColumn(4).setPreferredWidth(50);
		table.getColumnModel().getColumn(5).setResizable(false);
		table.getColumnModel().getColumn(5).setPreferredWidth(50);
		table.getColumnModel().getColumn(6).setResizable(false);
		table.getColumnModel().getColumn(6).setPreferredWidth(50);
		table.getColumnModel().getColumn(7).setResizable(false);
		table.getColumnModel().getColumn(7).setPreferredWidth(50);
		table.getColumnModel().getColumn(8).setResizable(false);
		table.getColumnModel().getColumn(8).setPreferredWidth(50);
		table.getColumnModel().getColumn(9).setResizable(false);
		table.getColumnModel().getColumn(9).setPreferredWidth(50);

Schleifen sind vermutlich auch schon mal durch genommen worden??

Zu Deinem Problem:

		table.addMouseListener(new MouseAdapter() {
			public void mouseClicked(MouseEvent arg0) {
				int x = table.getSelectedColumn();				
				int y = table.getSelectedRow();		
				
				[code] table.setDefaultRenderer( Color.class, new MyTableCellRenderer() );

System.out.println(x +" "+ y);
}
});[/code]

So funktioniert das nicht, den CellRenderer setzt man einmalig bei der Initialisierung der JTable. Beim Klicken auf der JTable ändert man der Inhalt (bzw. den des Models). Der CellRenderer kann dann anhand des Inhalts einer Zelle "entscheiden" wie der diese darstellt. ==> noch mal einen Blick ins Tutorial werfen.

 

[adriano-celenta]

Hallo Michael...

Danke für deine schnelle Antwort. Du hast mich schon ein Stück weiter gebracht.
Punkt 1. habe ich angepasst.
Punkt 2. noch nicht angepasst. Diesen Quellcode hat mir WindowsPro Builder automatisch generiert.
Punkt 3. habe ich angepasst.
Punkt 4. habe ich soweit angepasst wie es im Tutorial steht. Ich habe das übersehen, dass man nur einmalig den DefaultTableCellRenderer initialisieren muss.

Eine Frage habe ich da noch: Wie schaffe ich es, dass die selektierten Zellen, die Farbe danach bei behalten?

table = new JTable();
		table.setModel(new DefaultTableModel(
			new Object[][] {
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
				{null, null, null, null, null, null, null, null, null, null},
			},
			new String[] {
				"A", "B", "C", "D", "E", "F", "G", "H", "I", "J"
			}
		) {
			boolean[] columnEditables = new boolean[] {
				false, false, false, false, false, false, false, false, false, false
			};
			public boolean isCellEditable(int row, int column) {
				return columnEditables[column];
			}
		});
		for (int i = 0; i <= 9; i++) {
			table.getColumnModel().getColumn(i).setResizable(false);
			table.getColumnModel().getColumn(i).setPreferredWidth(50);
			
		}
		
		table.setRowHeight(20);
		table.setBackground(SystemColor.desktop);
		table.setSelectionMode(ListSelectionModel.SINGLE_SELECTION); 

		table.setColumnSelectionAllowed(true);
		table.setRowSelectionAllowed(true);
		table.setCellSelectionEnabled(true);
		table.setBounds(68, 125, 223, 200);

		table.setDefaultRenderer(Object.class, new MyTableCellRenderer2());
[/Java]

Der DefaultTableCellRenderer
[code=Java]
public class MyTableCellRenderer2 extends DefaultTableCellRenderer {
	private Color colorSelected = Color.green;
	private Color colorFocus = Color.red;
	private Color colorNormal = Color.blue;

	public MyTableCellRenderer2(){
        setOpaque( true );
    }
	
	public Component getTableCellRendererComponent(JTable table, Object value,
			boolean isSelected, boolean hasFocus, int row, int column) {
		System.out.println("-----------------------------------------------");
		setForeground(Color.BLACK);

		if (hasFocus) {
			setBackground(colorFocus);
		} else if (isSelected) {
			setBackground(colorSelected);
		} else {
			setBackground(colorNormal);
		}

		setValue(value);
		return this;
	}
}

Und danke nochmals für deine Unterstützung.

LG

Adriano-Celentano

 

[bERt0r]

Für ein Schiffeversenken Spiel würdest du dir mit einem Panel auf dem ein GridLayout liegt glaube ich leichter tun.