Problem mit Mouselistener und jtable

[dennisT]

Ich habe ein Programm, wo ich bei einem JTextField eine Taste drücken kann und sich dann ein JDialog aufbaut mit einer JTable drin. Man kann dort dann doppelklick auf einen Eintrag machen und dieser Eintrag erscheint dann in dem JTF. Das klappt auch ganz gut, aber wenn dich das ein zweites mal machen will bekomme ich einen Fehler. Die Codestelle sieht so aus:
[JAVA=406]jt_hilfstabelle.addMouseListener(new MouseAdapter(){
public void mouseClicked(MouseEvent me){
if (me.getClickCount () == 2) {
hilfseingabeeinheit=(jt_hilfstabelle.getValueAt(jt_hilfstabelle.getSelectedRow(), 1).toString());
System.out.println(hilfseingabeeinheit);
jtf_einheit.setText(hilfseingabeeinheit);
jd_hilfsauswahl.dispose();
}
}
});[/code]

Wenn du diesen Schritt weglasse, kommt kein Fehler mehr, also denke ich, dass hier der Fehler liegt

hilfseingabeeinheit=(jt_hilfstabelle.getValueAt(jt_hilfstabelle.getSelectedRow(), 1).toString());

Der Fehler lautet:

Exception occurred during event dispatching:
java.lang.IndexOutOfBoundsException: Invalid index
	at javax.swing.DefaultRowSorter.convertRowIndexToModel(DefaultRowSorter.java:497)
	at javax.swing.JTable.convertRowIndexToModel(JTable.java:2610)
	at javax.swing.JTable.getValueAt(JTable.java:2685)
	at ...$Lauscher$2$2.mouseClicked(....java:409)
	at java.awt.AWTEventMulticaster.mouseClicked(AWTEventMulticaster.java:253)
	at java.awt.Component.processMouseEvent(Component.java:6266)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
	at java.awt.Component.processEvent(Component.java:6028)
	at java.awt.Container.processEvent(Container.java:2041)
	at java.awt.Component.dispatchEventImpl(Component.java:4630)
	at java.awt.Container.dispatchEventImpl(Container.java:2099)
	at java.awt.Component.dispatchEvent(Component.java:4460)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4574)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4247)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4168)
	at java.awt.Container.dispatchEventImpl(Container.java:2085)
	at java.awt.Window.dispatchEventImpl(Window.java:2478)
	at java.awt.Component.dispatchEvent(Component.java:4460)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:178)
	at java.awt.Dialog$1.run(Dialog.java:1046)
	at java.awt.Dialog$3.run(Dialog.java:1098)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.awt.Dialog.show(Dialog.java:1096)
	at java.awt.Component.show(Component.java:1563)
	at java.awt.Component.setVisible(Component.java:1515)
	at java.awt.Window.setVisible(Window.java:842)
	at java.awt.Dialog.setVisible(Dialog.java:986)
	at ...$Lauscher$2.keyPressed(....java:421)
	at java.awt.Component.processKeyEvent(Component.java:6221)
	at javax.swing.JComponent.processKeyEvent(JComponent.java:2801)
	at java.awt.Component.processEvent(Component.java:6040)
	at java.awt.Container.processEvent(Container.java:2041)
	at java.awt.Component.dispatchEventImpl(Component.java:4630)
	at java.awt.Container.dispatchEventImpl(Container.java:2099)
	at java.awt.Component.dispatchEvent(Component.java:4460)
	at java.awt.KeyboardFocusManager.redispatchEvent(KeyboardFocusManager.java:1850)
	at java.awt.DefaultKeyboardFocusManager.dispatchKeyEvent(DefaultKeyboardFocusManager.java:712)
	at java.awt.DefaultKeyboardFocusManager.preDispatchKeyEvent(DefaultKeyboardFocusManager.java:990)
	at java.awt.DefaultKeyboardFocusManager.typeAheadAssertions(DefaultKeyboardFocusManager.java:855)
	at java.awt.DefaultKeyboardFocusManager.dispatchEvent(DefaultKeyboardFocusManager.java:676)
	at java.awt.Component.dispatchEventImpl(Component.java:4502)
	at java.awt.Container.dispatchEventImpl(Container.java:2099)
	at java.awt.Window.dispatchEventImpl(Window.java:2478)
	at java.awt.Component.dispatchEvent(Component.java:4460)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:178)
	at java.awt.Dialog$1.run(Dialog.java:1046)
	at java.awt.Dialog$3.run(Dialog.java:1098)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.awt.Dialog.show(Dialog.java:1096)
	at java.awt.Component.show(Component.java:1563)
	at java.awt.Component.setVisible(Component.java:1515)
	at java.awt.Window.setVisible(Window.java:842)
	at java.awt.Dialog.setVisible(Dialog.java:986)
	at ...$Lauscher.actionPerformed(....java:437)
	at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
	at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
	at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
	at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
	at javax.swing.AbstractButton.doClick(AbstractButton.java:357)
	at javax.swing.plaf.basic.BasicMenuItemUI.doClick(BasicMenuItemUI.java:1223)
	at javax.swing.plaf.basic.BasicMenuItemUI$Handler.mouseReleased(BasicMenuItemUI.java:1264)
	at java.awt.Component.processMouseEvent(Component.java:6263)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
	at java.awt.Component.processEvent(Component.java:6028)
	at java.awt.Container.processEvent(Container.java:2041)
	at java.awt.Component.dispatchEventImpl(Component.java:4630)
	at java.awt.Container.dispatchEventImpl(Container.java:2099)
	at java.awt.Component.dispatchEvent(Component.java:4460)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4574)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4238)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4168)
	at java.awt.Container.dispatchEventImpl(Container.java:2085)
	at java.awt.Window.dispatchEventImpl(Window.java:2478)
	at java.awt.Component.dispatchEvent(Component.java:4460)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:174)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)

Vielleicht kann mir da jemand helfen?

edit:// Beim zweiten durchlauf gibt getselectedrow -1 aus. Wie kann ich das beheben?

 

[André Uhres]

Beim zweiten Durchlauf gibt getselectedrow -1 aus. Wie kann ich das beheben?

Dein Code funktioniert bei mir. Falls du noch Hilfe brauchst, mach bitte ein kurzes, selbständiges und kompilierbares Beispiel.

 

[dennisT]

Hier ist eine verkürzte Version des Programmes, wo Funktion und Fehler reproduzierbar sind.

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.table.*;
import java.io.*;
import java.text.*;
import java.util.*;
import java.sql.*;
import javax.swing.border.*;

public class javaforum extends JFrame {

  Container cp;
  Color hintergrundfarbe;
  SimpleDateFormat sdf;
  JMenu jm_uhr;

  private static javaforum fenster = new javaforum("Fürs Java-Forum");

  JTextField jtf_name;
  JTextField jtf_einheit;
  
  JTable jt_tabelle;
  JTable jt_hilfstabelle;

  String hilfseingabeeinheit="";

  JPanel jp_center = new JPanel();
  JPanel jp_south = new JPanel();
  JPanel jp_anlegen = new JPanel();
  
  JMenuItem jmi_anlegen = new JMenuItem();
  
  JMenuBar jmb = new JMenuBar();
  
  JDialog jd_eingabe = new JDialog(this, true);
  JDialog jd_hilfsauswahl = new JDialog(this, true);
  
  DefaultTableModel tabelle;
  DefaultTableModel hilfstabelle;
  
  int zeile = 0;
  int neu = 0;
  String[][] row = new String[1000][3];
  
  String rowData[][] = {};

  String  columnNames[]={"Typ", "Einheit"};

  public javaforum (String title) {
    super(title);
    cp=getContentPane();
    hintergrundfarbe = cp.getBackground();
    Lauscher lausch = new Lauscher();
    sdf = new SimpleDateFormat("HH:mm:ss");

    ///////////////////////////////////MENU/////////////////////////////////////
    
    JMenu jm_daten = new JMenu("Daten");

              jmi_anlegen = new JMenuItem("Anlegen");
              jmi_anlegen.addActionListener(lausch);

              jm_daten.add(jmi_anlegen);

              jmb.add(jm_daten);
              
              setJMenuBar(jmb);
    ////////////////////////////////////////////////////////////////////////////

    treiber_laden();
    
          jd_eingabe.setLayout(new BorderLayout());
          jd_eingabe.setSize(250,300);
          jd_eingabe.setLocation(400,200);
          
          jd_hilfsauswahl.setLayout(new BorderLayout());
          jd_hilfsauswahl.setSize(250,300);
          jd_hilfsauswahl.setLocation(400,200);
    
    JLabel jl_name = new JLabel ("Name:",JLabel.LEFT);
    JLabel jl_einheit = new JLabel ("Einheit:",JLabel.LEFT);

    jtf_name = new JTextField(15);
    jtf_einheit = new JTextField(15);
    

           
           jp_anlegen.setBorder(new TitledBorder("Neue Daten"));
           jp_anlegen.setLayout(new GridLayout(2,2));
           jp_anlegen.add(jl_name);
           jp_anlegen.add(jtf_name);
           jp_anlegen.add(jl_einheit);
           jp_anlegen.add(jtf_einheit);

    cp.add(jp_center,"Center");
    cp.add(jp_south,"South");

    this.setDefaultCloseOperation(EXIT_ON_CLOSE);
  }

  class Lauscher implements ActionListener {

    public void actionPerformed(ActionEvent e) {
      
      if( e.getSource() == jmi_anlegen){

        jtf_name.setText("");
        jtf_einheit.setText("");
        jtf_einheit.addKeyListener(new KeyAdapter(){
          public void keyPressed(KeyEvent e) {
            int k = e.getKeyCode();
            if(k==120){


              File tabledata = new File("table.txt");
              if(tabledata.exists()){
                try{
                  Scanner s_table = new Scanner(tabledata);
                  zeile = s_table.nextInt();
                  rowData = new String[zeile][2];
                  for(int i=0;i<zeile;i++){
                    for(int j=0;j<2;j++){
                      rowData[i][j] = s_table.next();
                    }
                  }
                  s_table.close();
                }
                catch(Exception ex){}
              }


                hilfstabelle = new DefaultTableModel(rowData, columnNames){
                  public boolean isCellEditable(int row, int column){
                    return false;
                  }
                };

              jt_hilfstabelle = new JTable(hilfstabelle);
              jt_hilfstabelle.setAutoCreateRowSorter(true);

              jt_hilfstabelle.addMouseListener(new MouseAdapter(){
                public void mouseClicked(MouseEvent me){
                  if (me.getClickCount () == 2) {

                    System.out.println(jt_hilfstabelle.getSelectedRow());
                    hilfseingabeeinheit=(jt_hilfstabelle.getValueAt(jt_hilfstabelle.getSelectedRow(), 1).toString());
                    System.out.println(hilfseingabeeinheit);
                    jtf_einheit.setText(hilfseingabeeinheit);
                    jd_hilfsauswahl.dispose();
                  }
                }
              });

              JScrollPane sp = new JScrollPane();
              sp.getViewport().add(jt_hilfstabelle);
              sp.setBorder(new TitledBorder("Einheiten"));
              jd_hilfsauswahl.add(sp);
              jd_hilfsauswahl.setVisible(true);
            }
          }
        });
        
        jd_eingabe.add(jp_anlegen,"North");
        jd_eingabe.repaint();
        jd_eingabe.setVisible(true);
      }
    }
  }

    public static void treiber_laden() {
      try {
        Class.forName("com.mysql.jdbc.Driver");
      }
      catch (Exception e) {
        System.out.println(e.getMessage());
      }
    }

    public static void main(String[] args) throws Exception {

      fenster.setSize(1025,730);
      fenster.setLocation(000,000);
      fenster.setExtendedState(JFrame.MAXIMIZED_BOTH);
      fenster.setVisible(true);
    }
  }

Hier für die TXT-Datei table.txt zum test:

3
1
g
1
kg
1
t

Wär echt super wenn ihr mir da helfen könntet.

// Mein richtiges Programm läuft über mysql aber das sollte im Moment egal sein.

 

[André Uhres]

Wenn du im Einheitendialog die Scrollpane ersetzen willst, musst du die alte zuerst wegräumen, etwa so:

jd_hilfsauswahl.getContentPane().removeAll();//<--------------------------
jd_hilfsauswahl.add(sp);

 

[dennisT]

Funktioniert, danke dir

edit:// Problem noch nicht ganz gelöst, wenn ich jetzt jd_eingabe schließe, neu öffne und wieder mit F9 jd_hilfseingabe öffne, öffnet sich beim 2. mal das jd_hilfseingabe nochmal nachdem ich auf einen eintrag doppelklick gemacht hab, beim dritten öffnen, passiert das ganze 3mal und so weiter. Woran kanns liegen?

edit2:// Hat sich erledigt, der keylistener wurde immer wieder neu hinzugefügt.

 

[dennisT]

Ich hoffe es ist OK wenn ich einen Doppelpost mache, besser als nen neuen Thread

Hier mal das Programm nochmal:

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.table.*;
import java.io.*;
import java.text.*;
import java.util.*;
import java.sql.*;
import javax.swing.border.*;

public class javaforum extends JFrame {

  Container cp;
  Color hintergrundfarbe;

  private static javaforum fenster = new javaforum("Fürs Java-Forum");

  JTextField jtf_name;
  JTextField jtf_einheit;
  
  JTable jt_tabelle;
  JTable jt_hilfstabelle;

  String hilfseingabeeinheit="";

  JPanel jp_center = new JPanel();
  JPanel jp_south = new JPanel();
  JPanel jp_anlegen = new JPanel();
  
  JMenuItem jmi_anlegen = new JMenuItem();
  
  JMenuBar jmb = new JMenuBar();
  
  JDialog jd_eingabe = new JDialog(this, true);
  JDialog jd_hilfsauswahl = new JDialog(this, true);
  
  DefaultTableModel tabelle;
  DefaultTableModel hilfstabelle;
  
  int zeile = 0;
  int neu = 0;
  String[][] row = new String[1000][3];
  
  String rowData[][] = {};

  String  columnNames[]={"Typ", "Einheit"};

  public javaforum (String title) {
    super(title);
    cp=getContentPane();
    hintergrundfarbe = cp.getBackground();
    Lauscher lausch = new Lauscher();

    ///////////////////////////////////MENU/////////////////////////////////////
    
    JMenu jm_daten = new JMenu("Daten");

              jmi_anlegen = new JMenuItem("Anlegen");
              jmi_anlegen.addActionListener(lausch);

              jm_daten.add(jmi_anlegen);

              jmb.add(jm_daten);
              
              setJMenuBar(jmb);
    ////////////////////////////////////////////////////////////////////////////

          jd_eingabe.setLayout(new BorderLayout());
          jd_eingabe.setSize(250,300);
          jd_eingabe.setLocation(400,200);
          
          jd_hilfsauswahl.setLayout(new BorderLayout());
          jd_hilfsauswahl.setSize(250,300);
          jd_hilfsauswahl.setLocation(400,200);
    
    JLabel jl_name = new JLabel ("Name:",JLabel.LEFT);
    JLabel jl_einheit = new JLabel ("Einheit:",JLabel.LEFT);

    jtf_name = new JTextField(15);
    jtf_einheit = new JTextField(15);
    
           jtf_einheit.addKeyListener(new MyKeyListener());
           
           jp_anlegen.setBorder(new TitledBorder("Neue Daten"));
           jp_anlegen.setLayout(new GridLayout(2,2));
           jp_anlegen.add(jl_name);
           jp_anlegen.add(jtf_name);
           jp_anlegen.add(jl_einheit);
           jp_anlegen.add(jtf_einheit);

    cp.add(jp_center,"Center");
    cp.add(jp_south,"South");
    
    //jt_hilfstabelle.addMouseListener(new MyMouseListener());

    this.setDefaultCloseOperation(EXIT_ON_CLOSE);
  }

  class Lauscher implements ActionListener {

    public void actionPerformed(ActionEvent e) {
      
      if( e.getSource() == jmi_anlegen){

        jtf_name.setText("");
        jtf_einheit.setText("");
        
        jd_eingabe.add(jp_anlegen,"North");
        jd_eingabe.repaint();
        jd_eingabe.setVisible(true);
      }
    }
  }


    public static void main(String[] args) throws Exception {

      fenster.setSize(1025,730);
      fenster.setLocation(000,000);
      fenster.setExtendedState(JFrame.MAXIMIZED_BOTH);
      fenster.setVisible(true);
    }
    
    class MyKeyListener  extends KeyAdapter {
       public void keyPressed(KeyEvent e)
       {
          int k = e.getKeyCode();
            if(k==120){

              File tabledata = new File("table.txt");
              if(tabledata.exists()){
                try{
                  Scanner s_table = new Scanner(tabledata);
                  zeile = s_table.nextInt();
                  rowData = new String[zeile][2];
                  for(int i=0;i<zeile;i++){
                    for(int j=0;j<2;j++){
                      rowData[i][j] = s_table.next();
                    }
                  }
                  s_table.close();
                }
                catch(Exception ex){}
              }


                hilfstabelle = new DefaultTableModel(rowData, columnNames){
                  public boolean isCellEditable(int row, int column){
                    return false;
                  }
                };

              jt_hilfstabelle = new JTable(hilfstabelle);
              jt_hilfstabelle.setAutoCreateRowSorter(true);

              //jt_hilfstabelle.addMouseListener(new MyMouseListener());

              JScrollPane sp = new JScrollPane();
              sp.getViewport().add(jt_hilfstabelle);
              sp.setBorder(new TitledBorder("Einheiten"));
              jd_hilfsauswahl.getContentPane().removeAll();
              jd_hilfsauswahl.add(sp);
              jd_hilfsauswahl.setVisible(true);

            }
       }
    }
    
    class MyMouseListener extends MouseAdapter{
      public void mouseClicked(MouseEvent me){
        if (me.getClickCount () == 2) {

                    System.out.println(jt_hilfstabelle.getSelectedRow());
                    hilfseingabeeinheit=(jt_hilfstabelle.getValueAt(jt_hilfstabelle.getSelectedRow(), 1).toString());
                    System.out.println(hilfseingabeeinheit);
                    jtf_einheit.setText(hilfseingabeeinheit);
                    jd_hilfsauswahl.dispose();
                    
        }
      }
    }
    
  }

Problem diesmal findet ihr bei den "//jt_hilfstabelle.addMouseListener(new MyMouseListener());"

Ich bekomem es nicht hin, dass es richtig funktioniert. Wenn ich den Mouselistener innerhalb vom Keylistener adde funktioniert alles. Wenn ich es ausserhalb vom Keylistener adde, funktioniert es nichtmehr, oder die Tabelle wird nichtmehr angezeigt im JDialog. Ich kann es ja auch nicht im Keylistener lassen, weil sonst immer wieder der MouseListener hinzugefügt wird. ???:L

Hier nochmal für die Leute die es testen wollen, die table.txt.

3
1
g
1
kg
1
t

Danke schonmal.

 

[André Uhres]

Funktioniert bei mir wenn ich es im Keylistener lasse. Das ist eigentlich auch logisch, denn die alten Mouselistener werden ja mit [c]jd_hilfsauswahl.getContentPane().removeAll();[/c] abgehängt .

 

[dennisT]

Achso okay, danke =)