Tabelle reagiert ungewollt auf Mausklick

[Lupus93]

Hallo wertes Forum

Ich plage mich schon seit einer Weile mit einem Problem rum und hoffe,
hier Hilfe zu finden.
Ich habe einen Testfall nachgeschrieben, in dem das Problem identisch wie im Hauptprogramm auftritt,
weshalb ich diesen hier poste. Der Unterschied besteht dann lediglich in den Daten, die im Hauptprogramm
aus einer Datenbank kommen.
Die Fehlerbeschreibung:
Man wählt A aus und bestätigt den Button, es wird die Tabelle a angezeigt.
Man wählt darauf hin B aus und bestätigt wieder den Button, Tabelle b wird angezeigt.
Klickt man jetzt in die Tabelle (Daten oder Head), wird auf einmal wieder Tabelle a angezeigt?!

Vielen Dank schonmal für eure Hilfe!

Code:

package fehler;

import javax.swing.*;
import javax.swing.table.DefaultTableModel;

import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

public class Table_Combo {

	static void addComponent(Container cont, GridBagLayout gbl, Component c,
			int x, int y, int width, int height, double weightx,
			double weighty, Insets insets)

	{

		GridBagConstraints gbc = new GridBagConstraints();
		gbc.fill = GridBagConstraints.BOTH;
		gbc.gridx = x;
		gbc.gridy = y;
		gbc.gridwidth = width;
		gbc.gridheight = height;
		gbc.weightx = weightx;
		gbc.weighty = weighty;
		gbc.insets = insets; // oben,links,unten,rechts
		gbl.setConstraints(c, gbc);
		cont.add(c);

	}

	public static void main(String[] args) {
		JFrame testframe = new JFrame();
		testframe.setSize(300, 300);
		testframe.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		final Container c = testframe.getContentPane();
		final GridBagLayout gbl = new GridBagLayout();
		c.setLayout(gbl);

		final JComboBox test_box = new JComboBox();
		String[] inhalt = { "a", "b" };
		for (String s : inhalt)
			test_box.addItem(s);
		final JButton test_button = new JButton("Anzeigen");

		addComponent(c, gbl, test_box, 0, 0, 1, 1, 0, 0, new Insets(0, 0, 0, 0));
		addComponent(c, gbl, test_button, 1, 0, 1, 1, 0, 0, new Insets(0, 0, 0,
				0));

		ActionListener al = new ActionListener() {

			@Override
			public void actionPerformed(ActionEvent e) {
				if (e.getSource().equals(test_button)) {
					int index = test_box.getSelectedIndex();

					// hier beginnt das Problem:

					DefaultTableModel model = new DefaultTableModel();
					JTable table = new JTable(model);
					JScrollPane pane = new JScrollPane(table);
					addComponent(c, gbl, pane, 0, 1, 2, 1, 1.0, 1.0,
							new Insets(0, 0, 0, 0));

					if (index == 0) {
						String[] head = { "Kopf_a", "Kopf_a" };
						String[][] inhalt = { { "Inhalt_a", "Inhalt_a" },
								{ "Inhalt_a", "Inhalt_a" } };

						model.setDataVector(inhalt, head);
						model.fireTableDataChanged();

						// diesen Weg bin ich vorher gegangen, der jetzige
						// sollte besser sein:
						
						// table = new JTable(inhalt, head);
						// pane= new JScrollPane(table);
						c.validate();

					}

					if (index == 1) {
						String[] head2 = { "Kopf_b", "Kopf_b" };
						String[][] inhalt2 = { { "Inhalt_b", "Inhalt_b" },
								{ "Inhalt_b", "Inhalt_b" } };

						model.setDataVector(inhalt2, head2);
						model.fireTableDataChanged();
						c.validate();

					}

				}

			}
		};
		test_button.addActionListener(al);
		testframe.setVisible(true);
	}

}

 

[TKausL]

Du addest hier Tabellen ohne zu berücksichtigen, dass die alte auch noch existiert. Somit hast du beim 2. Klick 2 Tabellen übereinander und es können solche Probleme auftreten. Entweder löschst du die alte jedesmal oder du belässt es bei einer und befüllst sie nur mit neuen Daten.

 

[ssoul26]

Du addest hier Tabellen ohne zu berücksichtigen, dass die alte auch noch existiert. Somit hast du beim 2. Klick 2 Tabellen übereinander und es können solche Probleme auftreten. Entweder löschst du die alte jedesmal oder du belässt es bei einer und befüllst sie nur mit neuen Daten.

Da hat TKausL recht! Leg vorher deine Tabelle mit Model an und verändere nur das Modell, anstelle jedesmal alles neu zu instanziieren.

package fehler;
import javax.swing.*;
import javax.swing.table.DefaultTableModel;

import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
public class s {
   private static DefaultTableModel model;

   static void addComponent(Container cont, GridBagLayout gbl, Component c, int x, int y,
            int width, int height, double weightx, double weighty, Insets insets) {
      GridBagConstraints gbc = new GridBagConstraints();
      gbc.fill = GridBagConstraints.BOTH;
      gbc.gridx = x;
      gbc.gridy = y;
      gbc.gridwidth = width;
      gbc.gridheight = height;
      gbc.weightx = weightx;
      gbc.weighty = weighty;
      gbc.insets = insets; // oben,links,unten,rechts
      gbl.setConstraints(c, gbc);
      cont.add(c);
   }

   public static void main(String[] args) {
        JFrame testframe = new JFrame();
        testframe.setSize(300, 300);
        testframe.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        final Container c = testframe.getContentPane();
        final GridBagLayout gbl = new GridBagLayout();
        c.setLayout(gbl);
 
        final JComboBox test_box = new JComboBox();
        String[] inhalt = { "a", "b" };
        for (String s : inhalt)
            test_box.addItem(s);
        final JButton test_button = new JButton("Anzeigen");
 
        addComponent(c, gbl, test_box, 0, 0, 1, 1, 0, 0, new Insets(0, 0, 0, 0));
        addComponent(c, gbl, test_button, 1, 0, 1, 1, 0, 0, new Insets(0, 0, 0,
                0));
 
        
        model= new DefaultTableModel();
        JTable table = new JTable(model);
        JScrollPane pane = new JScrollPane(table);
        addComponent(c, gbl, pane, 0, 1, 2, 1, 1.0, 1.0,
                new Insets(0, 0, 0, 0));
        ActionListener al = new ActionListener() {
 
            @Override
            public void actionPerformed(ActionEvent e) {
                if (e.getSource().equals(test_button)) {
                    int index = test_box.getSelectedIndex();
 
                    // hier beginnt das Problem:

 
                    if (index == 0) {
                        String[] head = { "Kopf_a", "Kopf_a" };
                        String[][] inhalt = { { "Inhalt_a", "Inhalt_a" },
                                { "Inhalt_a", "Inhalt_a" } };
 
                        model.setDataVector(inhalt, head);
                     
                    }
 
                    if (index == 1) {
                        String[] head2 = { "Kopf_b", "Kopf_b" };
                        String[][] inhalt2 = { { "Inhalt_b", "Inhalt_b" },
                                { "Inhalt_b", "Inhalt_b" } };
 
                        model.setDataVector(inhalt2, head2);
                   
                    }
                    c.validate();
 
                }
 
            }
        };
        test_button.addActionListener(al);
        testframe.setVisible(true);
    }
}

 

[Lupus93]

Hallo

Danke für den Wink mit dem Zaunpfahl, manchmal sieht man den Wald vor lauter Bäumen nicht.
Wie ich auf die Idee gekommen bin, das in die Actionperformed zu schreiben...keine Ahnung.
Das hier muss über den ActionListener, und alles läuft:

final DefaultTableModel model = new DefaultTableModel();
		final JTable table = new JTable(model);
		final JScrollPane pane = new JScrollPane(table);
		addComponent(c, gbl, pane, 0, 1, 2, 1, 1.0, 1.0,
				new Insets(0, 0, 0, 0));

Danke!

Edit: Genau diese Lösung, nur eben mit final.

 

[Lupus93]

Hallo

Also, habe alles im Hauptprogramm geändert und klappt einwandfrei, vielen Dank nochmal!
Jetzt habe ich ein anderes Problem, das zwar unabhängig von diesem ist, aber da hier mein Code schon steht,
Müll ich das Forum nicht damit zu.
Außerdem geht es mir dabei ums Verständnis, habe da schon öfter Probleme gehabt, und wie man sieht,
fang ich mit Layouts etc. grade erst an.

Problem:
Bei Klick auf Button a entstehen 2 Tabellen nebeneinander.
B brauch wieder nur eine Tabelle, also soll die zweite verschwinden.
Der Code dafür:

boolean tabelle2_exist; // nur zur Information
if(tabelle2_exist){
c.remove(pane2);
c.validate();
}

Jetzt sieht die Sache danach recht kryptisch aus: Die richtige Tabelle liegt über der alten, diese ist noch sichtbar.
Minimiert und Maximiert man das Fenster wieder, ist alles gut.
Also gehts um das Neuladen des Fensters, wie mache ich das mit richtig sauberem Code (die Workarounds bis jetzt wollt ihr garnicht wissen)?

Danke!

 

[ssoul26]

Anstatt die gleiche Tabelle ständig zu removen und neu zu adden, versuch es doch mal mit .setVisible(boolean b) Methode. Eventuell musst du die Tableheader extra ansprechen -> .getTableHeader.setVisible(boolean b).

 

[Lupus93]

Hallo

Habe ich schon versucht, nur anscheinend nicht richtig.
Danke!

if(tabelle2_exist){
// table2.setVisible(false);   Nur die Tabelle, Head steht noch
// table2.getTableHeader().setVisible(false);   Head weg
pane2.setVisible(false);  Das ganze Pane weg, damit auch der Hintergrund weg ist
c.validate();
}

Klappt tadellos...

Danke und einen schönen Tag noch!