Swing RadioButtons in JTable einfügen

[Ratzer]

Hallo,
ich möchte gerne eine JTable erstellen mit 5 Spalten. 2 Spalten zeigen den Vor- bzw. Nachnamen an und in den Spalten 3-5 soll jeweils ein RadioButton pro Zeile vorhanden sein. Die Namen lese ich aus einer Datei aus und schreibe sie in eine ArrayList. Mein Problem ist nun, wie ich die RadioButtons in die Tabelle einfügen kann. Um die Tabelle mit den Daten aus der ArrayList zu füllen, muss ich ja eine neue Klasse schreiben, die von AbstractTableModel abgeleitet ist. Die Methode, die für den Inhalt in der Tabelle zuständig ist, heißt ja getValueAt(). Allerdings bringt das ja nichts, wenn ich folgendes in die Methode reinschreibe:

return frame.add(new JRadioButton("blablabla", true));

Dann würde ja in der Tabelle einfach ein String an der Stelle auftauchen. Wie kriege ich das denn hin, dass der JRadioButton in der Tabelle angezeigt wird?

 

[SlaterB]

Java CodeGuru: (none)

 

[Ratzer]

In dem Beispiel wurde die JTable allerdings durch das Array befüllt. Ich befülle das JTable ja durch eine ArrayList, die ihre Daten aus einer Datei bekommt.
Kann ich das Beispiel trotzdem benutzen?

 

[Ratzer]

OK, ich habe den TableRenderer und den CellEditor jetzt einfach mal komplett von deiner Seite übernommen. Mein Problem ist jetzt allerdings, dass die RadioButtons sich nicht anklicken lassen. Also man kann zwar drauf klicken, aber der RadioButton wird nicht markiert. Ich gebe euch mal den Code meiner Klasse, die das AbstractTableModel erweitert.

public class MeineTabelle extends AbstractTableModel {

    private ArrayList<Mitglied> model;
    private String[] columnNames = {"Nachname", "Vorname", "Anwesend", "Abwesend", "Fehlend"};

	MeineTabelle(ArrayList model, Protokoll frame) {
		this.model = model;
	}

    @Override
    public String getColumnName(int columnIndex) {
        return columnNames[columnIndex];
    }

	public int getRowCount() {
		return model.size();
	}

	public int getColumnCount() {
		return 5;
	}

    @Override
    public boolean isCellEditable(int rowIndex, int columnIndex) {
        return columnIndex == 2 || columnIndex == 3 || columnIndex == 4;
    }

	public Object getValueAt(int row, int column) {
        Mitglied m = model.get(row);
        JRadioButton rb1 = new JRadioButton("Anwesend", true);
        JRadioButton rb2 = new JRadioButton("Abwesend");
        JRadioButton rb3 = new JRadioButton("Fehlend");
        ButtonGroup group = new ButtonGroup();
        group.add(rb1);
        group.add(rb2);
        group.add(rb3);

        if (column == 0)
           return m.getNachname();
        else if (column == 1)
            return m.getVorname();
        else if (column == 2)
            return rb1;
        else if (column == 3)
            return rb2;
        else if (column == 4)
            return rb3;
        return null;
	}


}

Was kann ich ändern, damit man die RadioButtons markieren kann?

 

[SlaterB]

'meine Seite' klingt gut, macht mich ja zum Guru

du darst nicht erst in der getValueAt-Methode RadioButtons erstellen
(dass das bei JEDER Zelle passiert, selber wenn gar keinerr sondern nur ein simpler Nachname, oder ansonsten höchsten einer der vielen RadioButtons benötigt wird, ist eine hohe Verschwendung)

sondern wie im Beispiel muss das Teil des Models sein, also in jedem Mitglied-Objekt gespeichert

> In dem Beispiel wurde die JTable allerdings durch das Array befüllt. Ich befülle das JTable ja durch eine ArrayList, die ihre Daten aus einer Datei bekommt.

ob nun Array oder ArrayList ist doch ganz egal, kann man immer ineinander umformen,
auch woher die Daten kommen kann keinen Einfluss haben,
wenn die Tabelle da ist, sind die Daten auch schon da, mehr muss man aus JTable-Sicht nicht wissen

ein JRadioButton im Mitglied-Objekt ist bestimmt nicht so schön, alternativ könntest du in deinem TableModel eine zweite Liste nur der RadioButtons einbauen

möglich ist bestimmt auch, im Model nur booleans vorzuhalten und den Renderer immer denselben eigenen RadioButton malen zu lassen, nur mit aktueller Belegung aus dem Model,
aber alles kompliziert, vielleicht findest du bei google noch mehr Beispiele zu RadioButton + JTable, hatte selber auch nur kurz nachgeschaut

 

[Ratzer]

OK, ich habe die RadioButtons dann einfach in das Mitglied reingepackt. Der Konstruktor sieht dann so aus:

public Mitglied(String nachname, String vorname, String nick) {
        this.nachname = nachname;
        this.vorname = vorname;
        this.nick = nick;
        bg = new ButtonGroup();
        rb1 = new JRadioButton(Protokoll.option1, true);
        rb2 = new JRadioButton(Protokoll.option2);
        rb3 = new JRadioButton(Protokoll.option3);
        bg.add(rb1);
        bg.add(rb2);
        bg.add(rb3);
    }

Die getValueAt() Methode aus dem TabelModel sieht nun so aus:

public Object getValueAt(int row, int column) {
        Mitglied m = model.get(row);        

        if (column == 0)
           return m.getNachname();
        else if (column == 1)
            return m.getVorname();
        else if (column == 2)
            return m.getButton1();
        else if (column == 3)
            return m.getButton2();
        else if (column == 4)
            return m.getButton3();
        return null;
	}

Mein Problem ist nun, dass die ButtonGroup irgendwie nicht richtig funktioniert. Pro Zeile ist ja in den letzten drei Spalten jeweils ein RadioButton. Wenn man allerdings den zweiten RadioButton in der ersten Zeile anklickt, dann ist dieser markiert, aber der erste RadioButton bleibt markiert. Das gleiche passiert mit dem dritten RadioButton in der Zeile. Wenn ich den auch anklicke, sind alle drei RadioButtons in derselben Zeile markiert, obwohl immer nur einer markiert sein sollte.
Das komische ist, wenn ich den zweiten RadioButton in der zweiten Zeile markiere, dann wird dieser auch ausgewählt, aber die ersten beiden RadioButtons in der ersten Zeile verlieren ihre Markierung. Wie kann ich diese Fehler beheben?

 

[SlaterB]

in dem Code-Beispiel ist

JTable table = new JTable(dm) {
      public void tableChanged(TableModelEvent e) {
        super.tableChanged(e);
        repaint();
      }
    };

enthalten, hast du das auch als JTable?
ist ziemlich unschön aber falls es klappt wäre das ja eine schnelle Lösung

Problem könnte sein, dass Teile der JTable nicht neu gezeichnet werden,
denn normalerweise bewirken Änderungen einzelner Zellen nicht Änderungen anderer Zellen oder Zeilen,

wie letztesmal gilt: mit mehr Aufwand ließe sich auch da was ordentliches bauen, aber ich müsste da selber erst basteln,
keine Lust (falls ich es überhaupt schaffen könnte)

 

[Ratzer]

Ja, das habe ich bei mir drinne, nur mit dem Unterschied, dass ich dem Konstruktor nicht das DefaultTableModel übergebe, weil ich das ja erst später setze:

tabelle = new JTable() {
            @Override
            public void tableChanged(TableModelEvent e) {
                super.tableChanged(e);
                repaint();
            }
        };
        
        ArrayList<Mitglied> mitglieder = Mitglied.einlesen();        
        tabelle.setModel(new MeineTabelle(mitglieder));

 

[Ebenius]

Ich würde setValueAt(Object, int, int) des TableModels überschreiben. Im Falle dass Spalte 2, 3, oder 4 (0-basiert) gesetzt wird einfach fireTableRowChanged() aufrufen. Das repaint() in tableChanged(TableModelEvent) sollte damit auch unnötig sein. In etwa (im Browser geschrieben, ungetestet):

model = new DefaultTableModel() {
  @Override
  public void setValueAt(Object o, int row, int column) {
    switch(column) {
      case 2:
      case 3:
      case 4:
        fireTableRowChanged(row);
        break;
      default:
        super.setValueAt(o, row, column);
    }
  }
}

Ebenius

 

[SlaterB]

jo, bzw. JTable.tableChanged() kommt gar nicht dran, wenn setValueAt nicht überschrieben wird und ein Event feuert,
repaint() der ganzen Tabelle ist bei den gurus nötig, weil dort noch weitere Zeilen betroffen sind

 

[Ratzer]

Ich habe die Methode jetzt so umgeschrieben:

public void setValueAt(Object value, int row, int column) {
        if (column >= 2 && column <= 4)
            fireTableRowsUpdated(2, 4);
        else {
            super.setValueAt(value, row, column);
        }
    }

Die Methode fireTableRowChanged(row) existiert gar nicht. Deswegen habe ich fireTableRowsUpdated gewählt, was vermutlich dieselbe Wirkung hat. Auf jeden Fall funktionieren die RadioButtons jetzt richtig. Danke schon mal für die Hilfe. Übrigens habe ich die repaint() Methode nicht mehr verwendet und es funktioniert trotzdem.
Jetzt habe ich allerdings noch ein letztes Problem. In der Tabelle werden die Überschriften nicht angezeigt, obwohl ich folgende Methoden in das TableModel geschrieben habe:

private String[] columnNames = {"Nachname", "Vorname", Protokoll.option1,
                                          Protokoll.option2, Protokoll.option3};
public String getColumnName(int columnIndex) {
        return columnNames[columnIndex];
    }

Woran kann das liegen?

 

[Ebenius]

Die Methode fireTableRowChanged(row) existiert gar nicht. Deswegen habe ich fireTableRowsUpdated gewählt [...]

Genau die meinte ich. Wie gesagt: Im Browser getippt.

Du musst die JTable in eine JScrollPane packen, damit der Tabellenkopf sichtbar ist. Hast Du das getan?

Ebenius

 

[Ratzer]

Jo, das habe ich gerade eben getan. Jetzt geht auch alles. Danke.

 

[Ratzer]

Jetzt ist mir gerade noch eingefallen, dass ich gerne die Einträge in den Spalten 2-4 zentriert haben möchte. Wie mache ich das?
So funktioniert es nämlich nicht:

public class RadioButtonRenderer extends DefaultTableCellRenderer {

    @Override
    public Component getTableCellRendererComponent(JTable table, Object value,
            boolean isSelected, boolean hasFocus, int row, int column) {
        if (value == null) {
            return null;
        }
        if (column >= 2 && column <= 4) {
            setVerticalAlignment(JLabel.CENTER);
	        setHorizontalAlignment(JLabel.CENTER);
        }

        return (Component) value;
    }
}

 

[Ebenius]

Du musst natürlich die RadioButtons zentrieren, nicht das Label (von dem der CellRenderer unnötiger Weise erbt und) das nie benutzt wird. Got it?

Ebenius

 

[Ratzer]

Da hast du natürlich Recht. So gehts:

public class RadioButtonRenderer implements TableCellRenderer {

    public Component getTableCellRendererComponent(JTable table, Object value,
            boolean isSelected, boolean hasFocus, int row, int column) {
        if (value == null) {
            return null;
        }
        if (column >= 2 && column <= 4) {
            ((JRadioButton)value).setHorizontalAlignment(SwingConstants.CENTER);
        }

        return (Component) value;
    }
}

 

[Ebenius]

D'accord. :-D

Ebenius

 

[Noobalter]

Seit wann erzeugt man radiobutton in der Doc-Klasse

Hat da nicht jemand ne andere Lösung

halt nach der NORM

 

[Ebenius]

Hast Du ein konkretes Problem zu lösen? Oder holst Du öfter mal anlasslos einen 8 Monate alten Thread aus der Versenkung?

Ebenius