setCellRenderer für EINZELNE Zelle

[seoman]

Hallo Leute,
ich weiß nicht ob das Titel absolut passend getroffen ist, aber ihr werdet sehen.
Ich habe vor, Schiffe versenken mit einer jTable zu realisieren.
Hierzu brauche ich aber eine Art "Identifikationswert" für jede einzelne Zelle,
um zu wissen wie ich sie beispielsweise bei einem Schuss einfärben soll.
Dafür dachte ich mir am flexibilsten sei ein eigener CellRenderer für jeden Zelltyp
(Schiff, Schuss, Wasser), den ich jederzeit mit getCellRenderer auslesen kann.
Die CellRenderer zu erstellen und so ist ja alles kein Problem,
aber ich traf im Internet(Foren, etc) immer nur auf das Zuweisen
eines CellRenderers an eine Spalte.
Folglich meine Fragen:
Wie funktioniert die Zuweisung (setCellRenderer) auf eine einzelne Zelle?
Oder gibt es einen besseren/leichteren/flexibleren Lösungsweg
(z. B. anderer"Identifikationswert")?

Vielen, vielen Dank im voraus
Seoman

 

[Beni]

An eine einzelne Zelle? Gar nicht.

Benutz das TableModel. Da schreibst du rein, für welche Zelle was gilt (ist da ein Schiff, Wasser, ...), und ein einzelner Renderer kann dann ein solcher Eintrag des Models interpretieren, und darstellen.

Siehe auch FAQ :wink:

 

[seoman]

@Beni: Sehr umfangreicht FAQ, besonders das Tuto zu jTable gefällt mir.
So ganz habe ich deine Antwort aber nicht umsetzen können,
meine Fragen dazu:

1. Wie schreibe ich denn einen Wert, den der Benutzer nicht sehen soll ?
(setValueAt wird das wohl nicht können !?)
2. Wieso weiß mein DefaultCellRenderer nichts von der bisherigen Hintergrundfarbe
der Zelle, die im "aktuellen Durchlauf" gefärbt wird? (wenn ich setBackground(getBackground())
oder gar nichts färbe, nimmt er die Farbe der zuletzt gefärbten Zelle...)?

@rest:
Um das ganze ein bisschen zu konkretisieren:

	/**
	 * @param tableContainingCells JTable
	 * @param x int
	 * @param y int
	 * @param size int
	 * @param dir int (Richtung des Schiffes)
	 * @param action int (0=Vorschau,1=Schiff setzen,2=Schuss)
	 */

private void actionPerformed(final JTable tableContainingCells, final int x, final int y, final int size, final int dir, final int action)
	{
		// CellRenderer setzen
		tableContainingCells.setDefaultRenderer(Object.class, new DefaultTableCellRenderer()
		{
			public Component getTableCellRendererComponent(JTable table, Object value,
				boolean isSelected, boolean hasFocus, int row, int column)
			{
				super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column); 
				// Pruefen, in welche Richtung gefärbt gesetzt werden soll
				if (dir == 0)
				{
					// Koordinaten überprüfen
					if (row == y && column >= x && column < (x + size))
					{
						setBackground(getColor(action,getBackground()));
					}
				} else 
				{
					// Koordinaten überprüfen
					if (column == x && row >= y && row < (y + size))
					{
						setBackground(getColor(action,getBackground()));
					}
				}
				// Zelle zurückgeben
				return this;
			}
		});
		// jTable neu zeichnen
		tableContainingCells.repaint();
	}
	
	private Color getColor(int action, Color defaultColor) 
	{
		// Pruefen, welche Aktion ausgeführt wird
		switch (action)
		{
			// Aktion: Vorschau
			case 0:
				// Hintergrundfarbe zurückgeben
				return Color.BLACK;

			// Aktion: Schiff setzen
			case 1:
				// Hintergrundfarbe zurückgeben
				return Color.GREEN;

			// Aktion: Schuss
			case 2:
				// Pruefen, ob an Koordinate ein Treffer ist
				  // ???
				// Hintergrundfarbe zurückgeben
				return Color.WHITE;
			default:
				// Hintergrundfarbe zurückgeben
				return defaultColor;
		}		
	}

Bisher habe ich noch nicht mit dem TableModel gearbeitet,
weil eben noch Frage 1 offen stand...

 

[Beni]

Der Renderer stellt den Inhalt einer Zelle des Models dar.

Also: Model hat viele Zellen, eine davon wird dem Renderer übergeben, und der stellt diese Zelle irgendwie dar.

Der wichtige Punkt: der Renderer ist total frei, wie er die Zelle darstellt.

Du könntest das so machen:
Das Model füllst du mit solche einer Struktur

public class Zelle{
  private boolean ship; // true falls hier ein Schiff ist
  private boolean visited; // true falls der Spieler schon hierhin geschossen hat
}

Und der Renderer wird dann etwa so aussehen:

public class Renderer extends DefaultTableCellRenderer{
  public Component getTableCellComponent(...){
    Zelle z = (Zelle)value;
    if( z.isVisited() ){
       if( z.isShip() )
          setBackground( Color.RED );
       else
           setBackground( Color.BLUE );
    }
    else{
       setBackground( Color.GRAY );
    }

    // mit value = "", dem leeren String
     return super.getTableCellRendererComponent( table, "", ... );
  }
}

 

[seoman]

Aber wenn ich als Value einen leeren String übergebe,
weiß ich ja später gar nicht mehr ob dort beispielsweise ein Schiff platziert ist..?

 

[Beni]

Ob dort ein Schiff ist steht im Model :wink: Du musst unterscheiden zwischen den Daten die du als Programmierer kennst, und den Daten die der Benutzer angezeigt bekommt.

 

[seoman]

Eigentlich alles klar soweit.
Mein einziges Problem ist nun,
das ich gar keinen int in mein
TableModel schreiben kann
(setValueOf erwartet Objekt..).
Bin ich tatsächlich zu diesem hässlichen
Ausweich-Manöver über einen String gezwungen
(das bei mir ohnehin nicht funktioniert)??

Setzen:
(value,x und y sind int's)

tableModelContainingCells.setValueAt(String.valueOf(value),x,y);

Auslesen:
(value,x und y sind int's)

int test = Integer.valueOf(mytableModel.getValueAt(row,column).toString()).intValue());

thx,
seoman

 

[Beni]

hi, nimm doch Integer (nicht zu verwechseln mit int) anstelle von String. Das spart das mühsame parsen (und gibt auch ein bisschen Geschwindigkeit).

 

[seoman]

Habe das jetzt alles realisiert.
Mein letztes Problem ist,
das auch meine Vorschau gemerkt wird.
Deshalb habe ich setze ich sie gleich nach dem
Neuzeichnen auf "Wasser" zurück.
Danach zeichne ich bis zum nächsten Setzen nicht mehr neu.
Anscheinend tut er dies aber doch,
nämlich in der Methode setValueAt von meinem TableModel.
Da das alles ja sehr schnell geht, sehe ich natürlich nur Wasser.
???:L
Meine Frage:
Wie kann es sein, das die setValueAt-Methode meines TableModels
meine ganze Tabelle neu rendert bzw. wie kann ich das verhindern??

 

[Beni]

Verhinden kannst du das nicht. So eine Tabelle wird einfach ab und zu neu gezeichnet.
Du könntest höchstens das zurücksetzen auf Wasser (hab nicht ganz kapiert, für was das gut ist) verzögern, indem du einen Thrad, oder einen javax.swing.Timer verwendest (damit kannst du das Programm "teilen" in mehrere parallel laufende Prozesse. Einen lässt du wie gewohnt Zeichnen, Benutzereingaben bearbeiten, ... .Und den anderen lässt du ein oder zwei Sekunden schlafen, und setzt dann Wasser).