JCellRenderer läuft nur einmal in bestimmte Bedingung

[Silkstring]

Hallo, ich habe mir gestern das JTable Tutorial von Beni durchgelesen und habe mir dort seinen "MyTableCellRenderer" abgeguckt. Ich brauche nun einen CellRenderer der in Abhängigkeit des Wertes "unsichtbar" eines Feld-Objekts (dem aus der Tabellenzelle) die einzelne Zelle farbig markiert, in meinem Beispiel hab ich das gemacht indem ich die Schriftfarbe einfach grau gesetzt hab. Ich habe in der Tabelle ne Menge Daten von denen einige dieses "unsichtbar" Flag haben, allerdings wird immer nur exakt das erste mit diesem Flag entsprechend markiert. Dazu hab ich nen kleinen Screenshot:

[EDIT]Auf diesem Bild haben alle Elemente außer "p_test5" dieses "unsichtbar" Flag, aber nur das erste Vorkommende wird markiert.

Das ist mein CellRenderer:

public class MyTableCellRenderer 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);

      Feld feld = (Feld) value;

      if (feld.getName().equals("") == false) {
        // Anzuzeigender Text
        setText(feld.getName();

        // Unsichtbar - Hintergrundfarbe hellgrau
        if (feld.getUnsichtbar())
          setForeground(Color.LIGHT_GRAY);
        else
          setForeground(Color.BLACK); //Ohne explizites neusetzen werden ALLE Zellen grau!!
      }
      else
        setText("");

      return this;
   }
}

Ich habe übrigens noch ein zweites Problem. Wenn ich eine Zelle auswähle wird automatisch die gesamte Zeile mitmarkiert. (Siehe Bild) Kann man das abschalten?

Danke im Voraus

 

[Mag1c]

Hi,

bist du 100 pro sicher, daß nicht nur "p_test9" das Unsichtbar-Flag gesetzt hast ?

Es wäre übrigens ein etwas besserer Stil, wenn du die Component von "super.getTableCellRendererComponent" hernehmen würdest, daran dann setText / setBackground aufrufst und diese dann per return zurückgibst.

zur Selektion solltest du dir mal

JTable#setSelectionMode
JTable#setSelectionModel

anschauen.

Gruß
Mag1c

 

[Silkstring]

Danke für die schnelle Antwort
Ja alle von mir genannten Felder haben definitiv dieses unsichtbar-flag.

 

[Mag1c]

Hmm,

ist das exakt dein ausgeführter Code (ist ein Typo drin, es fehlt ne Klammer) ? Ansonnsten müsstest du mehr Code posten (z.B. den, wo du die Daten für das TableModel erzeugst).

Gruß
Mag1c

 

[Illuvatar]

Es wäre übrigens ein etwas besserer Stil, wenn du die Component von "super.getTableCellRendererComponent" hernehmen würdest, daran dann setText / setBackground aufrufst und diese dann per return zurückgibst.

Das kommt aufs gleiche heraus, weil super.getTableCellRendererComponent(bla) auch nur this zurückgibt.
Edit: Ach besserer Stil - naja ka... ich finds so ok

Also bei mir funktioniert dein Code gut:

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class JTableTest extends JFrame
{
  // Anfang Variablen
  JTable jt = new JTable (new Object[][]{
    {
      new Feld ("p_test5", false), new Feld ("xyz", false)
    },{
      new Feld ("p_test9", true), new Feld ("xyz", false)
    },{
      new Feld ("p_test7", true), new Feld ("xyz", false)
    },{
      new Feld ("p_test8", true), new Feld ("xyz", false)
    },{
      new Feld ("p_test6", true), new Feld ("xyz", false)
    } ,{
      new Feld ("p_test10", true), new Feld ("xyz", false)
    }
  }, new Object[]{
    "bla1", "bla2"
  });
  // Ende Variablen

  public JTableTest(String title)
  {
    // Frame-Initialisierung
    super(title);
    addWindowListener(new WindowAdapter() {
      public void windowClosing(WindowEvent evt) { System.exit(0); } });
    Container cp = getContentPane();
    cp.setLayout(new BorderLayout());

    // Anfang Komponenten
    jt.setDefaultRenderer (Object.class, new MyTableCellRenderer());
    add ("Center", jt);
    // Ende Komponenten

    setSize (800,600);
    setLocationRelativeTo (null);
    setVisible(true);
  }

  // Anfang Ereignisprozeduren
  // Ende Ereignisprozeduren

  private class Feld
  {
    public Feld (String bla, boolean invis)
    {
      this.bla = bla;this.invis = invis;
    }
    public String bla;
    public boolean invis;
  }
  private class MyTableCellRenderer extends javax.swing.table.DefaultTableCellRenderer{
    public Component getTableCellRendererComponent(JTable table, Object value,
                                                  boolean isSelected, boolean hasFocus,
                                                  int row, int column)
    {
      super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);

      Feld feld = (Feld) value;

      if (feld.bla != null) {
        // Anzuzeigender Text
        setText(feld.bla);

        // Unsichtbar - Hintergrundfarbe hellgrau
        if (feld.invis)
          setForeground(Color.LIGHT_GRAY);
        else
          setForeground(Color.BLACK); //Ohne explizites neusetzen werden ALLE Zellen grau!!
      }
      else
        setText("");

      return this;
    }
  }

  public static void main(String[] args) {
    new JTableTest("JTableTest");
  }
}

 

[Silkstring]

hmm, also ich benutze jdk 1.4.2, übriger Code wäre der hier:

Hier wird die Tabelle gesetzt:

    tablemodel = new MyTableModel(feldliste, ueberschriften);
    jT_felder = new JTable(tablemodel);
    jT_felder.setDefaultEditor(Object.class, new MyCellEditor(new JTextField()));
    jT_felder.setDefaultRenderer(Object.class, new MyTableCellRenderer());
    JScrollPane jsp = new JScrollPane(jT_felder);

mein TableModel:

public class MyTableModel extends DefaultTableModel {
  public MyTableModel(Object[][] p0, Object[] p1) {
    super(p0, p1);
  }
  public MyTableModel(int r, int c) {
    super(r, c);
  }
  public boolean isCellEditable(int r, int c) {
    return true;
  }
}

und mein CellEditor (Der mir ebenso enorm Kopfzerbrechen bereitet):

public class MyCellEditor extends DefaultCellEditor {
  public MyCellEditor(JTextField jTF) {
    super(jTF);
    jTF.setEditable(true);
  }

  public boolean isCellEditable(EventObject e) {
    if (e instanceof MouseEvent) {
      MouseEvent me = (MouseEvent) e;
      if (me.getClickCount() == 2) { // Doppelklick
        Object feld = super.editorComponent;
        if (feld instanceof JTextField) {
          new STDFeldEigenschaften((Feld) feld);
        }
      }
    }
    return false;
  }
}

hm illuvatar.. wahrscheinlich funktioniert es wirklich nur bei mir nicht
aber - um nochmal eine frage vorweg reinzuwerfen:

bei dem CellEditor ist es folgendermaßen:
Nach einem Doppelklick auf eine Zelle soll sich ein Eingabedialog öffnen (STDFeldEigenschaften), dieser braucht ein Feld als Parameter, ich dachte das ich mir dieses über super.editorComponent bzw super.getCellEditorValue() holen kann. Leider geben beide nur jeweils ein leeres JTextField zurück mit dem man nichts anfangen kann, da getText() dementsprechend nur einen leeren String zurückgibt. Der Feldname in der Zelle würde mir auch schon genügen da ich mir über ihn das entsprechende Feld holen kann das ich brauche.. weiß jemand da vielleicht ne Lösung? Ich hatte auch schon versucht den Dialogaufruf in die isCellEditable Methode vom CellRenderer reinzumachen aber das funktioniert nicht weil ich da kein Event habe womit ich ihn bedingt aufrufen kann. :|