JTable: Speichern probleme

[ZentrumderMacht]

Hallo zusammen,

mein kleines Projekt erzielt kleine erfolge kurz vor abschluss bin ich auf ein Problem gestoßen.

Sobald ich eine Zelle in meiner Tabelle ändere und nicht aus dieser Zelle rausspringe und dann speichern drücke übergeht er diesen Datensatz einfach?! o0 warum?! gibt es ein Befehl das automatisch aus der Zelle rausspringt ist wichtig


Grüße AK

 

[SlaterB]

table.putClientProperty("terminateEditOnFocusLost", Boolean.TRUE);

 

[Zentrum]

also hab das jetzt so probiert leider verliert er den focus nicht ....

	public void speichern() {
		verantwortlicherTable.putClientProperty("terminateEditOnFocusLost", Boolean.TRUE); 
		verantwortlicherTable.speicherDaten();
	}

 

[SlaterB]

speichern() wird wahrscheinlich beim Button-Klick ausgeführt, dann war der Focus-Wechsel ja schon,
dann bringt eine Änderung der Konfiguration der JTable wenig,
außer dass zumindest beim nächsten Edititeren usw. das Problem nicht mehr auftritt


der Befehl muss besser ganz am Anfang zur Initialisierung der JTable, kurz hinter "new JTable()",
dann ist das Verhalten von Anfang an aktiv, also auch beim ersten Wechsel hin zum Speichern-Button

 

[zentrum]

kann es zu problemen führen wenn speichern() aus einem interface kommt?! Weil beim Löschen Button funktioniert es dort wird der Focus verloren aber bei Speichern(), das kommt eig aus einem Interface, also speichern() muss da sein....

 

[SlaterB]

siehe auch meine Antwort von 11:34,

nein, Interface hat damit nichts zu tun,
Löschen ist kaum vergleichbar mit Speichern, denn dort kommt es doch nicht auf den genauen Inhalt an oder?
möglich aber auch dass es eben erst mit dem zweiten Buttonklick klappt, wie beschrieben

 

[zentrum]

der speichern button liegt aber auch in einer JTOolbar vllt liegt es daran?!

 

[zentrum]

hab es jetzt direkt dahinter geschrieben den befehl

 

[Michael...]

Normalerweise sollte es mit putClientProperty... (einmalig gesetzt z.B. bei der Initialisierung und nicht vor jedem Speichern) funktionieren.
Zur Not kann man auch den Celleditor manuell beenden.

Verwendest Du vielleicht einen selbstgeschriebenen CellEditor?

 

[zentrum]

nope verzweifel langsam...
das sind meine Methoden, in meinen TableModel!abgeleitet von Abstracttablemodel...

public void speicherDaten() {
		
		boolean ueberpruefung = true;
		/*ArrayList<VerantwortlicherEntity> vergleichsliste = new ArrayList<VerantwortlicherEntity>();
		vergleichsliste = (ArrayList<VerantwortlicherEntity>) delegate
		.suchenallerVerantwortlicher();
		
		
		for (VerantwortlicherEntity entity : list){
			String hallo;
			hallo = entity.getName();
			
		}*/
		
		int i=1;
		for (VerantwortlicherEntity entity : list) {
			
			if (entity.getName().equals("") || entity.getName().equals(null)) {
				FrameUtilities
				.showWarningDialog(null, "Fehler",
						"Es existiert ein leeres Feld bei Datensatz "+i );
				ueberpruefung = false;
			}
			else{
				delegate.speichernVerantwortlicher(entity);
			}
			i++;
		}
		/*if (ueberpruefung) {
			delegate.speichernVielerVerantwortlicher(list);
			FrameUtilities.showMessageDialog(null, "Erfolgreich!",
					"Speichern Erfolgreich");
		}**/
	}

	/*
	 * deleteRow() 
	 * Löscht ein Datensatz und gibt eine Meldung wenn dieser
	 * Datensatz ein leeres Feld enthält. Beim erfolgreichem Löschen wird ebenso
	 * eine Meldung rausgeschickt.
	 */
	public void deleteRow(int position) {
		VerantwortlicherEntity entity = new VerantwortlicherEntity();
		entity = list.get(position);
		
		
		if (entity.getName().equals("") || entity.getName().equals(null)){
			FrameUtilities.showWarningDialog(null, "Fehler",
					"Es existiert ein leeres Feld");
		} else {
			delegate.loeschenVerantwortlicher(entity);
			list.remove(position);
			FrameUtilities.showMessageDialog(null, "Erfolgreich!",
					"Löschen Erfolgreich");
			fireTableDataChanged();
		}
		fireTableDataChanged();
		
	}

das ist meine Table

public class VerantwortlicherTable extends JTable {
	VerantwortlicherTableModel model = new VerantwortlicherTableModel();
	

	public VerantwortlicherTable() {
		setModel(model);
	}

	public Class<?> getColumnClass(int column) {
		if (column == 1) {
			return Boolean.class;
		}
		if (column == 2) {
			return Integer.class;

		}
		return super.getColumnClass(column);
	}

	public void deleteRow(int position) {
		model.deleteRow(position);
	}

	public void holeDaten() {
		model.holeDaten();
	}
	public void anlegenDatensatz(int position){
		if(position == 0){
			position = model.getRowCount();
		}
		model.anlegenDatensatz(position);
	}

	public void speicherDaten() {
		model.speicherDaten();
	}

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

	public void aktualisieren() {
		model.aktualisieren();
		
	}

Und in meinem Modul inzialisere ich meine Table und die holt sich das Model. Also setzt ich in mein Modul die Codezeile^^

 

[Michael...]

In dem Code ist nichts drin, was Aufschluss bgzl. Deines Problems gibt.

Noch ein Hinweis zu folgender Zeile:

if (entity.getName().equals("") || entity.getName().equals(null)) {

null ist kein Objekt ;-) auf null überprüft man mit

entity.getName()==null

und man überprüft sowas an erster Stelle der Bedingung. Wenn entity.getName() tatsächlich mal eine null Referenz sein sollte, würde bei Deinem Code eine NullPointerException fliegen.

 

[zentrum]

public class VerantwortlicherModul extends AbstractModul {

	/** Creates new form TestPanel */
	public VerantwortlicherModul() {

		initComponents();
		init();
		setName("Verantwortlicher");
		verantwortlicherTable.holeDaten();
		zaehleraktualisieren();
	}

	void init() {
		verantwortlicherTable.putClientProperty("terminateEditOnFocusLost", Boolean.TRUE); 
		verantwortlicherTable.setAutoCreateRowSorter(true);
		int anzahlSpalten = verantwortlicherTable.getRowCount() - 1;
		jLabel2.setText((new Integer(anzahlSpalten)).toString());
		verantwortlicherTable.getModel().addTableModelListener(
				new TableModelListener() {

					public void tableChanged(TableModelEvent e) {
						jtabletablechanged(e);
					}
				});

	}

	/**
	 * This method is called from within the constructor to initialize the form.
	 * WARNING: Do NOT modify this code. The content of this method is always
	 * regenerated by the Form Editor.
	 */
	// <editor-fold defaultstate="collapsed" desc="Generated Code">
	private void initComponents() {
		java.awt.GridBagConstraints gridBagConstraints;

		jLabel1 = new javax.swing.JLabel();
		jLabel2 = new javax.swing.JLabel();
		jPanel1 = new javax.swing.JPanel();
		jScrollPane2 = new javax.swing.JScrollPane();
		verantwortlicherTable = new VerantwortlicherTable();
		
		jButton1 = new javax.swing.JButton();
		jButton2 = new javax.swing.JButton();
		jComboBox1 = new javax.swing.JComboBox();
		_tfxMandant = new javax.swing.JLabel();

		setLayout(new java.awt.GridBagLayout());

		jLabel1.setText("Anzahl Datensätze: ");
		gridBagConstraints = new java.awt.GridBagConstraints();
		gridBagConstraints.gridx = 0;
		gridBagConstraints.gridy = 7;
		gridBagConstraints.gridwidth = 2;
		gridBagConstraints.fill = java.awt.GridBagConstraints.HORIZONTAL;
		gridBagConstraints.anchor = java.awt.GridBagConstraints.WEST;
		gridBagConstraints.insets = new java.awt.Insets(0, 4, 4, 0);
		add(jLabel1, gridBagConstraints);

		jLabel2.setText("0");
		gridBagConstraints = new java.awt.GridBagConstraints();
		gridBagConstraints.gridx = 2;
		gridBagConstraints.gridy = 7;
		gridBagConstraints.fill = java.awt.GridBagConstraints.HORIZONTAL;
		gridBagConstraints.anchor = java.awt.GridBagConstraints.WEST;
		gridBagConstraints.insets = new java.awt.Insets(0, 0, 4, 0);
		add(jLabel2, gridBagConstraints);

		jScrollPane2.setViewportView(verantwortlicherTable);

		jButton1.setText("Einfügen");
		jButton1.addActionListener(new java.awt.event.ActionListener() {
			public void actionPerformed(java.awt.event.ActionEvent evt) {
				jButton1ActionPerformed(evt);
			}
		});

		jButton2.setText("Löschen");
		jButton2.addActionListener(new java.awt.event.ActionListener() {
			public void actionPerformed(java.awt.event.ActionEvent evt) {
				jButton2ActionPerformed(evt);
			}
		});

		javax.swing.GroupLayout jPanel1Layout = new javax.swing.GroupLayout(
				jPanel1);
		jPanel1.setLayout(jPanel1Layout);
		jPanel1Layout
				.setHorizontalGroup(jPanel1Layout
						.createParallelGroup(
								javax.swing.GroupLayout.Alignment.LEADING)
						.addGroup(
								javax.swing.GroupLayout.Alignment.TRAILING,
								jPanel1Layout
										.createSequentialGroup()
										.addComponent(
												jScrollPane2,
												javax.swing.GroupLayout.DEFAULT_SIZE,
												639, Short.MAX_VALUE)
										.addPreferredGap(
												javax.swing.LayoutStyle.ComponentPlacement.RELATED)
										.addGroup(
												jPanel1Layout
														.createParallelGroup(
																javax.swing.GroupLayout.Alignment.TRAILING,
																false)
														.addComponent(
																jButton1,
																javax.swing.GroupLayout.DEFAULT_SIZE,
																javax.swing.GroupLayout.DEFAULT_SIZE,
																Short.MAX_VALUE)
														.addComponent(
																jButton2,
																javax.swing.GroupLayout.DEFAULT_SIZE,
																92,
																Short.MAX_VALUE))
										.addContainerGap()));
		jPanel1Layout
				.setVerticalGroup(jPanel1Layout
						.createParallelGroup(
								javax.swing.GroupLayout.Alignment.LEADING)
						.addGroup(
								jPanel1Layout
										.createSequentialGroup()
										.addContainerGap()
										.addGroup(
												jPanel1Layout
														.createParallelGroup(
																javax.swing.GroupLayout.Alignment.LEADING)
														.addComponent(
																jScrollPane2,
																javax.swing.GroupLayout.PREFERRED_SIZE,
																362,
																javax.swing.GroupLayout.PREFERRED_SIZE)
														.addGroup(
																jPanel1Layout
																		.createSequentialGroup()
																		.addComponent(
																				jButton1)
																		.addPreferredGap(
																				javax.swing.LayoutStyle.ComponentPlacement.RELATED)
																		.addComponent(
																				jButton2)))
										.addContainerGap(
												javax.swing.GroupLayout.DEFAULT_SIZE,
												Short.MAX_VALUE)));

		gridBagConstraints = new java.awt.GridBagConstraints();
		gridBagConstraints.gridx = 0;
		gridBagConstraints.gridy = 1;
		gridBagConstraints.gridwidth = 22;
		gridBagConstraints.gridheight = 6;
		gridBagConstraints.fill = java.awt.GridBagConstraints.BOTH;
		gridBagConstraints.weightx = 500.0;
		gridBagConstraints.weighty = 500.0;
		add(jPanel1, gridBagConstraints);

		jComboBox1.setModel(new javax.swing.DefaultComboBoxModel(
				new String[] { "Walbusch" }));
		gridBagConstraints = new java.awt.GridBagConstraints();
		gridBagConstraints.gridx = 1;
		gridBagConstraints.gridy = 0;
		gridBagConstraints.gridwidth = 2;
		gridBagConstraints.insets = new java.awt.Insets(3, 0, 3, 0);
		add(jComboBox1, gridBagConstraints);

		_tfxMandant.setText("Mandant");
		gridBagConstraints = new java.awt.GridBagConstraints();
		gridBagConstraints.gridx = 0;
		gridBagConstraints.gridy = 0;
		gridBagConstraints.anchor = java.awt.GridBagConstraints.WEST;
		gridBagConstraints.insets = new java.awt.Insets(7, 11, 8, 10);
		add(_tfxMandant, gridBagConstraints);
	}// </editor-fold>

	private void jtabletablechanged(TableModelEvent e) {

	}

	public void zaehleraktualisieren() {
		int i = verantwortlicherTable.zaehlenAnzahl();
		Integer meinInteger = new Integer(i);
		String s = meinInteger.toString();
		jLabel2.setText(s);
	}

	private void jButton1ActionPerformed(java.awt.event.ActionEvent evt) {
		int position = verantwortlicherTable.getSelectedRow();
		if (verantwortlicherTable.getSelectedRow() == -1) {
			verantwortlicherTable.anlegenDatensatz(0);
		} else if (verantwortlicherTable.getSelectedRow() != -1) {
			verantwortlicherTable.anlegenDatensatz(position);
		}
		zaehleraktualisieren();
	}

	public void setzeDaten() {
		verantwortlicherTable.holeDaten();
	}

	private void jButton2ActionPerformed(java.awt.event.ActionEvent evt) {
		int position;
		position = verantwortlicherTable.getSelectedRow();
		int i = FrameUtilities.showYesNoCancelDialog(null, null,
				"Wollen Sie wirklich den Verantwortlichen löschen");
		switch (i) {
		case FrameUtilities.JA_OPTION:
			verantwortlicherTable.deleteRow(position);
			break;
		case FrameUtilities.NEIN_OPTION:
			break;
		case FrameUtilities.ABBRECHEN_OPTION:
			break;
		default:
			FrameUtilities.showMessageDialog(null, "Fehler", "Fehler!");
		}

		zaehleraktualisieren();
	}

	// Variables declaration - do not modify

	private javax.swing.JLabel _tfxMandant;
	private javax.swing.JButton jButton1;
	private javax.swing.JButton jButton2;
	private javax.swing.JComboBox jComboBox1;
	private javax.swing.JLabel jLabel1;
	private javax.swing.JLabel jLabel2;
	private javax.swing.JPanel jPanel1;
	private javax.swing.JScrollPane jScrollPane2;
	private VerantwortlicherTable verantwortlicherTable;

	// End of variables declaration

	// *******************************************************************
	// Implementiert Modul
	// *******************************************************************
	public void open() {

	}

	public void aktualisieren() { 
		verantwortlicherTable.aktualisieren();
	}

	public void speichern() { 
		verantwortlicherTable.speicherDaten();
	}

	public void deactivate() {
		super.deactivate();

	}

	public void setModus() {

	}

Das ist mein Modul....

 

[SlaterB]

tja, viel Code, aber immer noch unbekannte Klassen, javax.swing.GroupLayout ist auch nicht jedermanns Sache (meine zumindest nicht)

hier zusammengekürzt zu einem lauffähigen Testprogramm, wenn du dort ein Feld gerade editierst und dann Speichern klickst,
ist die System.out.println-Ausgabe richtig

public class Test
{
    public static void main(String[] args)
    {
        new VerantwortlicherModul().setVisible(true);
    }
}

class VerantwortlicherModul
    extends JFrame
{

    public VerantwortlicherModul()
    {
        initComponents();
        init();
        verantwortlicherTable.holeDaten();
        setSize(700, 400);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    }

    void init()
    {
        verantwortlicherTable.putClientProperty("terminateEditOnFocusLost", Boolean.TRUE);
    }

    private void initComponents()
    {
        jScrollPane2 = new JScrollPane();
        verantwortlicherTable = new VerantwortlicherTable();

        jButton1 = new JButton();
        jScrollPane2.setViewportView(verantwortlicherTable);


        jButton1.setText("Speichern");
        jButton1.addActionListener(new ActionListener()
            {
                public void actionPerformed(ActionEvent evt)
                {
                    speichern();
                }
            });
        setLayout(new FlowLayout());
        add(jButton1);
        add(jScrollPane2);
    }

    private JButton jButton1;
    private JScrollPane jScrollPane2;
    private VerantwortlicherTable verantwortlicherTable;

    public void speichern()
    {
        verantwortlicherTable.speicherDaten();
    }
}


class VerantwortlicherTableModel
    extends DefaultTableModel
{

    public boolean isCellEditable(int row, int column)
    {
        return true;
    }
}


class VerantwortlicherTable
    extends JTable
{
    VerantwortlicherTableModel model = new VerantwortlicherTableModel();


    public VerantwortlicherTable()
    {
        setModel(model);
    }

    public void holeDaten()
    {
        model.setDataVector(new Object[][]
            {
                {"A1", "B1"},
                {"A2", "B2"}}, new Object[]
            {"Col A", "Col B"});
    }

    public void speicherDaten()
    {
        System.out.println("speichern");
        for (int i = 0; i < model.getRowCount(); i++)
        {
            for (int j = 0; j < model.getRowCount(); j++)
            {
                System.out.println(i + ", " + j + ": " + model.getValueAt(i, j));
            }
        }
    }
}