CellRenderer für ganze Spalte

[Basti4231]

Hey Leute,

ich habe eine JTable und habe für eine der Spalten einen eigenen Renderer geschrieben.

public class GregorianCalendarRenderer extends DefaultTableCellRenderer {

	public GregorianCalendarRenderer() {
	super();
	}

	@Override
	public void setValue(Object value) {
		setText((value == null) ? "" : getFormattedValue(value));
		System.out.println(getFormattedValue(value));
	}

	private String getFormattedValue(Object value) {
		if (value instanceof GregorianCalendar) {
			GregorianCalendar gc = (GregorianCalendar) value;
			return gc.get(GregorianCalendar.DAY_OF_MONTH)
					+ "."
					+ gc.get(GregorianCalendar.MONTH)
					+ "."
					+ String.valueOf(gc.get(GregorianCalendar.YEAR)).substring(
							2);
		} else
			return "";
	}
}

Ich benutze diesen Renderer für die Spalte, die mit GregorianCalendar arbeitet (Die 1. nämlich Datum)

TableColumnModel colModel = table.getColumnModel();
		colModel.getColumn(0).setCellRenderer(new GregorianCalendarRenderer());

Wenn ich jetzt aber meinen Table anzeigen lasse, so ist keine Formatierung passiert. Mir ist aufgefallen, dass die überschriebene setValue() -Methode nie aufgerufen wird.


Wo ist hier das Problem?


Gruß Basti

 

[SlaterB]

ich habe gerade 5 Min. verbracht, ein beliebiges Beispiel aus dem Internet als Grundlage zu suchen,
MultiLine Table : Grid TableSwing ComponentsJava
alle dortigen Renderer rauszuschmeißen, deine Klasse und die beiden Zeilen für den Aufruf einzufügen,

und was soll man sagen, natürlich geht es, zumindest kommt eine Ausgabe falls noch ergänzt,
hätte ich mir alles sparen können

ohne vollständiges Testprogramm ist der Fehler nicht zu bestimmen, es müssen 100 Einzelschritte korrekt ineinandergreifen damit alles funktioniert,
wenn die Variable 'table' ein andere Tabelle enhält, als in der GUI zu sehen, hat der schönste Renderer keine Funktion

edit:
schon 34 Themen, nichts über solche simplen Wahrheiten bisher gelernt?

 

[xehpuk]

Sieht sehr abenteuerlich aus, was du da machst.

Man überschreibt/implementiert normalerweise die Methode

TableCellRenderer#getTableCellRendererComponent(JTable, Object, boolean, boolean, int, int)

Des Weiteren gibt es [JAPI]java.text.SimpleDateFormat[/JAPI], um Daten nach einem Pattern zu formatieren.

Hier ein kleines Beispiel zum Testen:

import java.awt.Component;
import java.text.DateFormat;
import java.text.SimpleDateFormat;
import java.util.ArrayList;
import java.util.Calendar;
import java.util.GregorianCalendar;
import java.util.List;

import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.SwingUtilities;
import javax.swing.table.AbstractTableModel;
import javax.swing.table.DefaultTableCellRenderer;

class GregorianCalendarRenderer extends DefaultTableCellRenderer {
	private final DateFormat format = new SimpleDateFormat("d.M.yy");
	
	@Override
	public Component getTableCellRendererComponent(JTable table, Object value,
			boolean isSelected, boolean hasFocus, int row, int column) {
		final JLabel component = (JLabel) super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
		final GregorianCalendar calendar = (GregorianCalendar) value;
		final String text = format.format(calendar.getTime());
		component.setText(text);
		return component;
	}
}

class GregorianCalendarModel extends AbstractTableModel {
	private final List<GregorianCalendarEntity> data = new ArrayList<GregorianCalendarEntity>();
	
	@Override
	public int getRowCount() {
		return data.size();
	}

	@Override
	public int getColumnCount() {
		return 2;
	}

	@Override
	public String getColumnName(final int column) {
		return getColumnClass(column).getSimpleName();
	}
	
	@Override
	public Class<?> getColumnClass(int columnIndex) {
		switch (columnIndex) {
		case 0:
			return GregorianCalendar.class;
		case 1:
			return String.class;
		}
		return super.getColumnClass(columnIndex);
	}
	
	@Override
	public Object getValueAt(final int rowIndex, final int columnIndex) {
		final GregorianCalendarEntity entity = data.get(rowIndex);
		switch (columnIndex) {
		case 0:
			return entity.getCalendar();
		case 1:
			return entity.getString();
		default:
			throw new IndexOutOfBoundsException();
		}
	}
	
	public void addEntity(final GregorianCalendarEntity entity) {
		data.add(entity);
		final int rowCount = getRowCount();
		fireTableRowsInserted(rowCount, rowCount);
	}
}

class GregorianCalendarEntity {
	private final GregorianCalendar calendar;
	private final String string;
	
	public GregorianCalendarEntity(final GregorianCalendar calendar, final String string) {
		this.calendar = calendar;
		this.string = string;
	}

	public GregorianCalendar getCalendar() {
		return calendar;
	}

	public String getString() {
		return string;
	}
}

public class Main {
	public static void main(final String[] args) {
		SwingUtilities.invokeLater(new Runnable() {
			@Override
			public void run() {
				final JFrame frame = new JFrame();
				frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
				final GregorianCalendarModel model = new GregorianCalendarModel();
				for (int i = 0; i < 10; i++)
					model.addEntity(new GregorianCalendarEntity((GregorianCalendar) Calendar.getInstance(), String.valueOf(i)));
				final JTable table = new JTable(model);
				table.setDefaultRenderer(GregorianCalendar.class, new GregorianCalendarRenderer());
				frame.setContentPane(new JScrollPane(table));
				frame.pack();
				frame.setLocationRelativeTo(null);
				frame.setVisible(true);
			}
		});
	}
}

 

[SlaterB]

setValue() ist doch nicht ohne Grund eine öffentliche dokumentierte Methode,
die zu überschreiben statt des doppelten getTableCellRendererComponent(6 Argumente)-Gewurschtels erscheint mir überaus elegant,
falls es ausreicht, wie hier der Fall

noch besser den geänderten Wert an super.setValue() weiterreichen statt direkt selber setText() aufzurufen

 

[xehpuk]

Hast Recht, so ists besser:

class GregorianCalendarRenderer extends DefaultTableCellRenderer {
	private final DateFormat format = new SimpleDateFormat("d.M.yy");
	
	@Override
	protected void setValue(Object value) {
		final GregorianCalendar calendar = (GregorianCalendar) value;
		final String text = format.format(calendar.getTime());
		super.setValue(text);
	}
}

Hatte den Ansatz nicht weiter verfolgt, weil der TE meinte, die Methode werde nicht aufgerufen.

 

[Basti4231]

Wie SlaterB vermutet hat, lag es an vorhergehendem Code.

Der Fehler war, dass das Model des JTable zu spät gesetzt wurde.

Funktioniert jetzt...

Danke für eure anregenden Beiträge