JTable probleme

[NoWitzness]

Hallo liebe Gemeinde!

Ich habe ein Problem und hoffe dass mir vielleicht wer aus der Patsche helfen kann?!
Ich habe eine JTable (THX Turtle!), bei der ich aus den markierten Rows, beim Klick auf den Button "Add", eine neue JTable erstellen will. In das Panel darunter... Leider komm ich nicht weiter. Die JTable wird aus einem Resultset erstellt.

Leider lassen sich die Boolean Checkboxen auch nicht anklicken. bei einem Doppelklick auf die Checkbox, steht zwar false da und ich true reinschreiben - ändert sich aber nicht. Hat auch hierzu wer Rat, wie ich die Checkboxen einfach anklicken kann?

Wäre für jeden Rat Dankbar :toll:

package optik;

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.Insets;
import java.awt.SystemColor;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.sql.ResultSet;
import java.sql.ResultSetMetaData;
import java.sql.SQLException;
import java.util.ArrayList;

import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JSplitPane;
import javax.swing.JTable;
import javax.swing.WindowConstants;
import javax.swing.table.AbstractTableModel;
import javax.swing.table.DefaultTableColumnModel;
import javax.swing.table.TableColumn;

import logik.DatenbankVerbindung;

public class Table1 implements ActionListener{

	private JFrame jFrame = new JFrame();

	private DatenbankVerbindung dbv = new DatenbankVerbindung();
	private JPanel aa = new JPanel(new BorderLayout());
	private JPanel bb = new JPanel(new BorderLayout());

	private JButton add = new JButton("Bearbeiten");
	private JButton delete = new JButton("Loeschen");

	private JSplitPane splitPane;
	private JTable tbl1, tbl2;
	private JScrollPane scrollPane1, scrollPane2;

	private GridBagLayout gbl = new GridBagLayout();
	private GridBagConstraints gbc = new GridBagConstraints();
	private JPanel buttonPanel = new JPanel(gbl);

	private String produkt = "";

	public Table1(int id, String produkt) {

		jFrame.setTitle(produkt);
		this.produkt = produkt;
		gbc.gridx = 0;
		gbc.gridy = 0;
		gbc.insets = new Insets (5,0,30,5);
		gbc.anchor = GridBagConstraints.EAST;
		add.setPreferredSize(new Dimension(130, 25));
		add.addActionListener(this);
		gbl.setConstraints(add, gbc);
		buttonPanel.add(add);

		gbc.gridx = 1;
		gbc.gridy = 0;
		gbc.insets = new Insets (5,5,30,0);
		gbc.anchor = GridBagConstraints.WEST;
		delete.setPreferredSize(new Dimension(130, 25));
		delete.addActionListener(this);
		gbl.setConstraints(delete, gbc);
		buttonPanel.add(delete);

		dbv.verbinden();
		ResultSet rs = dbv.produktAuflistung(produkt);
		TableModelFromRS tmfr = new TableModelFromRS(rs);
		DefaultTableColumnModel dtcm = tmfr.GetColumnModel();
		tbl1 = new JTable(tmfr, dtcm);
		tbl2 = new JTable();
		dtcm.getColumn(0).setCellRenderer(tbl1.getDefaultRenderer(Boolean.class));
		scrollPane1 = new JScrollPane(tbl1);
		scrollPane2 = new JScrollPane(tbl2);
		scrollPane1.getVerticalScrollBar().setUnitIncrement(15);
		scrollPane2.getVerticalScrollBar().setUnitIncrement(15);


		aa.add(BorderLayout.CENTER, tbl2);
		aa.setBorder(BorderFactory.createLineBorder(Color.BLACK, 2));
		aa.setPreferredSize(new Dimension(800, 325));

		bb.add(BorderLayout.SOUTH, buttonPanel);
		bb.add(BorderLayout.CENTER, scrollPane1);
		bb.setBorder(BorderFactory.createLineBorder(Color.BLACK, 2));
		bb.setPreferredSize(new Dimension(800, 375));

		splitPane = new JSplitPane(JSplitPane.VERTICAL_SPLIT, bb, aa);

		splitPane.setDividerLocation(0.70);
		splitPane.setDividerSize(0);
		splitPane.setResizeWeight(0.70);
		splitPane.setEnabled(false);

		jFrame.setSize(850, 700);
		jFrame.setLocationRelativeTo(null);
		jFrame.setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);
		jFrame.setBackground(SystemColor.window);
		jFrame.setContentPane(splitPane);
		jFrame.setVisible(true);
	}

	@Override
	public void actionPerformed(ActionEvent e) {
		dbv.verbinden();
		ResultSet rs = dbv.produktAuflistung(produkt);
		String[] spaltenNamen = null;
		int rsSpaltenAnzahl = 0;
		ResultSetMetaData metaData;
		int [] selectetRows = null;
		try {
			if (e.getSource() == add){

				metaData = rs.getMetaData();
				rsSpaltenAnzahl = metaData.getColumnCount(); 
				spaltenNamen = new String[rsSpaltenAnzahl];
				selectetRows = tbl1.getSelectedRows();
			}
		} catch (SQLException e1) {
			e1.printStackTrace();
		}	
		for(int i = 0; i < spaltenNamen.length; i++) {
			TableColumn tc = new TableColumn(i);
			tc.setHeaderValue(spaltenNamen[i]);
		}
		jFrame.repaint();
		
		
	}


	class TableModelFromRS extends AbstractTableModel {

		private static final long serialVersionUID = 1L;
		private String[] columName;
		private ArrayList<Object[]> zeilenList;
		TableModelFromRS(ResultSet rs) {

			ResultSetMetaData metaData;					// Metadaten - beinhalten alle infos zur Tabelle
			try {
				metaData = rs.getMetaData();
				// Anzahl der Zeilen im Resultset
				int rsSpaltenAnzahl = metaData.getColumnCount() + 1; 
				columName = new String[rsSpaltenAnzahl];

				columName[0] = "ADD";							//Erste Spalte ein Boolean
				for (int i = 1; i < rsSpaltenAnzahl; ++i) {   			
					columName[i] = metaData.getColumnLabel(i);		// Beinhaltet jetzt alle SpaltenNamen
					System.out.println(columName[i]);
				}
				// Alle Zeilen abspeichern
				zeilenList = new ArrayList<Object[]>();  			//neue ArrayList bestehend aus Objekt Arrays

				while (rs.next()) {										// solange spalten im Resultset sind ...
					Object[] zeilenArray = new Object[rsSpaltenAnzahl];	// ... wird jeweils ein neues Array vom Typ Objekt erstellt mit der Länge der Spalten im Resultset [rsSpaltenAnzahl]	
					// jeweils in die Erste Spalte wird ein Boolean wert angelegt.
					zeilenArray[0] = new Boolean(false);  // False, weil nicht markiert

					for (int i = 1; i < rsSpaltenAnzahl; i++) {   		// Jede Zeile einzeln einlesen
						zeilenArray[i] = rs.getObject(i);
					}
					zeilenList.add(zeilenArray);
				}
			} catch (SQLException e) {
				e.printStackTrace();
			}
		}

		@Override
		public int getRowCount() {
			return zeilenList.size();
		}

		@Override
		public int getColumnCount() {
			return columName.length;
		}

		@Override
		public Object getValueAt(int rowIndex, int columnIndex) {
			return zeilenList.get(rowIndex)[columnIndex];
		}

		public boolean isCellEditable(int row, int column) {
			if(column == 0)
				return true;
			else
				return false;
		}

		public DefaultTableColumnModel GetColumnModel() {
			DefaultTableColumnModel dtcm = new DefaultTableColumnModel();
			for(int i = 0; i < columName.length; i++) {
				TableColumn tc = new TableColumn(i);
				tc.setHeaderValue(columName[i]);
				dtcm.addColumn(tc);
			}
			return dtcm;
		}

	}
}

Lg

 

[turtle]

Du solltest ein lauffähiges Beispiel bauen, in dem das Problem sichtbar wird.

Unter anderem, haben wir nicht die DB, die du nutzt und zweitens ist es schlechter Stil, deine GUI so eng mit der DB zu verzahnen. Daher sollte dein Beispiel ohne "Nebenschauplätze" laufen können.

Was du schreibst, hört sich für mich nach Inplace-Editing an.

Du möchtest anscheinend in der JTable direkt Werte ändern und diese sollen im TabelModel geändert werden.

Wenn ich dich da richtig verstanden habe, sage ich das du dich genauer in eine JTable einarbeiten musst. Insbesondere wird dort unterschieden zwischen TableCellRenderer und einem TableCellEditor. Dieser Unterschied ist im Tutorial beschrieben.

Genauere Hilfe ist erst möglich, wenn wir Code haben, der auch laufen kann.

 

[NoWitzness]

Tag

Leider hab ich keinen Plan wie ich das Lauffähig bekommen soll? Das ist ja meine Frame

Das baut doch auf dem rs auf, somit kann ich das nicht umbaun :noe:
aber vielleicht hilft dir der Screenshot??

Das erste Problem ist sofort zu sehen - ich hab keine Hacken bei der Checkbox, sondern kann bei doppelklick darauf, nur etwas reinschreiben... wird aber, auch wenn ich true reinschreibe, nicht gespeichert.

Und das Zweite Anliegen wäre, dass im unteren Teil der SplitPane (auf dem Bild weiss und südlich) eine neue Tabelle bekomme, die aus den makierten Zeilen im nördlichen Bereich erstellt wird. Da scheitere ich aber weil ich das erste Anliegen schon nicht lösen kann. Wo ist der Fehler, damit ich die Checkboxen nicht "anhacken", makieren, auswählen oder sonst was kann?
Weiss mich leider nicht besser auszudrücken ...

Aber dennoch danke für deine Bemühungen.

PS.: den Link hab ich schon durchgemacht, trotzdem wird mein Fehler nicht durchsichtiger :noe:

 

[turtle]

Die JTable fragt mit getValueAt nach dem anzuzeigenden Wert.

Hier hast du ja schon über getColumnClass der JTable gesagt, das eine Spalte Boolean-Werte darstellt, worauf das als Checkbox "gemalt" wird.

Über isCellEditable fragt die JTable an, ob eine bestimme Zelle editiert werden darf. In meinem beispiel gebe ich mal zum Test generell true zurück.

Änderst du einen Wert gibt JTable über setValueAt den neuen (GUI)-Wert an das Model, damit dieses seine Daten updaten kann.

Alles zusammen habe ich wieder in ein kleines Beispiel gegossen. Hier siehst du übrigens auch, wie genau zwischen Memorydaten/JDBC-Daten und der GUI unterschieden werden kann. In deinem Beispiel hantierst du ständig mit mehreren Strings. Es gibt aber kein Objekt, welches ALLE Daten einer Zeile zusammenfasst (Klasse Daten in meinem Beispiel)

package table;

import java.awt.BorderLayout;
import java.awt.Component;
import java.text.NumberFormat;
import java.text.ParseException;
import java.text.SimpleDateFormat;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Date;
import java.util.List;
import java.util.Locale;

import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.table.AbstractTableModel;
import javax.swing.table.TableCellRenderer;

@SuppressWarnings("serial")
public class Table extends JPanel {
    private MeinTableModel tableModel;
    private List<Daten> listDaten;
    private JTable table;

    public Table() {
	setLayout(new BorderLayout());
	// @formatter:off
	Daten[] memoryDaten = new Daten[] { 
		new Daten("Turtle", 20, false,"1.1.1993", 5431.123),
		new Daten("Andreas", 10, false,"10.5.2003",60),
		new Daten("Bärbel", 30, true,"20.02.1986", 998811),
		new Daten("Christian", 40, false,"11.11.1980", 123),
		new Daten("Doris", 50, false,"22.02.1970", 1234),
		new Daten("Emil", 66, true,"7.2.1966", 12345),
		new Daten("Franziska", 70, true,"12.9.1955",123456),
		new Daten("Gerhard", 30, false,"16.08.1983",1234567),
		new Daten("Heike", 33, false,"17.08.1986", 12345678),
		new Daten("Ingo", 22, false,"30.12.1992", 123456789),
	};
	// @formatter:on
	listDaten = Arrays.asList(memoryDaten);
	tableModel = new MeinTableModel(listDaten);

	table = new JTable(tableModel);
	JScrollPane scrollPane = new JScrollPane(table);
	add(scrollPane, BorderLayout.CENTER);
    }

    public static void main(String[] args) {
	JFrame frame = new JFrame("Table Test");
	frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	frame.add(new Table());
	frame.pack();
	frame.setVisible(true);
    }
}

@SuppressWarnings("serial")
class MeinTableModel extends AbstractTableModel {
    private List<Daten> listDaten;
    private List<String> listColumns;

    public MeinTableModel(List<Daten> daten) {
	this.listDaten = new ArrayList<Daten>();
	for (Daten datenSatz : daten) {
	    listDaten.add(datenSatz);
	}
	listColumns = new ArrayList<String>();
	listColumns.add("Name");
	listColumns.add("Rentner");
	listColumns.add("Geburtstag");
	listColumns.add("Alter");
	listColumns.add("Gehalt");

    }

    public void addiereNeueDaten(List<Daten> neueDaten) {
	for (Daten daten : neueDaten) {
	    listDaten.add(daten);
	}
	fireTableStructureChanged();
    }

    @Override
    public int getRowCount() {
	return listDaten.size();
    }

    public Class<?> getColumnClass(int columnIndex) {
	switch (columnIndex) {
	case 0: // Name
	    return String.class;
	case 1: // Rentner
	    return Boolean.class;
	case 2: // Geburtstag
	    return Date.class;
	case 3:// Alter
	    return Integer.class;
	case 4:// Gehalt
	    return Double.class;
	}
	return null;
    }

    @Override
    public int getColumnCount() {
	return listColumns.size();
    }

    public String getColumnName(int column) {
	return listColumns.get(column);
    }

    @Override
    public boolean isCellEditable(int rowIndex, int columnIndex) {
	return true;
    }

    @Override
    public void setValueAt(Object aValue, int rowIndex, int columnIndex) {
	if (aValue != null) {
	    Daten daten = listDaten.get(rowIndex);
	    switch (columnIndex) {
	    case 0:
		daten.setName((String) aValue);
		break;
	    case 1:
		daten.setRentner((Boolean) aValue);
		break;
	    case 2:
		daten.setGeburtstag((Date) aValue);
		break;
	    case 3:
		daten.setAlter((Integer) aValue);
		break;
	    case 4:
		daten.setGehalt((double) aValue);
		break;
	    }
	}
    }

    @Override
    public Object getValueAt(int rowIndex, int columnIndex) {
	Daten daten = listDaten.get(rowIndex);
	if (daten != null) {
	    switch (columnIndex) {
	    case 0:
		return daten.getName();
	    case 1:
		return daten.isRentner();
	    case 2:
		return daten.getGeburtstag();
	    case 3:
		return daten.getAlter();
	    case 4:
		return daten.getGehalt();
	    }
	}
	return null;
    }
}

class Daten {
    private String name;
    private Boolean rentner;
    private Date geburtstag;
    private Integer alter;
    private double gehalt;
    private SimpleDateFormat sdf = new SimpleDateFormat("dd.MM.yyyy");

    public Daten(String name, Integer alter, Boolean rentner, String geb, double gehalt) {
	try {
	    this.name = name;
	    this.alter = alter;
	    this.rentner = rentner;
	    this.geburtstag = sdf.parse(geb);
	    this.gehalt = gehalt;
	} catch (ParseException e) {
	    e.printStackTrace();
	}
    }

    public String getName() {
	return name;
    }

    public void setName(String name) {
	this.name = name;
    }

    public Boolean isRentner() {
	return rentner;
    }

    public void setRentner(Boolean rentner) {
	this.rentner = rentner;
    }

    public Date getGeburtstag() {
	return geburtstag;
    }

    public void setGeburtstag(Date geburtstag) {
	this.geburtstag = geburtstag;
    }

    public int getAlter() {
	return alter;
    }

    public void setAlter(Integer alter) {
	this.alter = alter;
    }

    public double getGehalt() {
	return gehalt;
    }

    public void setGehalt(double gehalt) {
	this.gehalt = gehalt;
    }

}

@SuppressWarnings("serial")
class CurrencyRenderer extends JLabel implements TableCellRenderer {

    public CurrencyRenderer() {
	super();
    }

    public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) {

	if (isSelected) {
	    setForeground(table.getSelectionForeground());
	    super.setBackground(table.getSelectionBackground());
	} else {
	    setForeground(table.getForeground());
	    setBackground(table.getBackground());
	}
	Locale currentLocale = Locale.GERMAN;
	NumberFormat numberFormatter = NumberFormat.getNumberInstance(currentLocale);
	Double quantity = (Double) value;
	setText(numberFormatter.format(quantity));
	setHorizontalAlignment(RIGHT);
	return this;
    }
}

 

[NoWitzness]

Die JTable fragt mit getValueAt nach dem anzuzeigenden Wert.

Hier hast du ja schon über getColumnClass der JTable gesagt, das eine Spalte Boolean-Werte darstellt, worauf das als Checkbox "gemalt" wird.

Über isCellEditable fragt die JTable an, ob eine bestimme Zelle editiert werden darf. In meinem beispiel gebe ich mal zum Test generell true zurück.

Änderst du einen Wert gibt JTable über setValueAt den neuen (GUI)-Wert an das Model, damit dieses seine Daten updaten kann.

Alles zusammen habe ich wieder in ein kleines Beispiel gegossen. Hier siehst du übrigens auch, wie genau zwischen Memorydaten/JDBC-Daten und der GUI unterschieden werden kann. In deinem Beispiel hantierst du ständig mit mehreren Strings. Es gibt aber kein Objekt, welches ALLE Daten einer Zeile zusammenfasst (Klasse Daten in meinem Beispiel)

package table;

.....

vielen Dank, aber leider weiss ich nicht wie ich daraus auf mein Problem schliessen soll, dass die Checkboxen nicht zum Anklicken sind! Die checkbox IST SCHON EDITIERBAR, aber ich kann keinen Hacken, kein Häckchen, 'son blöden Strich reinmachen!!!!!

Ich will ja meinen Code nicht gänzlich neu schreiben sondern ihn so lassen und ggf. die Checkboxen zum Anklicken haben. Ist es möglich dass du mir dabei hilfst?

Ansonsten dennoch danke!

 

[turtle]

Ich möchte dir ja nicht zu nahe treten, aber etwas mehr Eigeninitiative ist hier schon vonnöten.

Dein Code sind 210 Zeilen Code und mein Beispiel sind 252 Zeilen. Da solltest du in der Lage sein, Unterschiede zu finden und beim Debuggen zu erkennen, warum es bei dir nicht läuft. Das ist kein gänzliches Neuschreiben, sondern normales Ändern eines Programms. Wenn du das mal sagst, wenn mehrere Tausend Zeilen neu zu schreiben wären, aber bei ca. 250:noe:

Da kann ich wenig bis gar nicht helfen, insbesondere auch deswegen, weil du es nicht schaffst, ein eigenständig laufendes Programm zu schreiben. Was ich übrigens komplett nicht verstehe, da du ja ein Beispiel hast, welches als Vorlage dienen kann.

Und meine Hinweis zu setValueAt hast du wohl überhaupt noch nicht richtig nachvollzogen. Denn dann hättest du gesehen, das in meinem Beispiel kein Doppelklick auf der Checkbox-Spalte nötig ist.

Bei einem Einfach-Klick, wird bei isCellEditable nachgragt, ob die Zelle editiert werden darf-->Antwort ja/true. Darauf wir, der Boolesche-Wert geändert und setValueAt aufgerufen und der neue Wert ins Model geschrieben.

Also solltest du doch nachvollziehen können, ob bei dir es genau so abläuft, oder?