La multiplication des fenêtres dans Java App

Question

un autre problème. Je voulais faire doubleclick sur JTable qui ouvre une nouvelle fenêtre avec forme. Donc, finalement, je l'ai fait de cette façon:

table.setSelectionMode(ListSelectionModel.SINGLE_SELECTION); 

    table.getSelectionModel().addListSelectionListener(new ListSelectionListener(){ 
     public void valueChanged(ListSelectionEvent event){ 
      int viewRow = table.getSelectedRow(); 
      if(viewRow < 0) 
       System.out.println("LOL"); 
      else{ 
       final int modelRow = table.convertRowIndexToModel(viewRow); 
       table.addMouseListener(new MouseAdapter(){ 
        public void mouseClicked(MouseEvent e){ 
         if(e.getClickCount() == 2) 
          try { 
           new BookForm(); 
          } catch (IOException e1) { 
           e1.printStackTrace(); 
          } 

        } 
       }); 
      } 

     } 
    }); 

Cela fonctionne, mais pas parfait. La première fois que je clique deux fois sur JTable ça ouvre 2 fenêtres (pourquoi pas une?), La prochaine fois ça ouvre 4 fenêtres, ensuite 6 autres fenêtres, etc. Des idées? Peut-être que je devrais avoir à utiliser une méthode différente? Merci pour l'aide!

Answer 1

Prenez une seconde pour regarder par-dessus votre code ...

Chaque fois que les changements de sélection, vous ajoutez une nouvelle MouseListener

table.getSelectionModel().addListSelectionListener(new ListSelectionListener(){ 
    public void valueChanged(ListSelectionEvent event){ 
     int viewRow = table.getSelectedRow(); 
     if(viewRow < 0) 
      System.out.println("LOL"); 
     else{ 
      // You add a new mouse listener... 
      final int modelRow = table.convertRowIndexToModel(viewRow); 
      table.addMouseListener(new MouseAdapter(){ 
       public void mouseClicked(MouseEvent e){ 
        if(e.getClickCount() == 2) 
         try { 
          new BookForm(); 
         } catch (IOException e1) { 
          e1.printStackTrace(); 
         } 

       } 
      }); 
     } 

    } 
}); 

Donc, quand vous « enfin » double-cliquez sur une ligne, vous aura 1-n MouseListener s inscrits sur la table ...

Vous pouvez tout simplement se débarrasser de l'auditeur de sélection et simple ajouter le MouseListener directement à la table ...

table.addMouseListener(new MouseAdapter(){ 
    public void mouseClicked(MouseEvent e){ 
     if(e.getClickCount() == 2) 
      int selectedRow = table.getSelectedRow(); 
      if (selectedRow > -1) { 
       int modelRow = table.convertRowIndexToModel(selectedRow); 
       try { 
        new BookForm(); 
       } catch (IOException e1) { 
        e1.printStackTrace(); 
       } 
      } 
    } 
}); 

Jetez aussi un coup d'œil à The Use of Multiple JFrames: Good or Bad Practice? avant de bombarder votre barde utilisateur avec beaucoup de nouvelles fenêtres ...