Doppelter Aufruf von Table?

[Giftstachel]

Moinmoin, liebe Leidensgenossinen und Leidensgenossen

meine heutige preisfrage lautet:
ich möchte in meinem programm daten einer internetseite einlesen, parsen und verarbeiten. das klappt auch, also kein problem.
Das Prob. stellt sich jedoch in folgendem code.

...machwas...

public void actionPerformed (ActionEvent e)
    {
		
    	if (e.getSource()==bende)
    	{
    		System.exit(0);
    	}
    	if (e.getSource()==bread)
    	{
    		Table table = new Table();
    		table.Runtable();
    	}
    }

....machnochvielmehr....

nun wird meine table, die ihrerseits den parser aufruft dort einmal initialisiert, (Table table = new Table()

und somit auch das einlesen aus dem inet gestartet.
dann, damit die table auch angezeigt wird, muss ich table.Runtable(); aufrufen.

hier die table:

public class Table extends JPanel 
{
        public Table() {
        	super(new GridLayout(1,0));
        	Get get = new Get();
            
        	String[] columnNames = {"Nummer ",
        							"1",
        	                                                "2",
        	                                                "3",
        	                                                "4",
        	                                                "5",
        	                                                "6",
        	                                                "7",
        	                                                "Zeit/Datum"};

            try{
            	System.out.println("Table holt daten");
        	   	Object[][] data = get.holsStöckle();
                final JTable table = new JTable(data, columnNames);
                table.setAutoCreateRowSorter(true);
                table.setPreferredScrollableViewportSize(new Dimension(1000, 750));
                table.setFillsViewportHeight(true);
                JScrollPane scrollPane = new JScrollPane(table);
                add(scrollPane);
            }
            catch ( MalformedURLException e ) {
        		System.out.println( "MalformedURLException: " + e );
        	}
        	catch ( IOException e ) {
        		System.out.println( "IOException: " + e );
        	}         
        }

        private void createAndShowGUI() {
            JFrame frame = new JFrame("Tabelle");
            frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);//.EXIT_ON_CLOSE);

            Table newContentPane = new Table();
            newContentPane.setOpaque(true); //content panes must be opaque
            frame.setContentPane(newContentPane);

            frame.pack();
            frame.setVisible(true);
        }

        
        public void Runtable() {
            javax.swing.SwingUtilities.invokeLater(new Runnable() {
                public void run() {
                    createAndShowGUI();
                }
            });
        }

hat jemand spontan eine Idee, wie ich eines von den 2 mal einlesen der daten vehindern kann?

danke für die Hilfe.

 

[mhauert]

So wie ich das jetzt auf die schnelle sehe, ist der Aufruf in Zeile 13 überflüssig.

Es wird da ja nur ein neues Table- Objekt angelegt, aber damit wird ja nichts gemacht.

Da ja die Mehtode createAndShowGUI ihrerseits selbst ein Table- Objekt anlegt und dann im Frame anzeigt, ist nur der Aufruf in Zeile 40 nötig.

 

[Giftstachel]

leider nicht, da klasse 1 die class main ist, und Table die class Table.

wenn ich das Table table = new Table(); auskommentiere, kann table.Runtable(); nicht resolved werden.

 

[mhauert]

Ah, alles klar, hatte ich übersehen.

Wenn es nicht wichtig sein sollte das die Tabellenerstellung und das Anzeigen getrennt sind,
würde ich einfach die Methode createAndShowGUI mit in den Konstruktor von Table schreiben.

Eine weitere Möglichkeit wäre, in createAndShowGUI eben keine Neue Tabelle zu erstellen,
sonder die zu nehmen die halt schon existiert.

Die Klasse ist ja von JPanel abgeleitet und kann daher auch wie ein JPanel behandelt werden:

Probier in dem Fall mal Folgendes:

private void createAndShowGUI() {
            JFrame frame = new JFrame("Tabelle");
            frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);//.EXIT_ON_CLOSE);
            frame.add(this);     //fügt eben dieses Objekt in den Frame ein
            frame.pack();
            frame.setVisible(true);
        }

Kann das jetzt zwar nicht testen, aber es müsste funzen...

Dritte Möglichkeit wäre die beiden Dinge zu trennen, also ein Table Objekt anlegen,
und dann der Methode createAndShowGUI zu übergeben.
Dabei steht die Methode aber in einer Anderen Klasse:

private void createAndShowGUI(Table table) {
            JFrame frame = new JFrame("Tabelle");
            frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);//.EXIT_ON_CLOSE);
            frame.add(table);
            frame.pack();
            frame.setVisible(true);
        }

 

[Giftstachel]

super optimal. das haut sogar noch besser hin, als das was ich in zwischen versucht hatte.

dickes dankeschön