Swing JTable Markierung

[JensArpe]

Huhu,

habe folgendes kleines Problemchen mit folgendem Code:

jTable1 = new JTable(new ParameterEditorTabellenModell(new ParameterListe("paramliste"))){

    public Component prepareRenderer(TableCellRenderer renderer, int Index_row, int Index_col) {
        Component comp = super.prepareRenderer(renderer, Index_row, Index_col);
        if (isCellSelected(Index_row, Index_col)) {
            comp.setBackground(Color.blue);
        } else if (!(Boolean)getModel().getValueAt(Index_row, 6)){
            comp.setBackground(new Color(140, 140, 140));
        }  else {
            comp.setBackground(Color.white);
        }       
        return comp;
    }
};

Meine Tabelle hat 7 Spalten. Bis auf Spalte 7 sind das alles Ints und Strings. Feld 7 ist ein Boolean Feld (wird als ComboBox gerendert).

Jetzt wird aber im Code nur bei der Anweisung in Zeile 5/6 die ganze Zeile inklusive der ComboBox eingefärbt. In beiden else-Teilen wird aber nur alles außer der ComboBox eingefärbt.

Was mach ich da wohl falsch?

 

[KrokoDiehl]

Ich nehme an, dass du eine CheckBox anstelle einer ComboBox meinst (das mit dem Häkchen)?
Ich verstehe deine Aussage so, das

comp.setBackground(Color.blue);

funktioniert, die anderen

comp.setBackground(...);

nicht.
Klingt seltsam ... du kannst ja mal probieren, dass im Falle einer CheckBox die Eigenschaften

setOpaque(true)

und

setContentAreaFilled(true)

gesetzt werden.

 

[palador]

Erstmal ein bissl code:

JTable jTable1 = new JTable(model) {

			public Component prepareRenderer(TableCellRenderer renderer, int Index_row, int Index_col) {
 			// ...
			}
			// neu
			@Override
			public Component prepareEditor(TableCellEditor editor, int row, int column) {
				Component comp = super.prepareEditor(editor, row, column);
				if(column == 6) {
					comp.setBackground(Color.BLUE);
				}
				return comp;
			}
			// END neu
		};

kurz: zum Verändern und zum Bearbeiten werden verschiedene Components eingesetzt (Renderer und Editors). Wenn du auf dein Häkchen klickst veränderst du ja den Zellenwert und dazu wird der Editor verwendet.

 

[JensArpe]

Hmm.. also danke mal für eure Antworten. Und ja ich meinte natürlich CheckBox

Und es ist so wie KrokoDiehl sagte. Das eine

comp.setBackground(Color.blue);

in Zeile 6 geht. Es markiert auch die ganze Zeile in blau. Die anderen beiden in Zeile 8 und 10 hingegen gehen nicht. Da wird alles markiert außer der CheckBox.

@palador Ich will ja noch garnichts editieren, sondern einfach nur markieren. Ich habe deinen Code mal reingemacht und es wurde zwar blau, aber vermutlich weil beim reinklicken ja die ganze Zeile markiert wird.

Lustigerweise wollte ich mal nen KSKB machen um euch zu zeigen daß es nicht geht.. Aber komischerweise kommt der Fehler da nicht.. sondern nur in meinem Programm ;(

 

[KrokoDiehl]

Probier mal bitte folgenden Zusatz aus. Er ist nicht hübsch geschrieben, aber vielleicht kommt es dem Problem näher. Ich arbeite eigentlich mit eigenen Renderer-Implementierungen und bei einem CheckBox-Renderer habe ich jene beiden Methoden an nötig beschrieben:

jTable1 = new JTable(new ParameterEditorTabellenModell(new ParameterListe("paramliste"))){
 
    public Component prepareRenderer(TableCellRenderer renderer, int Index_row, int Index_col) {
        Component comp = super.prepareRenderer(renderer, Index_row, Index_col);
        if (comp instanceof JCheckBox)
        {
            ((JCheckBox) comp).setOpaque(true);
            ((JCheckBox) comp).setContentAreaFilled(true);
        }

        if (isCellSelected(Index_row, Index_col)) {
            comp.setBackground(Color.blue);
        } else if (!(Boolean)getModel().getValueAt(Index_row, 6)){
            comp.setBackground(new Color(140, 140, 140));
        }  else {
            comp.setBackground(Color.white);
        }       
        return comp;
    }
};

 

[palador]

kannst du wenigstens mal ein paar screenshots bereitstellen? ich kann mir zwar vorstellen, was zu meinst, nur um sicherzugehen.

 

[JensArpe]

So startet das Programm. Eigentlich sollte jetzt die eine Zeile durchgehend markiert sein, und sonst soollte der Hintergrund in der letzten Spalte weiß sein wie in den anderen auch.



Jetzt markier ich was anderes. Und siehe da er malt alles blau.



wenn ich dir grau hinterlegte Zeile markiere gehts ebefalls.

 

[palador]

in der tat: seltsam, das alles.
welches look&feel benutzt du denn? kannst du mal ein anderes probieren, nur um diesen faktor auszuschließen? auch wenn ich das für unwahrscheinlich halte...

 

[JensArpe]

Davor: com.sun.java.swing.plaf.gtk.GTKLookAndFeel

Nun: javax.swing.plaf.metal.MetalLookAndFeel




Nun ja... bleibt immer noch die Frage: warum geht es denn mit der blauen Markierung?

 

[palador]

Also ist erstmal das L&F schuld an der falschen Darstellung?!
Für diesen Fall als Workarround (zum probieren):

// prepareRenderer(...) nicht mehr überschreiben
		JTable jTable1 = new JTable(model);

		TableCellRenderer renderer = new TableCellRenderer() {

			final JCheckBox checkBox = new JCheckBox();
			final JLabel intLabel = new JLabel();
			final Border focusBorder = UIManager.getBorder("Table.focusCellHighlightBorder");
			final Border noFocusBorder = BorderFactory.createEmptyBorder(1, 1, 1, 1);

			{
				checkBox.setHorizontalAlignment(SwingConstants.CENTER);
				intLabel.setHorizontalAlignment(SwingConstants.RIGHT);

				checkBox.setOpaque(true);
				intLabel.setOpaque(true);
			}

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

				int modelColumn = table.convertColumnIndexToModel(column);

				if (modelColumn == 6) {
					result = checkBox;
					checkBox.setSelected((Boolean) value);
				} else {
					result = intLabel;
					intLabel.setText(Integer.toString((Integer) value));
				}

				result.setBorder(hasFocus ? focusBorder : noFocusBorder);
				result.setFont(table.getFont());

				Color bgClr;
				if (isSelected) {
					bgClr = Color.BLUE;
				} else if (Boolean.TRUE == table.getModel().getValueAt(table.convertRowIndexToModel(row), 6)) {
					bgClr = Color.LIGHT_GRAY;
				} else {
					bgClr = Color.WHITE;
				}
				result.setBackground(bgClr);

				return result;
			}
		};

		jTable1.setDefaultRenderer(Integer.class, renderer);
		jTable1.setDefaultRenderer(Boolean.class, renderer);

ansonsten: mach doch zum testen an jede stelle wo du die farbe setzt ein sysout wie

if(row == 6) System.out.println("set bgclr BLUE");

Nicht das dein L&F den TableRenderer cached oder so... kann ich mir zwar wieder nicht vorstellen, aber wir währen erstmal zu 100% sicher, dass die verschiedenen Farben überhaupt gesetzt werden.

Und btw: die row und column coordinaten die du in der prepareRenderer(..., int row, int column) bekommst sind View-Koordinaten. du must sie mit table.convertColumnIndexToModel(...) oder table.convertRowIndexToModel(...) in Modell-Koordinaten umwandeln, ansonsten bekommst du seltsame Ergebnisse, wenn sich die Reihenfolge deiner Zeilen durch z. B. sortieren ändert.

 

[JensArpe]

So.. vielen Danke für den Workaround. Klappt jetzt alles wunderbar.

Und es wird für jede Spalte (auch die mit der ComboBox) die setBackground-Methode aufgerufen. Das hatte ich vorher schon mal getestet.