Problem mit Listener und Tab-Inhalt auslesen!

[Spyder]

Hallo,

habe ein Programm wo mehrere Tabs dynamisch erstellt werden und
auf diesen befinden sich dann wieder dynamisch erstellte JComboBoxs.
Auf Butto-Druck sollen dann die Inhalte des jeweiligen Tabs in eine Datenbank
geschrieben werden. Mein Problem liegt darin, dass nur der erste Tab
funktionier und bei den anderen eine Nullpointerexception ausgeworfen wird.

Ich denke das Problem liegt darin das der Listener nicht auf die ComboBoxen der
richtigen Instanz vom Tab zugreift, sondern nur auf die zu letzt erstellte Instanz.
Meine Frage ist nun, wie ich auf die richtige zugreifen kann.

Die Tabs befinden sich im Array group[] und die ComboBoxen in den Array heim[] und gast[].

public class dialog_setpreround extends JDialog
{
	JPanel mainpanel = new JPanel();
	JPanel[] group = new JPanel[10];
	private DatabaseMetaData mcoMetaData = null;
	
	//Hilfsvariablen
	int i=1, j=0;
	int stunden;
	int minuten;
	
	public dialog_setpreround(Frame owner){
		super(owner, "Vorrunde generieren", true);
		this.getContentPane().setLayout(new BorderLayout());
	    this.setLocation(100,100);
	    this.setSize(600,600);
	    generatetabs();
	    this.setVisible(true);
	}
	
	/*
	 * gerneratetabs() liest Tabellennamen (nur Gruppe*)
	 * aus der DB und erstellt daraus Tabs, speichert diese in group[]
	 * ->Ruft setTabContent auf
	 */
	public void generatetabs(){........}
	
	/*
	 * setTabContent setze alle Oberflächenelemente der Tabs
	 * ->Ruft getColumn(String table, String column) auf
	 */
	private void setTabContent(final int tab){
		int pause = Integer.parseInt(getColumn("System", "Pause")[0]);
		int dauer = Integer.parseInt(getColumn("System", "Spielzeit")[0]);
		final int length = Integer.parseInt(getColumn("System", "Vorrundenpaarungen")[0]);
		String startzeit = getColumn("System","Startzeit")[0];
		
		group[i].setLayout(null);
		
		final JComboBox[] heim = new JComboBox[50];
		final JComboBox[] gast = new JComboBox[50];
		final JComboBox[] feld = new JComboBox[50];
		final JLabel[] time = new JLabel[50];
		
		JLabel timehl = new JLabel("Startzeit:");
		timehl.setBounds(new Rectangle(120, 50, 120, 20));
		JLabel feldhl = new JLabel("Spielfeld:");
		feldhl.setBounds(new Rectangle(50, 50, 100, 20));
		JButton end = new JButton("Fertig");
		end.setBounds(new Rectangle(480, 450, 70, 20));
		
		int anzahl=Integer.parseInt(getColumn("System", "Spielfelder")[0]);
		int t;
		for (t=1; t<=length; t++){
			minuten=minuten+dauer+pause;
			if(minuten>=60){
				minuten=minuten-60;
				stunden=stunden+1;
			}
			
			String felder[] = new String[anzahl];
			for (int p=0; p<anzahl; p++){
				felder[p]=""+(p+1);
			}
			JLabel time1 = new JLabel("Uhr");
			time1.setBounds(new Rectangle(155, t*30+50, 120, 20));
			JLabel gegen = new JLabel("-");
			gegen.setBounds(new Rectangle(355, t*30+50, 120, 20));
			
			feld[j] = new JComboBox(felder);
			feld[j].setBounds(new Rectangle(50, t*30+50, 40, 20));
			feld[j].setSelectedIndex(i-1);
			time[j] = new JLabel(""+stunden+":"+minuten);
			time[j].setBounds(new Rectangle(120, t*30+50, 120, 20));
			heim[j] = new JComboBox(getColumn("Gruppe"+i, "Mannschaft"));
			heim[j].setBounds(new Rectangle(210, t*30+50, 120, 20));
			gast[j] = new JComboBox(getColumn("Gruppe"+i, "Mannschaft"));
			gast[j].setBounds(new Rectangle(380, t*30+50, 120, 20));
			
			group[i].add(heim[j]);
			group[i].add(gast[j]);
			group[i].add(time[j]);
			group[i].add(feld[j]);
			group[i].add(time1);
			group[i].add(gegen);
			j++;
		}
		
		JButton insert = new JButton("Eintragen");
		insert.setBounds(new Rectangle(400, t*30+100, 100, 20));
		
		group[i].add(timehl);
		group[i].add(feldhl);
		group[i].add(insert);
		group[i].add(end);
		

//Denke HIER ist das PROBLEM
		insert.addActionListener(new ActionListener(){
			public void actionPerformed(ActionEvent e) {
				String heim1, gast1, start;

				for (int p=0; p<length; p++){
					heim1 = (String) group[i].heim[p].getSelectedItem();
					gast1 = (String) gast[p].getSelectedItem();
					start = time[p].getText();
					int field = Integer.parseInt((String) feld[p].getSelectedItem());
					insertMatch(tab+1, field, start, heim1, gast1);
					System.out.println(heim1+gast1+start+field);
					System.out.println(tab);
				}
			}
		});
		
	}
	
	/*
	 * getTeam(String table, String column) gibt
	 * die Spalte Column aus der Tabelle table aus
	 * und schreibt alle Einträge in ein Array
	 */
	private String[] getColumn(String table, String column){........}

	/*
	 * instertMatch schreibt Paarung in DB
	 */
	private void insertMatch(int group, int field, String time, String heim, String gast){.....}
}

 

[Murray]

Ich sehe nicht, wo die Variable i variiert wird; teilweise sieht es so aus, als sollte i eine Art Schleifenindex sein, andererseits ist es aber auch ein "normales" Class-Member :bahnhof:
Du schreibst, dass generatetabs setTabContent aufruft; die Stelle würde mich mal interessieren: wird setTabContent dort einmal oder aber in einer Schleife aufgerufen?

 

[Spyder]

Hi, und danke für deine Antwort.
Bin noch Anfänger im Programmieren und hab bestimmt irgeneinen dummen Denkfehler
drin, find den aber nicht...
Achja die Methode generatetabs() wird ganz normal einmal im Konstruktor aufgerufen.
Hier der Code wo die Methode aufgerufen wird:

public void generatetabs(){
		JTabbedPane pane = new JTabbedPane();
	    try 
	    {
            Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
            Connection con = java.sql.DriverManager.getConnection("jdbc:odbc:turnier");
            
            mcoMetaData = con.getMetaData();
            ResultSet rs = mcoMetaData.getTables(null,null,null,null);
            
            while(rs.next())
            {
                String name = rs.getString("TABLE_NAME");
                
                if (name.equals("MSysRelationships")){
                }
                else if (name.equals("MSysAccessObjects")){
                }
                else if (name.equals("MSysACEs")){
                }
                else if (name.equals("MSysObjects")){
                }
                else if (name.equals("MSysQueries")){
                }
                else if (name.startsWith("Paarungen")){
                }
                else if (name.equals("System")){
                }
                else {
                	group[i] = new JPanel();
                	setTabContent(i);
                	pane.addTab(name, group[i]);
        		    i++;
                }
            }
            con.close();
	    }
	    catch (Exception e) {System.out.println("generateTabs Fehler: " + e.getMessage());}
	    
	    setContentPane(pane);
	}

 

[Murray]

Das Problem besteht darin, dass i am Ende von gerateTabs ja auf dem Endwert steht. Alle ActionListener verwenden also den gleichen Wert für i.

Lösen kannst Du das, indem Du den ActionListeners bei der Erzeugung die jeweiligen Indices mitgibst. Du könntest z.B. eine eigene Klasse für die ActionListener definieren:

public class MyActionListener implements ActionListener{

private int idx;

public MyActionListener( int idx) {
  this.idx = idx;
}

public void actionPerformed(ActionEvent e) {
            String heim1, gast1, start;

            for (int p=0; p<length; p++){
               heim1 = (String) group[ idx].heim[p].getSelectedItem();
               gast1 = (String) gast[p].getSelectedItem();
               start = time[p].getText();
               int field = Integer.parseInt((String) feld[p].getSelectedItem());
               insertMatch(tab+1, field, start, heim1, gast1);
               System.out.println(heim1+gast1+start+field);
               System.out.println(tab);
            }
         } 
}

In setTabContent werden die Listener dann so eingehängt:

     insert.addActionListener(new MyActionListener( i));

An Deiner Stelle würde ich die globale Variable i eliminieren und daraus eine lokale Hilfsvariable in generatetabs() machen. setTabContent könnte dann den jeweiligen Index als Parameter übergeben bekommen.

 

[Spyder]

Hi,
erstmal vielen Dank für deine Hilfe.
Hab den Listener jetzt in einer extra Klasse und die Variable i nur
noch lokal in der generatetabs methode.

Der Listener funktioniert auch soweit, das heißt er horcht am richtigen Tab, habe nur noch nicht ganz verstanden wie ich das in dem Listener implementieren muss damit der Inhalt aus den richtigen JComboBoxs geholt wird. Bekomme immer eine NullpointerException.

public void generatetabs(){
		JTabbedPane pane = new JTabbedPane();
		
	    try 
	    {
            int i=1;
            while(rs.next())
            {
               .........
                else {
                	group[i] = new JPanel();
                	setTabContent(i);
                	pane.addTab(name, group[i]);
        		    i++;
                }
            }

	    setContentPane(pane);
	}

private void setTabContent(int tab){
.........
insert.addActionListener(new preroundlistener(tab));
}

Wie muss jetzt der Zugriff auf die verschiedenen ComboBoxes aussehen? Hab da wohl irgendwie ein Blackout oder noch nicht das richtige Verständnis.

public class preroundlistener implements ActionListener{ 

	private int idx;

	public preroundlistener(int idx) {
	  this.idx = idx;
	}

	public void actionPerformed(ActionEvent e) {
	            String heim1, gast1, start;
	            int length = Integer.parseInt(getColumn("System", "Vorrundenpaarungen")[0]);
	            for (int p=0; p<length; p++){
	               //heim1 = (String) heim[p].getSelectedItem();
	               //gast1 = (String) gast[p].getSelectedItem();
	               //start = bla.time[p].getText();
	               //int field = Integer.parseInt((String) bla.feld[p].getSelectedItem());
	               //bla.insertMatch(idx, field, start, heim1, gast1);
	               //System.out.println(heim1+gast1+start+field);
	               System.out.println(idx);
	            }
	         } 
	
}

So funktioniert es noch nicht, hab schon etwas rumprobiert mit anderen Variablen übergeben und so, aber bekomme irgendwie nicht raus wie ich auf die richtigen Instanzen von heim[], gast[], etc zugreife.

 

[Murray]

Sorry, ich hatte Deinen Code nicht genau genug gelesen.

Ok, Du hast jetzt im Listener das richtige JPanel-Objekt (group[ tab]) und brauchst jetzt daraus jetzt bestimmte Elemente. Eine Möglichkeit: da Du ja die Reihenfolge steuerst, in der die Components in den Container eingehängt werden, kannst Du sie auch per Index herausbekommen.

In setTabContent werde jawohl (length)-mal jeweils 6 Components eingehängt: im ersten Durchlauf hat dadurch heim[j] den Index 0 und gast[j] den Index 2, im zweiten Durchlauf hat heim[j] den Index 7 und gast[j] den Index 8 usw.


Also könnte das im Listener dann so aussehen:

public void actionPerformed(ActionEvent e) {
               String heim1, gast1, start;
               int length = Integer.parseInt(getColumn("System", "Vorrundenpaarungen")[0]);
               for (int p=0; p<length; p++){
                  JComboBox heim = (JComboBox)(group[idx].get( p*6 + 0));
                  JComboBox gast = (JComboBox)(group[idx].get( p*6 + 1));
                  JLabel time = (JLabel)(group[idx].get( p*6 + 2));
                  JComboBox feld = (JComboBox)(group[idx].get( p*6 + 3));
               }
            }

 

[Spyder]

Hi,
nochmal Danke für deine Hilfe.
Mein Problem lag genau darin, dass ich nicht wusste wie ich auf die
einzelnen Elemente aus den Panels lese.
Die get-Methode die in Java implementiert ist funktioniert leider
nicht, bekomme den Fehler "The method get(int) is undefined for the type JPanel". Habe hier im Forum auch keine Lösung dazu gefunden.

Muss ich die get-Methode selber implementieren, wenn wie muss die aussehen, hab schon was versucht, hat aber nicht geklappt.

Danke für deine Gedult mit mir....


Gruß Christian

 

[Murray]

Sch***, da habe ich Dir Blödsinn geschreiben: die Methode heisst getComponent( int n) und wird von java.awt.Container geerbt.

Sorry!!

 

[Spyder]

Ok,
komisch habe selber nach ner Methode in der Java API gesucht aber nix gefunden, muss ich wohl übersehen habem.

Vielen Dank nochmal, endlich hab ich das lästige Problem gelöst.
Du hast mich ein ganzes Stück weiter gebracht