Java JTable: Farbe von einzelnen Zeilen setzten

[FrazeColder]

Moin zusammen,

ich würde gerne für jede meiner Zeilen die Farbe ändern, sobald ein bestimmter prozentualer Wert unterschritten ist. Dann wird meine eigene Methode setRowColor() aufgerufen, die die Row ID und die Farbe der Methode übergibt.

Soweit so gut, nur wird die ganze Tabelle immer in der Farbe des letzten Elementes eingefärbt. Heißt, ist die letzte Zeile Rot, werden alle Zeilen Rot. Und das, obwohl ich die richtige Farbe der entsprechenden Farbe übergebe. Das sehe ich, da ich mir pro Zeile die Farbe ausgeben lasse. Das hier ist mein aktuelles Beispiel:

Row 0: White
Row (0) & Cell (0) to java.awt.Color[r=255,g=255,b=255]
Row (0) & Cell (1) to java.awt.Color[r=255,g=255,b=255]
Row (0) & Cell (2) to java.awt.Color[r=255,g=255,b=255]


Row 1: Orange
Row (1) & Cell (0) to java.awt.Color[r=255,g=200,b=0]
Row (1) & Cell (1) to java.awt.Color[r=255,g=200,b=0]
Row (1) & Cell (2) to java.awt.Color[r=255,g=200,b=0]

Heißt, meine erste Zeile sollte eigtl. Weiß sein und meine zweite Zeile Orange. Aber wie bereits beschrieben, werden beide Zeilen Orange... Hier mein Code der setRowColor Methode:

sellTable: meine JTable

    private void setRowColor(int row, Color color){

        for (int i = 0; i < 3; i++) {
            TableCellRenderer tableCellRenderer = sellTable.getCellRenderer(row, i);
            Component c = sellTable.prepareRenderer(tableCellRenderer, row, i);
            c.setBackground(color);

            System.out.println("Row (" + row + ") & Cell (" + i + ") to " + color.toString());
        }

        System.out.println();
        System.out.println();

    }

Habt ihr eine Idee, wie ich das lösen kann oder wo mein Problem liegt?
Anscheinend ja an der Tabelle..

 

[andy_m4]

Wenns der DefaultTreeCellRenderer ist (wovon ich ausgehe), dann bietet der direkt eine Methode setBackgroundNonSelectionColor usw.
Damit würde ich es machen, statt mir die component zurück geben zu lassen und dort Attribute zu ändern.

https://docs.oracle.com/javase/10/docs/api/javax/swing/tree/DefaultTreeCellRenderer.html

    private void setRowColor(int row, Color color){

        for (int i = 0; i < 3; i++) {
            TableCellRenderer tableCellRenderer = sellTable.getCellRenderer(row, i);
            //Component c = sellTable.prepareRenderer(tableCellRenderer, row, i);
            //c.setBackground(color);
            tableCellRenderer.setBackgroundNonSelectionColor(color);

            System.out.println("Row (" + row + ") & Cell (" + i + ") to " + color.toString());
        }

        System.out.println();
        System.out.println();

    }

 

[Nolag]

Der korrekt Weg ist eigentlich einen eigenen CellRenderer zu implementieren, der die Farbe abhängig vom Zellenwert selbst ermitteln und setzen kann.
Hier ist ein Beispiel: https://stackoverflow.com/questions/5673430/java-jtable-change-cell-color

 

[Grimba]

Hilft dir das hier weiter?
https://stackoverflow.com/questions/15071668/cell-renderer-for-jtable-coloured-rows

So wie ich das verstehe, setzt du die ganze Zeit den gleichen CellRenderer ein. Mit der Zeit bekommt so jede Zelle, Reihe für Reihe, Spalte für Spalte, den gleichen CellRenderer zugewiesen (bzw. hatte sie ihn schon, du holst ihn ja vorher mit einem getter ab. Ist wohl der Default CellRenderer). Änderst du desse Farbe, ändern sich alle Zellen, denen dieser bereits zugewiesen wurde. Bei der letzten Farbe sind das halt alle vorherigen.
Du musst wohl verschiedene neu erzeugen.

(Irrtum nicht ausgeschlossen, meine Java Zeit ist etwas her... )

 

[andy_m4]

So wie ich das verstehe, setzt du die ganze Zeit den gleichen CellRenderer ein. Mit der Zeit bekommt so jede Zelle, Reihe für Reihe, Spalte für Spalte, den gleichen CellRenderer zugewiesen. Änderst du desse Farbe, ändern sich alle Zellen, denen dieser bereits zugewiesen wurde.

Du hast recht. Damit ist meine Lösung ähm ... nicht ganz passend :-)

 

[FrazeColder]

setBackgroundNonSelectionColor(color);

Diese Method gibt es bei mir leider nicht. Wo hast Du die her?

 

[Rossie]

Du musst wohl verschiedene neu erzeugen.)

Nein, wie @Nolag schon schrieb: einen eigenen CellRenderer implementieren (einfach DefaultTableCellRenderer ableiten) und dort die Hintergrundfarbe setzen.

 

[Grimba]

Das war eigentlich mit verschiedene neu erzeugen gemeint, das ist doch kein Widerspruch

 

[Rossie]

Das war eigentlich mit verschiedene neu erzeugen gemeint, das ist doch kein Widerspruch

So wie ich das verstehe, setzt du die ganze Zeit den gleichen CellRenderer ein [...] Du musst wohl verschiedene neu erzeugen.

Ich habe daraus gelesen, dass der Ansatz mit lediglich einem Renderer falsch ist. Das "verschiedene" im letzten Absatz bezieht sich aber wohl auf Farben. Trotzdem würde ich vermeiden im Renderer Objekte zu erzeugen. Für Farben Konstanten verwenden.