TableCellRender für ALLE Spalten

[pro_evo]

hi all,


ich hab folgendes vor :

Ein JTable zeigt Verzeichnisse an mit dem Spalten: Name, Größe (Double) , Typ (String), Zuletzt Geändert (Datum), Ist ein Ordner (Boolean)

soooo

Jetzt wollte ich die Zeilen, die Ordner sind anders farbig darstellen (Weil man auf die nämlich klicken kann)

Dumm ist der Renderer bezieht auf Spalten, aber egal es haut auch schon ganz gut hin, im JTable Konstruktor hab ich stehen :

	tab.setDefaultRenderer(java.lang.Object.class, new myTabelCellRenderer());

Dazu den Kommentar : // LOL allgemeiner gehts doch ned !

Problem is nämlich: die Spalten "Größe" und "Ist Ein Ordner" werden ned gerendert ...

hat da jemand ne Idee ? ...

danke im Voraus !


p.s.
Mein Renderer sieht übrigens so aus :

public class myTabelCellRenderer implements TableCellRenderer {

	JLabel l;
	public Component getTableCellRendererComponent (JTable table, Object value, 
						boolean isSelected, boolean hasFocus, int row, int column) {
		
		TableModel tmod = table.getModel();
		
		l = new JLabel();
		l.setOpaque(true);
		
		l.setBackground(new Color(175,200,219));
		
		if (((Boolean)tmod.getValueAt(row,4)).booleanValue()) {  //ist value ein Ordner?
			l.setBackground(new Color(238,206,145));
		}
		
		l.setText(value.toString());
		return l;
	}
}

 

[Sky]

vielleicht weil's sich Double bzw. Booleanwerte handelt??

Für diese Typen musst Du auch explizit den Renderer setzen!

 

[pro_evo]

Aber sind Boolean und Double (GROßER Anfangsbuchstabe) denn keine Objects ? Alle Klassen extenden doch standartmäßig Object.

 

[Beni]

Doch doch, es sind schon Objekte. Wenn aber das JTable z.B. einen Boolean sieht, dann guckt es in einer internen Tabelle nach, und sieht dort, dass für Boolean ein ganz spezieller Renderer verwendet werden soll.

In Pseudocode, was die JTable macht:

Map<Class, TableCellRenderer> renderer = ... // die wird irgendwann mal mit etwa 6 verschiedenen Renderern gefüllt.

public TableCellRenderer suchDenPassendenRenderer( Objekt x ){
  Class clazz = x.getClass(); //  Für ein Boolean wäre das "java.lang.Boolen".

  while( true ){
    if( map.get( clazz ) != null ) // nachgucken, ob es einen Eintrag für eine bestimmte Klasse gibt.
      return map.get( clazz ); // für diese Klasse gibts einen Renderer
    else
      clazz = clazz.getSuperClass();  // Nix gefunden, andere Klasse ausprobieren
                                                 // die Superclass von z.B. JButton ist AbstractButton
  }
}

 

[pro_evo]

ah ok danke das klingt einleuchtend

hab jetzt explizit für Boolean noch einen gesetzt und siehe da funzt ... bissel blöd er schreibt halt jetz true und fals in Worten rein und nimmer als Haken , ok is auch logisch (nutzt nimmer den Standart Boolean Renderer) , aber einfach eine Row der Tabelle (bzw. bestimmte Rows) farbig zu hinterlegen geht anscheinend nicht ....

bissel schoener siehts dann aus wenn Nur die Erste Spalte Farbig anders ist (anstatt mal so mal so) und das hab ich sogar hinbekommen, aber haette ja nie gedacht dass das funzt schaut euch das mal an *g* ( 2 und 3 sind die anderen Spalten die eben wie die Erste auch String sind) :

	public Component getTableCellRendererComponent (JTable table, Object value, 
						boolean isSelected, boolean hasFocus, int row, int column) {
		
		
		if (column == 2 || column == 3) { //ROFL FUNZT 
			JTable frischerTable = new JTable();												//  hier egal welcher TABLE ???
			return frischerTable.getDefaultRenderer(String.class).getTableCellRendererComponent(frischerTable, value, isSelected, hasFocus, row, column);
		}
		
		
		TableModel tmod = table.getModel();
		
		l = new JLabel();
		l.setOpaque(true);
		
		l.setBackground(new Color(175,200,219));
		
		if (((Boolean)tmod.getValueAt(row,4)).booleanValue()) {  //ist value ein Ordner?
			l.setBackground(new Color(238,206,145));
		}
		
		l.setText(value.toString());
		
		return l;

an der Stelle wo "frischerTable" übergeben wird scheint es egal zu sein welcher Table, schreib ich "table" rein siehts genauso aus ...

 

[Beni]

*Hust*, das gehört wohl in die Rubrik "Wie mache ich mein Programm möglichst ineffizient".
Also: pro Sekunde erstellst du mit diesem Code mehrere 100 JTables, wieso das nicht gut ist, kannst du selbst rausfinden ;-)

Ein paar Ideen:
1. solltest du nicht im Renderer auf das TableModel zugreiffen. Falls du mal das Model umorganisierst, fliegen die Exceptions. Besser ist es, wenn du alle benötigten Infos aus "value" direkt holst. (Du kannst ja auch was komplizierteres, als nur ein String übergeben ;-) )
2. l = new JLabel(); kannst du auch gleich knicken. Noch so ein Fall "Wie kriege ich mein Programm langsam". Benutze immer dasselbe Label. Die Component die du hier zurückgibst wird nur ein einziges mal benötigt um ein kleines Bildchen zu zeichnen, danach greifft die JTable nie mehr darauf zu.
3. Erbe doch ganz einfach von "DefaultTableCellRenderer". Dann kannst du am Anfang "getTableCellRendererComponent..." aufrufen, und hast mal die Standartfarben. Danach kannst du Hintergrund (falls notwendig) abändern, und die Component (im Falle des DefaultTableCellRenders muss man "return this;" schreiben) zurückgeben.

 

[pro_evo]

ahh Danke für den Hinweis, das stimmt natürlich ... hab auch in der DefaultTableCellRenderer Classe das hier gefunden :

The standard JLabel component was not designed to be used this way and we want to avoid triggering a revalidate each time the cell is drawn. This would greatly decrease performance because the revalidate message would be passed up the hierarchy of the container to determine whether any other components would be affected. So this class overrides the validate, revalidate, repaint, and firePropertyChange methods to be no-ops. If you write your own renderer, please keep this performance consideration in mind.

Ich denk das is genau das was du meinst oder ?


also ich habs jetzt so gemacht wie du vorgeschlagen hast :

Ok ich mach keine 100 JTables mehr und keine 100JLabels mehr, ABER an der Funktionalität hat sich nix getan ... im Gegenteil, die Spalten 2 und 3 sind Farbig unterlegt obwohl sie es nicht sein sollten (

Man sieht aber einen Unterschied : Spalte 1 ändert sich beim Markieren der Reihe nicht, Spalte 2 und 3 dagen schon (Standart Java Hellblau Markierung da) ... also irgendwas stimmt da ned ...


Zu deimem Punkt 1.: Ok ich habe jetzt aus dem JTable die Info ob die Reihe ein Ordner ist, aber das nutzt ja intern wiederum das Model nehme ich an oder ? Also kein Vorteil. Aber wie soll ich rein aus dem value rausbekommen ob die REIHE des Values (kann ja auch die Größe sein) ein Ordner ist ?

public class myDefaultTableCellRenderer extends DefaultTableCellRenderer {

	public myDefaultTableCellRenderer () {
		super();
	}
	public Component getTableCellRendererComponent(JTable table, Object value, 
			boolean isSelected, boolean hasFocus, int row, int column)  {
		
		if (column == 2 || column == 3) { 
	         return super.getTableCellRendererComponent(table,value, 
	    			 isSelected,  hasFocus,  row,  column);  // FARBE IS TROTZDEM GESETZT :(
	      } 
		
		this.setBackground(new Color(175,200,219));
		
		if (((Boolean)table.getValueAt(row,4)).booleanValue()) { //ist value-Reihe ein Ordner?
			this.setBackground(new Color(238,206,145));
		}
		
		setText(value.toString());
		return this;
	}
}

 

[Beni]

Ok, zu den Farben. Da hab ich nicht daran gedacht, dass das dauerhaft ist :?

Einfache Lösung: dann muss man sie halt von Hand setzen. Ich hab da ein Beispiel zusammenkopiert, der Renderer wählt je nach Spalte Rot als Hintergrund, oder die Standardfarben.

import java.awt.Color;
import java.awt.Component;

import javax.swing.JFrame;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.table.DefaultTableCellRenderer;

public class Test{
   public static void main( String[] args ){
      // Die Daten für das Table
      String[][] data = new String[][]{
            {"a", "b", "c", "d"},
            {"e", "f", "g", "h"},
            {"i", "j", "k", "l"}
      };
      
      // Die Column-Titles
      String[] title = new String[]{
            "A", "B", "C", "D"
      };
      
      // Das JTable initialisieren
      JTable table = new JTable( data, title );
      table.setDefaultRenderer( Object.class, new Renderer() );
      
      JFrame frame = new JFrame( "Demo" );
      frame.getContentPane().add( new JScrollPane( table ) );
      frame.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
      frame.pack();
      frame.setVisible( true );
   }
}

class Renderer extends DefaultTableCellRenderer{
    public Component getTableCellRendererComponent( JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column ) {
        super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
        if (isSelected) {
           setForeground(table.getSelectionForeground());
           setBackground(table.getSelectionBackground());
        }
        else if( column == 2 || column == 3 ) {
            setForeground( table.getForeground());
            setBackground( table.getBackground());
        }
        else{
            setBackground( Color.RED );
        }
        return this;
    }
}

Zum value:
In dieser Spalte speicherst du anstelle von

String dateiname

ein

Info dateiname

wobei Info das hier ist:

public class Info{
  private String name;
  private Boolean directory;

  set/get....
}

Dann kannst du im Renderer einen einfachen Cast machen.

getTableCellRendererComponent( ... ){
  Info info = (Info)value;
  if( info.isDirectory() ){...}
  else {...}
}

 

[pro_evo]

Danke das mit der Farbe Funzt

Zu dem Vaule:

Eigentlich eine gute Idee von dir, aber ich habe ja noch die Spalte Größe (Double).

Dann koennste es ja zu ner Cast Exception kommen, soll ich die Größen Info auch noch in die Info KLasse mit reinschreiben ?

 

[Beni]

Hm, die Spalte mit den Doubles benötigt auch zusätzliche Infos? Jetzt wirds langsam kompliziert :?

Ok, IMHO, und wenn ich viel Zeit hätte, würde ich etwas von diesem hier machen:
1. Für jede Spalte eine eigene Info-Klasse, oder
2. Für dieselbe row immer dasselbe Info-Objekt (unabhängig der column) zurückgeben, und verschiedene Renderer benutzen, welche nur Teile von dem Info-Objekt darstellen (bzw, ein Renderer der für verschiedene Columns was anderes macht. Z.b. einen weiteren Renderer aufruft. Ein Aufruf von "getCellRendererComponent" kann man ja einfach weiterleiten an ein anderes TableCellRenderer-Objekt), oder
3. Auf das Model zugreiffen, und einen dicken, fetten Kommentar mit Ausrufezeichen machen, das hier implizit gewissen Annahmen gemacht werden, und dass diese allenfalls das Programm zum Absturz bringen könnten (wegen ClassCastException).

 

[pro_evo]

ok dann nehm ich Nummer 3