Icon in JTable

[Guest]

Hallo zusammen,

hab da mal eine Frage zu o.g. Betreff.

Ich benutze einen TableCellRenderer der von DefaultTableCellRenderer abgeleitet ist.

NUn habe ich folgenden Code:

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


Component comp = super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);


String s = model.getValueAt(row, 6).toString();
		
		if(s.equalsIgnoreCase("K"))
		{
			setIcon(kIcon);
		}
		else
		{
			setIcon(null);
		}
		
		
		return comp;

Funktioniert auch (fast)
Mein Problem ist nun, dass ich lediglich vor den Text in Spalte 7 das Icon platzieren moechte, in Abhaengigkeit davon, ob in Spalte 6 ein "K" steht.

Mein Renderer setzt mir allerdings in jeder Zeile der betroffenen Spalte ein Icon.
Wie kann ich diesen Schoenheitsfehler korregieren?

 

[Niki]

Probiers mal so:

      super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);


      String s = model.getValueAt(row, 6).toString();
      
      if(s.equalsIgnoreCase("K"))
      {
         setIcon(kIcon);
      }
      else
      {
         setIcon(null);
      }
      
      
      return this;

 

[FenchelT]

ups sehe gerade, war eben gar nicht angemeldet :roll:

Nein, das funktioniert leider nicht, aber danke erstmal fuer DEine Hilfe.

Ich hole mir ja in die Variable comp den Wert aus Spalte 6 um zu gucken, ob dort ein K steht.

Nun moechte ich aber in Spalte 7 der gleiche Zeile ein Icon vor den Wert setzen.
Das return this macht mir genau das gleiche wie ein return comp; es setzt mir vor jeden Wert in jeder Spalte einer betroffenen Zeile ein Icon :?

Mein Problem besteht also weiterhin :bahnhof:

 

[Wildcard]

Ein Renderer stellt den Zustand von 'value' grafisch da.
Alle dafür notwendigen Informationen sollten in 'value' hinterlegt sein.

 

[FenchelT]

Hallo Wildcard,

wie soll ich das denn Deiner Meinung nach im value hinterlegen?

Ich kann im Model nicht hingehen und die Spalte als Icon.class zurueckgeben, da mir ansonsten der Text in der Spalte verloren geht.

Heisst, das Model liefert mir diese Spalte als Object zurueck.
Dementsprechend kann ich im Renderer in der setValue Methode nicht auf instanceof Icon abfragen.

Oder hab ich Dich jetzt ganz falsch verstanden?


Gruesse

 

[Wildcard]

Du willst für bestimmte Objekte ein Icon anzeigen. Dann sollte aus deinem Datenmodell ersichtlich sein für welche.
Dein 'value' objekt muss also alle notwendigen Informationen mitbringen (direkt oder indirekt) um es anzeigen zu können.

 

[FenchelT]

Du willst für bestimmte Objekte ein Icon anzeigen.

NUnja nicht so ganz. Ich moechte nicht anstatt, sondern sowohl als auch.
In Spalte 6 steht ein Buchstabe (z.B. k,i,o,l) In Spalte 7 steht eine Beschreibung als Text.
Nun moechte ich

if (spalte 6 == "K")
{
spalte 7 = entsprechendes Icon + Beschreibungstext
}
else if (spalte 6 == "L")
{
spalte 7 = entsprechendes Icon + Beschreibungstext
}
else
{
nur der Beschreibungstext
}


Mein Modell kann daher fuer Spalte 7 nur Object.class sein und nicht z.B. Icon.class
Das ist der Grund, warum ich dachte es ueber einen CellRenderer zu machen.

Wenn mein Vorhaben so nicht funktioniert (was mich wundern wuerde) muesste ich eine sep. Spalte
einfuegen und im Model sagen, Du bist Icon.class.

Wenn ich es so mache, wie in meinem ersten Codebeispiel, setzt er ja vor ALLE Texte in jeder Spalte ein Icon, in Abhaengigkeit davon, ob in Spalte 6 irgendwo ein "K" steht.

 

[Wildcard]

Du kannst dein Problem sicherlich lösen in dem du die Parameter row und column auswertest, aber was ich versuche dir zu sagen ist, dass sich das nicht nach einem sinnvollen Modell anhört.
Dein Renderer 'denkt' zu viel, bzw. ist hart kodiert. Du solltest über ein besseres Datenmodell nachdenken das alle benötigten Informationen enthält.

 

[FenchelT]

Okay, danke Dir, werde versuchen, die "Logik" ins Modell zu verlagern

Gruesse
FenchelT

 

[FenchelT]

@wildcard:

Danke nochmal fuer den Schupser in die richtige Richtung. Habe das Prinzip nun verstanden


Gruesse